[PATCH] isdn/gigaset: avoid cs->dev null pointer dereference
authorAkinobu Mita <akinobu.mita@gmail.com>
Mon, 30 Oct 2006 06:46:35 +0000 (22:46 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 30 Oct 2006 20:08:40 +0000 (12:08 -0800)
When gigaset_initbcs() is called, cs->dev is not initialized yet.  If
dev_alloc_skb() failed in this function, NULL poinster dereference will
happen at dev_warn().

Cc: Kai Germaschewski <kai.germaschewski@gmx.de>
Cc: Hansjoerg Lipp <hjlipp@web.de>
Cc: Tilman Schmidt <tilman@imap.cc>
Acked-by: Karsten Keil <kkeil@suse.de>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
drivers/isdn/gigaset/common.c

index aca165d43aa050952dbaaac63e34241cbfd10dd7..d8d256dadddf85cc37d51ab0508bc5dd4e5006e5 100644 (file)
@@ -616,7 +616,7 @@ static struct bc_state *gigaset_initbcs(struct bc_state *bcs,
        } else if ((bcs->skb = dev_alloc_skb(SBUFSIZE + HW_HDR_LEN)) != NULL)
                skb_reserve(bcs->skb, HW_HDR_LEN);
        else {
-               dev_warn(cs->dev, "could not allocate skb\n");
+               gig_dbg(DEBUG_INIT, "could not allocate skb\n");
                bcs->inputstate |= INS_skip_frame;
        }