On Wed, May 05, 2010 at 02:56:52PM +0800, wzt.wzt@xxxxxxxxx wrote: > Check kmalloc return value before use the buffer. > > Signed-off-by: Zhitong Wang <zhitong.wangzt@xxxxxxxxxxxxxxx> > > --- > .../staging/rtl8192u/ieee80211/ieee80211_softmac.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > index 27d9257..8718ac9 100644 > --- a/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > +++ b/drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c > @@ -1579,6 +1579,8 @@ static inline u16 auth_parse(struct sk_buff *skb, u8** challenge, int *chlen) > if(*(t++) == MFIE_TYPE_CHALLENGE){ > *chlen = *(t++); > *challenge = (u8*)kmalloc(*chlen, GFP_ATOMIC); > + if (!*challenge) > + return -ENOMEM; Great catch! thanks for the patch, now queued up. greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel