diff options
| author | Sergey Shtylyov <s.shtylyov@omp.ru> | 2023-07-19 23:22:53 +0300 |
|---|---|---|
| committer | Linus Walleij <linus.walleij@linaro.org> | 2023-07-28 22:08:58 +0200 |
| commit | 6ec89cd4d17bd5e818c335b72c736a5094ea66d7 (patch) | |
| tree | 9b0bc6421fb0d5372b65a105a9e0c3ebec7f5d0a /drivers/input/tablet/aiptek.c | |
| parent | ecfe9a015d3e1e46504d5b3de7eef1f2d186194a (diff) | |
| download | lwn-6ec89cd4d17bd5e818c335b72c736a5094ea66d7.tar.gz lwn-6ec89cd4d17bd5e818c335b72c736a5094ea66d7.zip | |
pinctrl: pinmux: handle radix_tree_insert() errors in pinmux_generic_add_function()
pinctrl_generic_add_function() doesn't check result of radix_tree_insert()
despite they both may return a negative error code. Linus Walleij said he
has copied the radix tree code from kernel/irq/ where the functions calling
radix_tree_insert() are *void* themselves; I think it makes more sense to
propagate the errors from radix_tree_insert() upstream if we can do that...
Found by Linux Verification Center (linuxtesting.org) with the Svace static
analysis tool.
Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Link: https://lore.kernel.org/r/20230719202253.13469-4-s.shtylyov@omp.ru
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/input/tablet/aiptek.c')
0 files changed, 0 insertions, 0 deletions