[PATCH 2/2] SCTP: fix wrong debug counting of bind_bucket - Kernel

This is a discussion on [PATCH 2/2] SCTP: fix wrong debug counting of bind_bucket - Kernel ; Should not count it if the allocation of the object is failed. Signed-off-by: Li Zefan --- net/sctp/socket.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/sctp/socket.c b/net/sctp/socket.c index d994d82..698b741 100644 --- a/net/sctp/socket.c +++ b/net/sctp/socket.c @@ -5761,8 ...

+ Reply to Thread
Results 1 to 2 of 2

Thread: [PATCH 2/2] SCTP: fix wrong debug counting of bind_bucket

  1. [PATCH 2/2] SCTP: fix wrong debug counting of bind_bucket

    Should not count it if the allocation of the object
    is failed.

    Signed-off-by: Li Zefan
    ---
    net/sctp/socket.c | 2 +-
    1 files changed, 1 insertions(+), 1 deletions(-)

    diff --git a/net/sctp/socket.c b/net/sctp/socket.c
    index d994d82..698b741 100644
    --- a/net/sctp/socket.c
    +++ b/net/sctp/socket.c
    @@ -5761,8 +5761,8 @@ static struct sctp_bind_bucket *sctp_bucket_create(
    struct sctp_bind_bucket *pp;

    pp = kmem_cache_alloc(sctp_bucket_cachep, GFP_ATOMIC);
    - SCTP_DBG_OBJCNT_INC(bind_bucket);
    if (pp) {
    + SCTP_DBG_OBJCNT_INC(bind_bucket);
    pp->port = snum;
    pp->fastreuse = 0;
    INIT_HLIST_HEAD(&pp->owner);
    --
    1.5.4.rc3

    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  2. Re: [PATCH 2/2] SCTP: fix wrong debug counting of bind_bucket

    From: Li Zefan
    Date: Tue, 01 Apr 2008 11:42:12 +0800

    > Should not count it if the allocation of the object
    > is failed.
    >
    > Signed-off-by: Li Zefan


    Also applied to net-2.6.26, thanks.
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

+ Reply to Thread