On 07/16/2011 01:49 AM, Dan Carpenter wrote:
On Thu, Jul 14, 2011 at 02:29:31PM -0700, Franky Lin wrote:
--- a/drivers/staging/brcm80211/brcmfmac/dhd_common.c
+++ b/drivers/staging/brcm80211/brcmfmac/dhd_common.c
@@ -827,10 +827,13 @@ static int brcmf_c_pattern_atoh(char *src, char *dst)
return -EINVAL;
}
for (i = 0; *src != '\0'; i++) {
+ unsigned long res;
char num[3];
strncpy(num, src, 2);
num[2] = '\0';
- dst[i] = (u8) simple_strtoul(num, NULL, 16);
+ if (kstrtoul(num, 16,&res))
+ return -1;
^^
Not a proper error code.
Hi Dan,
You are right. I actually had this patch before patch 21, which fixed
the other error paths in the function. This slipped through during
reordering of the patch. Will fix it.
Greg, Please drop this one.
Gr. AvS
--
Almost nobody dances sober, unless they happen to be insane.
-- H.P. Lovecraft --
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
