modpost: fix segfault in sym_is() with prefixed arches
authorMike Frysinger <vapier@gentoo.org>
Sat, 16 Jan 2010 21:57:34 +0000 (07:57 +1030)
committerLinus Torvalds <torvalds@linux-foundation.org>
Sun, 17 Jan 2010 19:00:53 +0000 (11:00 -0800)
commit3a5dd791abef032fe57fc652c0232913c696e59b
tree6bf20b6d67d45bf84730c3ce19b9514948a31f34
parent6ccf80eb15ccaca4d3f1ab5162b9ded5eecd9971
modpost: fix segfault in sym_is() with prefixed arches

The sym_is() compares a symbol in an attempt to automatically skip symbol
prefixes.  It does this first by searching the real symbol with the normal
unprefixed symbol.  But then it uses the length of the original symbol to
check the end of the substring instead of the length of the symbol it is
looking for.  On non-prefixed arches, this is effectively the same thing,
so there is no problem.  On prefixed-arches, since this is exceeds by just
one byte, a crash is rare and it is usually a NUL byte anyways.  But every
once in a blue moon, you get the right page alignment and it segfaults.

For example, on the Blackfin arch, sym_is() will be called with the real
symbol "___mod_usb_device_table" as "symbol" when looking for the normal
symbol "__mod_usb_device_table" as "name".  The substring will thus return
one byte into "symbol" and store it into "match".  But then "match" will
be indexed with the length of "symbol" instead of "name" and so we will
exceed the storage.  i.e. the code ends up doing:
char foo[] = "abc"; return foo[strlen(foo)+1] == '\0';

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
scripts/mod/file2alias.c