Re: gpio: omap: handle clk_prepare failure in probe
Andreas Kemnade <[email protected]>
| Newsgroups | gmane.linux.kernel.janitors,gmane.linux.kernel.gpio,gmane.linux.ports.arm.omap,gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
On Thu, 20 Aug 2026 12:47:05 +0200 Markus Elfring <[email protected]> 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