diff options
| author | Ben Hutchings <ben@decadent.org.uk> | 2013-03-08 12:43:32 -0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-03-14 11:29:51 -0700 |
| commit | d2c96b7257048e9a558f2ebe4fe884b51fd9016f (patch) | |
| tree | ef4f78def19091c0ece2c45daad02fb99222072a /drivers/usb/core/hub.c | |
| parent | abf73cb191a83045c3cc6f46ea78b94439957585 (diff) | |
| download | lwn-d2c96b7257048e9a558f2ebe4fe884b51fd9016f.tar.gz lwn-d2c96b7257048e9a558f2ebe4fe884b51fd9016f.zip | |
dmi_scan: fix missing check for _DMI_ signature in smbios_present()
commit a40e7cf8f06b4e322ba902e4e9f6a6b0c2daa907 upstream.
Commit 9f9c9cbb6057 ("drivers/firmware/dmi_scan.c: fetch dmi version
from SMBIOS if it exists") hoisted the check for "_DMI_" into
dmi_scan_machine(), which means that we don't bother to check for
"_DMI_" at offset 16 in an SMBIOS entry. smbios_present() may also call
dmi_present() for an address where we found "_SM_", if it failed further
validation.
Check for "_DMI_" in smbios_present() before calling dmi_present().
[akpm@linux-foundation.org: fix build]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Reported-by: Tim McGrath <tmhikaru@gmail.com>
Tested-by: Tim Mcgrath <tmhikaru@gmail.com>
Cc: Zhenzhong Duan <zhenzhong.duan@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/core/hub.c')
0 files changed, 0 insertions, 0 deletions