wext: fix potential private ioctl memory content leak
authorJohannes Berg <johannes.berg@intel.com>
Thu, 16 Sep 2010 22:38:25 +0000 (00:38 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 29 Oct 2010 04:04:15 +0000 (21:04 -0700)
commit45d787b8a946313b73e8a8fc5d501c9aea3d8847
tree1f4271a35521bc748ecb0d81a05f66c95c7b78d8
parent1aa14af44cc76d3e38ed4a0b321cab7323a59452
wext: fix potential private ioctl memory content leak

commit df6d02300f7c2fbd0fbe626d819c8e5237d72c62 upstream.

When a driver doesn't fill the entire buffer, old
heap contents may remain, and if it also doesn't
update the length properly, this old heap content
will be copied back to userspace.

It is very unlikely that this happens in any of
the drivers using private ioctls since it would
show up as junk being reported by iwpriv, but it
seems better to be safe here, so use kzalloc.

Reported-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
net/wireless/wext.c