netdev
[Top] [All Lists]

Re: [PATCH] Propagate error from sctp_proc_init

To: Olaf Kirch <okir@xxxxxxx>
Subject: Re: [PATCH] Propagate error from sctp_proc_init
From: Sridhar Samudrala <sri@xxxxxxxxxx>
Date: Wed, 14 Apr 2004 15:09:47 -0700 (PDT)
Cc: netdev@xxxxxxxxxxx, lksctp-developers@xxxxxxxxxxxxxxxxxxxxx
In-reply-to: <20040414134148.GC16591@xxxxxxx>
References: <20040414134148.GC16591@xxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
The patch looks fine. Dave, could you apply it both 2.6 and 2.4 trees.
It should apply to both of them with a little fuzz.

Thanks
Sridhar

On Wed, 14 Apr 2004, Olaf Kirch wrote:

> The following patch makes sure that an error in sctp_proc_init()
> causes module initialization to fail.
>
> Olaf
> --
> Olaf Kirch     |  The Hardware Gods hate me.
> okir@xxxxxxx   |
> ---------------+
>

--- linux-2.6.4/net/sctp/protocol.c.initfail    2004-02-19 11:36:37.000000000 
+0100
+++ linux-2.6.4/net/sctp/protocol.c     2004-04-14 15:36:19.000000000 +0200
@@ -1001,7 +1001,9 @@
                goto err_init_mibs;

        /* Initialize proc fs directory.  */
-       sctp_proc_init();
+       status = sctp_proc_init();
+       if (status)
+               goto err_init_proc;

        /* Initialize object count debugging.  */
        sctp_dbg_objcnt_init();
@@ -1165,6 +1167,7 @@
                             sizeof(struct sctp_hashbucket)));
 err_ahash_alloc:
        sctp_dbg_objcnt_exit();
+err_init_proc:
        sctp_proc_exit();
        cleanup_sctp_mibs();
 err_init_mibs:

<Prev in Thread] Current Thread [Next in Thread>