[PATCH][next] pinctrl: th1520: Fix potential null pointer dereference on func

Linus Walleij linus.walleij at linaro.org
Wed Oct 16 12:32:55 PDT 2024


On Wed, Oct 16, 2024 at 9:31 PM Linus Walleij <linus.walleij at linaro.org> wrote:
> On Wed, Oct 16, 2024 at 5:56 PM Colin Ian King <colin.i.king at gmail.com> wrote:
>
> > The initialization of muxtype deferences pointer func before func
> > is sanity checked with a null pointer check, hence we have a null
> > pointer deference issue. Fix this by only deferencing func with
> > the assignment to muxtype after func has been null pointer checked.
> >
> > Fixes: 1fc30cd92770 ("pinctrl: th1520: Factor out casts")
> > Signed-off-by: Colin Ian King <colin.i.king at gmail.com>
>
> Patch applied, added Reported-by Kees since he mailed about this too.

...which was actually the "other Kees" (Bakker), not the one the mailer
suggested.

Yours,
Linus Walleij



More information about the linux-riscv mailing list