elflink: Don't use strcmp on a non-NUL terminated string
authorMatt Fleming <matt.fleming@intel.com>
Thu, 1 Mar 2012 17:23:57 +0000 (17:23 +0000)
committerMatt Fleming <matt.fleming@intel.com>
Fri, 23 Mar 2012 16:56:15 +0000 (16:56 +0000)
commit6b28c0cfec3ecc43c9718fafbfd19297c3768701
tree12f4f04c94c86cdb6d05dcb8b1128f7bf915bad8
parent3a316db137afe3b4c9d7f04fcf47921f509ef36c
elflink: Don't use strcmp on a non-NUL terminated string

The syslinux implementation of strcmp() only checks for NUL in the
first argument and will use any NULs in the second argument for
comparison with the corresponding character in the first string. This
caused problems because at the time strcmp() is called 'path' isn't
NUL-terminated. Since we're only checking the first character of
'path' just code it explicitly.

This bug caused some modules not to load when resolving module
dependencies via the ELF DT_NEEDED field because the "." in the PATH
lookup code wouldn't be expanded to the current working directory. For
example, instead of expanding to "/foo/bar" if /foo was the cwd, the
path would be ".bar".

Signed-off-by: Matt Fleming <matt.fleming@intel.com>
com32/lib/sys/module/common.c