Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > The "ev->traffic_class" and "reply->ac" variables come from the network > and they're used as an offset into the wmi->stream_exist_for_ac[] array. > Those variables are u8 so they can be 0-255 but the stream_exist_for_ac[] > array only has WMM_NUM_AC (4) elements. We need to add a couple bounds > checks to prevent array overflows. > > I also modified one existing check from "if (traffic_class > 3) {" to > "if (traffic_class >= WMM_NUM_AC) {" just to make them all consistent. > > Fixes: bdcd81707973 (" Add ath6kl cleaned up driver") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 5d6751eaff67 ath6kl: add some bounds checking -- https://patchwork.kernel.org/patch/10885299/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches