Re: gpio: omap: handle clk_prepare failure in probe

From: Andreas Kemnade

Date: Thu Aug 20 2026 - 09:44:29 EST


On Thu, 20 Aug 2026 12:47:05 +0200
Markus Elfring <Markus.Elfring@xxxxxx> wrote:

> > About adding this to stable:
> > does it bother anyone?
>
> I imagine that it can be safer to avoid return value ignorance a bit more.
> https://cwe.mitre.org/data/definitions/252.html
>
Well, if that would depend on user input, esp. over the network, it would be
clear.
But here you need to patch the devicetree to add a clock there
that has a prepare() in the ancestry. shows the desired behavior to exploit
something.

And then the error check is done by clk_enable() anyways.
BTW: here the return value of that is not checked. That is the more
interesting issue here.

Quoting stable kernel rules:
"
- No "This could be a problem..." type of things like a "theoretical race
condition", unless an explanation of how the bug can be exploited is also
provided.
"
From taking that verbatim, I would say no CC stable.
But pragmatically, also to avoid noise in any security scanner, I would agree
to a CC stable here.

Regards,
Andreas