On Wed, Jun 11, 2014 at 11:30 AM, Luis R. Rodriguez <mcgrof@xxxxxxxxxxxxxxxx> wrote: > On Wed, Jun 11, 2014 at 12:23 AM, Arik Nemtsov <arik@xxxxxxxxxx> wrote: >> From: Eliad Peller <eliad@xxxxxxxxxx> >> >> alpha2 is defined as 2-chars array, but is used in multiple >> places as string (e.g. with nla_put_string calls), which >> might leak kernel data. >> >> Solve it by simply adding an extra char for the NULL >> terminator, making such operations safe. >> >> Signed-off-by: Eliad Peller <eliadx.peller@xxxxxxxxx> >> Reviewed-by: Johannes Berg <johannes.berg@xxxxxxxxx> > > Acked-by: Luis R. Rodriguez <mcgrof@xxxxxxxxxxxxxxxx> > > Thanks for the patch ! Curious how you ended up down finding the issue though. > i'm not sure. i think i noticed some crap in hostapd log and looked for it. Eliad. -- 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
