Skip to content

Commit

Permalink
staging: rtl8192e: Fix potential null dereference
Browse files Browse the repository at this point in the history
check memory is allocated in '*challenge' before using it.

Signed-off-by: Hema Prathaban <hemaklnce@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Hema Prathaban authored and Greg Kroah-Hartman committed May 15, 2013
1 parent 95d8ee5 commit ae05325
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion drivers/staging/rtl8192e/rtllib_softmac.c
Original file line number Diff line number Diff line change
Expand Up @@ -1802,7 +1802,9 @@ static inline u16 auth_parse(struct sk_buff *skb, u8** challenge, int *chlen)
if (*(t++) == MFIE_TYPE_CHALLENGE) {
*chlen = *(t++);
*challenge = kmalloc(*chlen, GFP_ATOMIC);
memcpy(*challenge, t, *chlen); /*TODO - check here*/
if (!*challenge)
return -ENOMEM;
memcpy(*challenge, t, *chlen);
}
}
return cpu_to_le16(a->status);
Expand Down

0 comments on commit ae05325

Please sign in to comment.