summaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/zd1201.c
diff options
context:
space:
mode:
authorEric Sesterhenn <snakebyte@gmx.de>2006-10-10 14:45:45 -0700
committerJohn W. Linville <linville@tuxdriver.com>2006-10-16 20:09:48 -0400
commit683f8c9e00d2aa911382186ca891bd221efaea74 (patch)
treefd93a36b300ed8ce9025b09624333c839375c932 /drivers/net/wireless/zd1201.c
parent7e4e8d99c2288a490a0806b9cb40016913312cfe (diff)
downloadop-kernel-dev-683f8c9e00d2aa911382186ca891bd221efaea74.zip
op-kernel-dev-683f8c9e00d2aa911382186ca891bd221efaea74.tar.gz
[PATCH] zd1201: Possible NULL dereference
If we enter the if(!zd) and set free to 1, we dereference zd in the exit code. Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/zd1201.c')
-rw-r--r--drivers/net/wireless/zd1201.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/wireless/zd1201.c b/drivers/net/wireless/zd1201.c
index 30057a3..36b29ff 100644
--- a/drivers/net/wireless/zd1201.c
+++ b/drivers/net/wireless/zd1201.c
@@ -193,10 +193,8 @@ static void zd1201_usbrx(struct urb *urb)
struct sk_buff *skb;
unsigned char type;
- if (!zd) {
- free = 1;
- goto exit;
- }
+ if (!zd)
+ return;
switch(urb->status) {
case -EILSEQ:
OpenPOWER on IntegriCloud