On Sun, 2010-10-10 at 14:07 -0400, Jason Cooper wrote:
> +++ b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
[]
> @@ -1280,9 +1280,9 @@ wl_iw_set_scan(struct net_device *dev,
> __func__, req->essid));
> return -EBUSY;
> } else {
> - g_specific_ssid.SSID_len =
> - min(sizeof(g_specific_ssid.SSID),
> - req->essid_len);
> + g_specific_ssid.SSID_len = min_t(unsigned char,
> + sizeof(g_specific_ssid.SSID),
> + req->essid_len);
You might be hiding truncation here.
It might be better to use
min_t(size_t, sizeof(g_specific_ssid.SSID), req->essid_len);
here and below.
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
