Commit 5b91bce6 authored by Kunwu Chan's avatar Kunwu Chan Committed by sanglipeng1
Browse files

net: dsa: vsc73xx: Add null pointer check to vsc73xx_gpio_probe 

stable inclusion
from stable-v5.10.209
commit b099b495e1da3334fe666774201ea774e8c262df
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I9U3NW

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b099b495e1da3334fe666774201ea774e8c262df



--------------------------------

[ Upstream commit 776dac5a662774f07a876b650ba578d0a62d20db ]

devm_kasprintf() returns a pointer to dynamically allocated memory
which can be NULL upon failure.

Fixes: 05bd97fc ("net: dsa: Add Vitesse VSC73xx DSA router driver")
Signed-off-by: default avatarKunwu Chan <chentao@kylinos.cn>
Suggested-by: default avatarJakub Kicinski <kuba@kernel.org>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20240111072018.75971-1-chentao@kylinos.cn


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarsanglipeng1 <sanglipeng1@jd.com>
parent c65b9687

drivers/net/dsa/vitesse-vsc73xx-core.c

+2 −0
Original line number Diff line number Diff line
@@ -1119,6 +1119,8 @@ static int vsc73xx_gpio_probe(struct vsc73xx *vsc) 


	vsc->gc.label = devm_kasprintf(vsc->dev, GFP_KERNEL, "VSC%04x",

				       vsc->chipid);

	if (!vsc->gc.label)

		return -ENOMEM;

	vsc->gc.ngpio = 4;

	vsc->gc.owner = THIS_MODULE;

	vsc->gc.parent = vsc->dev;