Re: [PATCH 1/2] depmod: do not allow partial matches with "search" directive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Anssi,

On Tue, Mar 18, 2014 at 8:26 PM, Anssi Hannula <anssi@xxxxxxxxxx> wrote:
> Currently e.g. "search foo foobar built-in" will cause unpredictable
> results if baz.ko is in both foo/ and foobar/, since "foo" in search may
> match both of those directories and the preferred module therefore
> depends on processing order.
>
> Fix the code to ensure that the match is performed on full pathname
> components only.
> ---
>  tools/depmod.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/depmod.c b/tools/depmod.c
> index 9f83ee8..328e578 100644
> --- a/tools/depmod.c
> +++ b/tools/depmod.c
> @@ -1153,10 +1153,10 @@ static int depmod_module_is_higher_priority(const struct depmod *depmod, const s
>                 DBG("search %s\n", se->builtin ? "built-in" : se->path);
>                 if (se->builtin)
>                         bprio = i;
> -               else if (newlen >= se->len &&
> +               else if (newlen > se->len && newpath[se->len] == '/' &&
>                          memcmp(se->path, newpath, se->len) == 0)
>                         newprio = i;
> -               else if (oldlen >= se->len &&
> +               else if (oldlen > se->len && oldpath[se->len] == '/' &&
>                          memcmp(se->path, oldpath, se->len) == 0)
>                         oldprio = i;
>         }
> --


Both patches have been applied. Thanks a lot.

I added some test cases for depmod. Could you take a look if they are ok?

--
Lucas De Marchi
--
To unsubscribe from this list: send the line "unsubscribe linux-modules" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux