diff options
author | Tang Bin <tangbin@cmss.chinamobile.com> | 2021-09-06 04:57:32 +0300 |
---|---|---|
committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2021-09-06 05:42:42 +0300 |
commit | 58ae4004b9c4bb040958cf73986b687a5ea4d85d (patch) | |
tree | df8b9345682d3a3016f639a056391578eb0fc2c5 /drivers/input/misc/cpcap-pwrbutton.c | |
parent | 3e204d6b76b29274cc8e57f8bd8d9873f04a7f48 (diff) | |
download | linux-58ae4004b9c4bb040958cf73986b687a5ea4d85d.tar.xz |
Input: cpcap-pwrbutton - handle errors from platform_get_irq()
The function cpcap_power_button_probe() does not perform
sufficient error checking after executing platform_get_irq(),
thus fix it.
Signed-off-by: Tang Bin <tangbin@cmss.chinamobile.com>
Link: https://lore.kernel.org/r/20210802121740.8700-1-tangbin@cmss.chinamobile.com
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers/input/misc/cpcap-pwrbutton.c')
-rw-r--r-- | drivers/input/misc/cpcap-pwrbutton.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/input/misc/cpcap-pwrbutton.c b/drivers/input/misc/cpcap-pwrbutton.c index 0abef63217e2..372cb44d0635 100644 --- a/drivers/input/misc/cpcap-pwrbutton.c +++ b/drivers/input/misc/cpcap-pwrbutton.c @@ -54,9 +54,13 @@ static irqreturn_t powerbutton_irq(int irq, void *_button) static int cpcap_power_button_probe(struct platform_device *pdev) { struct cpcap_power_button *button; - int irq = platform_get_irq(pdev, 0); + int irq; int err; + irq = platform_get_irq(pdev, 0); + if (irq < 0) + return irq; + button = devm_kmalloc(&pdev->dev, sizeof(*button), GFP_KERNEL); if (!button) return -ENOMEM; |