From: Dan Carpenter Date: Wed, 27 Feb 2013 05:13:45 +0000 (+0300) Subject: wlan-ng: add a bounds check X-Git-Tag: upstream/snapshot3+hdmi~5290^2~751 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=72c1c06d91c54beee35613434a5efdd7e8909302;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git wlan-ng: add a bounds check I'm not sure where these results come from, but it can't hurt to add a sanity check the array offset. The .results[] array on the next line has HFA384x_CHINFORESULT_MAX (16) elements. Signed-off-by: Dan Carpenter Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c index dc221f2..428a9be 100644 --- a/drivers/staging/wlan-ng/prism2sta.c +++ b/drivers/staging/wlan-ng/prism2sta.c @@ -1170,6 +1170,9 @@ static void prism2sta_inf_chinforesults(wlandevice_t *wlandev, result = &inf->info.chinforesult.result[n]; chan = le16_to_cpu(result->chid) - 1; + if (chan < 0 || chan >= HFA384x_CHINFORESULT_MAX) + continue; + chinforesult = &hw->channel_info.results.result[chan]; chinforesult->chid = chan; chinforesult->anl = le16_to_cpu(result->anl);