misc: tps6594-pfsm: Add NULL pointer check in tps6594_pfsm_probe()
[ Upstream commit a99b598d836c9c6411110c70a2da134c78d96e67 ] The returned value, pfsm->miscdev.name, from devm_kasprintf() could be NULL. A pointer check is added to prevent potential NULL pointer dereference. This is similar to the fix in commit 3027e7b15b02 ("ice: Fix some null pointer dereference issues in ice_ptp.c"). This issue is found by our static analysis tool. Signed-off-by: Chenyuan Yang <chenyuan0y@gmail.com> Link: https://lore.kernel.org/r/20250311010511.1028269-1-chenyuan0y@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
1063c586a3
commit
a8d1b4f219
@@ -260,6 +260,9 @@ static int tps6594_pfsm_probe(struct platform_device *pdev)
|
|||||||
pfsm->miscdev.minor = MISC_DYNAMIC_MINOR;
|
pfsm->miscdev.minor = MISC_DYNAMIC_MINOR;
|
||||||
pfsm->miscdev.name = devm_kasprintf(dev, GFP_KERNEL, "pfsm-%ld-0x%02x",
|
pfsm->miscdev.name = devm_kasprintf(dev, GFP_KERNEL, "pfsm-%ld-0x%02x",
|
||||||
tps->chip_id, tps->reg);
|
tps->chip_id, tps->reg);
|
||||||
|
if (!pfsm->miscdev.name)
|
||||||
|
return -ENOMEM;
|
||||||
|
|
||||||
pfsm->miscdev.fops = &tps6594_pfsm_fops;
|
pfsm->miscdev.fops = &tps6594_pfsm_fops;
|
||||||
pfsm->miscdev.parent = dev->parent;
|
pfsm->miscdev.parent = dev->parent;
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user