[ICMP]: Add return code to icmp_init.
authorDenis V. Lunev <den@openvz.org>
Fri, 29 Feb 2008 19:14:50 +0000 (11:14 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 29 Feb 2008 19:14:50 +0000 (11:14 -0800)
icmp_init could fail and this is normal for namespace other than initial.
So, the panic should be triggered only on init_net initialization path.

Additionally create rollback path for icmp_init as a separate function.
It will also be used later during namespace destruction.

Signed-off-by: Denis V. Lunev <den@openvz.org>
Acked-by: Daniel Lezcano <dlezcano@fr.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/icmp.h
net/ipv4/af_inet.c
net/ipv4/icmp.c

index 7bf714d9d7c7834cae35f80b0dfdab8252158f45..faba64db8ff057cd95be91d439f6a7a9d92d9c54 100644 (file)
@@ -48,7 +48,7 @@ struct sk_buff;
 extern void    icmp_send(struct sk_buff *skb_in,  int type, int code, __be32 info);
 extern int     icmp_rcv(struct sk_buff *skb);
 extern int     icmp_ioctl(struct sock *sk, int cmd, unsigned long arg);
-extern void    icmp_init(void);
+extern int     icmp_init(void);
 extern void    icmp_out_count(unsigned char type);
 
 /* Move into dst.h ? */
index a7a99ac856dc231ef11a6b24173a9be7a8145904..4f539bd48718eb0d26c692866a8d3f429fcbc5a6 100644 (file)
@@ -1430,7 +1430,8 @@ static int __init inet_init(void)
         *      Set the ICMP layer up
         */
 
-       icmp_init();
+       if (icmp_init() < 0)
+               panic("Failed to create the ICMP control socket.\n");
 
        /*
         *      Initialise the multicast router
index 98372db66c6698f498f6197e78a748120853a4c7..b345b3d9bcaafa15b7462de3c2bc9045c795d0e8 100644 (file)
@@ -1139,19 +1139,32 @@ static const struct icmp_control icmp_pointers[NR_ICMP_TYPES + 1] = {
        },
 };
 
-void __init icmp_init(void)
+static void __exit icmp_exit(void)
 {
-       struct inet_sock *inet;
        int i;
 
        for_each_possible_cpu(i) {
-               int err;
+               struct socket *sock;
+
+               sock = per_cpu(__icmp_socket, i);
+               if (sock == NULL)
+                       continue;
+               per_cpu(__icmp_socket, i) = NULL;
+               sock_release(sock);
+       }
+}
 
+int __init icmp_init(void)
+{
+       struct inet_sock *inet;
+       int i, err;
+
+       for_each_possible_cpu(i) {
                err = sock_create_kern(PF_INET, SOCK_RAW, IPPROTO_ICMP,
                                       &per_cpu(__icmp_socket, i));
 
                if (err < 0)
-                       panic("Failed to create the ICMP control socket.\n");
+                       goto fail;
 
                per_cpu(__icmp_socket, i)->sk->sk_allocation = GFP_ATOMIC;
 
@@ -1171,6 +1184,11 @@ void __init icmp_init(void)
                 */
                per_cpu(__icmp_socket, i)->sk->sk_prot->unhash(per_cpu(__icmp_socket, i)->sk);
        }
+       return 0;
+
+fail:
+       icmp_exit();
+       return err;
 }
 
 EXPORT_SYMBOL(icmp_err_convert);