]> rtime.felk.cvut.cz Git - lisovros/linux_canprio.git/commit
wmi: use memcmp instead of strncmp to compare GUIDs
authorThadeu Lima de Souza Cascardo <cascardo@holoscopio.com>
Sun, 28 Nov 2010 21:46:50 +0000 (19:46 -0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 9 Dec 2010 21:33:26 +0000 (13:33 -0800)
commit2dedf88ded12897aa394f09aa675038d96d79670
tree9def452d9c5d70e57d82335c43dd8863ea5caf89
parent53e87163a135b1c868f31327c7f0b34feb605506
wmi: use memcmp instead of strncmp to compare GUIDs

commit 8b14d7b22c61f17ccb869e0047d9df6dd9f50a9f upstream.

While looking for the duplicates in /sys/class/wmi/, I couldn't find
them. The code that looks for duplicates uses strncmp in a binary GUID,
which may contain zero bytes. The right function is memcmp, which is
also used in another section of wmi code.

It was finding 49142400-C6A3-40FA-BADB-8A2652834100 as a duplicate of
39142400-C6A3-40FA-BADB-8A2652834100. Since the first byte is the fourth
printed, they were found as equal by strncmp.

Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@holoscopio.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/platform/x86/wmi.c