depmod: module_is_higher_priority: fix modname length calculation
authorYauheni Kaliuta <yauheni.kaliuta@redhat.com>
Thu, 7 Dec 2017 19:16:08 +0000 (21:16 +0200)
committerLucas De Marchi <lucas.demarchi@intel.com>
Thu, 7 Dec 2017 21:08:23 +0000 (13:08 -0800)
commitbb83f6ac68fe66c6e17afcab2cd6c2712e5c570e
treed676b4d45013ab0c0aa3fe0e5cda815cb4ede6e7
parent809b9fb6cf6b1f5cd665a1da3fa296d705c2af2b
depmod: module_is_higher_priority: fix modname length calculation

depmod_module_is_higher_priority checks module's path if it is under
module root directory and if so uses relative to the root path to
lookup the module in override and search lists.

Originally only relative path was used in the function, so the
variables with full path and and path length were changed:

       newpath += cfg->dirnamelen + 1;
       newlen -= cfg->dirnamelen + 1;
       oldpath += cfg->dirnamelen + 1;
       oldlen -= cfg->dirnamelen + 1;

Commit 7da6884e7357ac05772e90f6d7e63b1948103fc4 (depmod: implement
external directories support) changed the logic since it need the
full path to the module for comparations as well.

Unfortunately, it introduce a mistake in calculation of the relative
paths replacing '-=' with assignment to a new variable -- the
'cfg->dirnamelen + 1' value must be substracted all together. It
breaks, for example, overrides lookup.

Fix the calculation by putting braces around the value in the
subsctuction expression.

Signed-off-by: Yauheni Kaliuta <yauheni.kaliuta@redhat.com>
tools/depmod.c