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

This is a discussion on [PATCH 1/2] SCTP: fix wrong debug counting of datamsg - Kernel ; Should not count it if the allocation of this object failed. Signed-off-by: Li Zefan --- net/sctp/chunk.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c index 4d3128f..099b0af 100644 --- a/net/sctp/chunk.c +++ b/net/sctp/chunk.c @@ -66,9 +66,10 ...

+ Reply to Thread
Results 1 to 2 of 2

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

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

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

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

    diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
    index 4d3128f..099b0af 100644
    --- a/net/sctp/chunk.c
    +++ b/net/sctp/chunk.c
    @@ -66,9 +66,10 @@ SCTP_STATIC struct sctp_datamsg *sctp_datamsg_new(gfp_t gfp)
    {
    struct sctp_datamsg *msg;
    msg = kmalloc(sizeof(struct sctp_datamsg), gfp);
    - if (msg)
    + if (msg) {
    sctp_datamsg_init(msg);
    - SCTP_DBG_OBJCNT_INC(datamsg);
    + SCTP_DBG_OBJCNT_INC(datamsg);
    + }
    return msg;
    }

    --
    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 1/2] SCTP: fix wrong debug counting of datamsg

    From: Li Zefan
    Date: Tue, 01 Apr 2008 11:41:15 +0800

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


    This is a bug fix, but for a debugging feature, so I
    have applied it to net-2.6.26
    --
    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