lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 14 Oct 2014 10:32:53 +0200
From:	Krzysztof Kozlowski <>
To:	Sebastian Reichel <>,
	Dmitry Eremin-Solenikov <>,
	David Woodhouse <>,,
Cc:	Krzysztof Kozlowski <>,
Subject: [RFT PATCH] power: bq2415x_charger: Properly handle ENODEV from

The power_supply_get_by_phandle() on error returns ENODEV or NULL.
The driver later expects obtained pointer to power supply to be
valid or NULL. If it is not NULL then it dereferences it in
bq2415x_notifier_call() which would lead to dereferencing ENODEV-value

Properly handle the power_supply_get_by_phandle() error case and abort probe.

Signed-off-by: Krzysztof Kozlowski <>
Fixes: faffd234cf85 ("bq2415x_charger: Add DT support")
Cc: <>
 drivers/power/bq2415x_charger.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/power/bq2415x_charger.c b/drivers/power/bq2415x_charger.c
index e384844a1ae1..d9c457217b6a 100644
--- a/drivers/power/bq2415x_charger.c
+++ b/drivers/power/bq2415x_charger.c
@@ -1579,8 +1579,13 @@ static int bq2415x_probe(struct i2c_client *client,
 	if (np) {
 		bq->notify_psy = power_supply_get_by_phandle(np, "ti,usb-charger-detection");
-		if (!bq->notify_psy)
+		if (!bq->notify_psy) {
 			return -EPROBE_DEFER;
+		} else if (IS_ERR(bq->notify_psy)) {
+			dev_err(&client->dev, "no 'ti,usb-charger-detection' property\n");
+			ret = PTR_ERR(bq->notify_psy);
+			goto error_2;
+		}
 	else if (pdata->notify_device)
 		bq->notify_psy = power_supply_get_by_name(pdata->notify_device);

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

Powered by blists - more mailing lists