Skip to content

Commit 4b73231

Browse files
Eric-Terminalbroonie
authored andcommitted
regulator: tps65185: check devm_kzalloc() result in probe
tps65185_probe() dereferences the allocation result immediately by using data->regmap. If devm_kzalloc() returns NULL under memory pressure, this leads to a NULL pointer dereference. Add the missing allocation check and return -ENOMEM on failure. Signed-off-by: Yufan Chen <ericterminal@gmail.com> Link: https://patch.msgid.link/20260222104035.90790-1-ericterminal@gmail.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent be70410 commit 4b73231

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

drivers/regulator/tps65185.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -332,6 +332,9 @@ static int tps65185_probe(struct i2c_client *client)
332332
int i;
333333

334334
data = devm_kzalloc(&client->dev, sizeof(*data), GFP_KERNEL);
335+
if (!data)
336+
return -ENOMEM;
337+
335338
data->regmap = devm_regmap_init_i2c(client, &regmap_config);
336339
if (IS_ERR(data->regmap))
337340
return dev_err_probe(&client->dev, PTR_ERR(data->regmap),

0 commit comments

Comments
 (0)