diff options
author | Xiaochen Wang <wangxiaochen0@gmail.com> | 2011-03-06 09:04:15 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-03-07 16:46:38 -0500 |
commit | 9603ff50b5d56e6ee64b10116ff640320732f9c8 (patch) | |
tree | 0e64dc27957623c7993c8d3769e312837de1e71c /drivers/staging/rtl8192e/ieee80211 | |
parent | 0875abf83df7d74bf9858c125e82835bd1ca349c (diff) |
staging: rtl8192e use kmemdup and check its return value
use kmemdup instead of kmalloc and memcpy,
and check its return value
Signed-off-by: Xiaochen Wang <wangxiaochen0@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/rtl8192e/ieee80211')
-rw-r--r-- | drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c index 88a9cd1958a..fc96676bb9c 100644 --- a/drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c +++ b/drivers/staging/rtl8192e/ieee80211/ieee80211_softmac.c | |||
@@ -1564,8 +1564,9 @@ static inline u16 auth_parse(struct sk_buff *skb, u8** challenge, int *chlen) | |||
1564 | 1564 | ||
1565 | if(*(t++) == MFIE_TYPE_CHALLENGE){ | 1565 | if(*(t++) == MFIE_TYPE_CHALLENGE){ |
1566 | *chlen = *(t++); | 1566 | *chlen = *(t++); |
1567 | *challenge = kmalloc(*chlen, GFP_ATOMIC); | 1567 | *challenge = kmemdup(t, *chlen, GFP_ATOMIC); |
1568 | memcpy(*challenge, t, *chlen); | 1568 | if (!*challenge) |
1569 | return -ENOMEM; | ||
1569 | } | 1570 | } |
1570 | } | 1571 | } |
1571 | 1572 | ||