| commit | b4f17a488ae2e09bfcf95c0e0b4219c246f1116a | |
| author | Hans de Goede <hdegoede@redhat.com> | |
| Sat, 3 Aug 2013 14:37:48 +0000 (3 16:37 +0200) | ||
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
| Mon, 12 Aug 2013 19:24:27 +0000 (12 12:24 -0700) | ||
| tree | 08e6bf805e5066d34a9b4d5176a79bbfce1b70ce | treesnapshot (tar.gz zip) |
| parent | 3f0d1c67fa20d524fdcb4a7dcda19ed76c59be44 | commitdiff |
usb: config->desc.bLength may not exceed amount of data returned by the device
While reading the config parsing code I noticed this check is missing, without
this check config->desc.wTotalLength can end up with a value larger then the
dev->rawdescriptors length for the config, and when userspace then tries to
get the rawdescriptors bad things may happen.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
While reading the config parsing code I noticed this check is missing, without
this check config->desc.wTotalLength can end up with a value larger then the
dev->rawdescriptors length for the config, and when userspace then tries to
get the rawdescriptors bad things may happen.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| drivers/usb/core/config.c | diffblobblamehistory |