Commit 9da4a56d authored Nov 26, 2023 by Kunwu Chan Committed by Greg Kroah-Hartman Jan 25, 2024

powerpc/powernv: Add a null pointer check in opal_powercap_init()



[ Upstream commit e123015c0ba859cf48aa7f89c5016cc6e98e018d ]

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

Fixes: b9ef7b4b ("powerpc: Convert to using %pOFn instead of device_node.name")
Signed-off-by: Kunwu Chan <chentao@kylinos.cn>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20231126095739.1501990-1-chentao@kylinos.cn


Signed-off-by: Sasha Levin <sashal@kernel.org>

parent e93d7cf4

arch/powerpc/platforms/powernv/opal-powercap.c

+6 −0

Original line number	Diff line number	Diff line
		@@ -196,6 +196,12 @@ void __init opal_powercap_init(void)

		j = 0;
		pcaps[i].pg.name = kasprintf(GFP_KERNEL, "%pOFn", node);
		if (!pcaps[i].pg.name) {
		kfree(pcaps[i].pattrs);
		kfree(pcaps[i].pg.attrs);
		goto out_pcaps_pattrs;
		}

		if (has_min) {
		powercap_add_attr(min, "powercap-min",
		&pcaps[i].pattrs[j]);