Commit b1356ed1 authored by Claudiu Beznea's avatar Claudiu Beznea Committed by Stephen Boyd
Browse files

clk: clocking-wizard: check return value of devm_kasprintf() 



devm_kasprintf() returns a pointer to dynamically allocated memory.
Pointer could be NULL in case allocation fails. Check pointer validity.
Identified with coccinelle (kmerr.cocci script).

Fixes: 2046338d ("ARM: mxs: Use soc bus infrastructure")
Signed-off-by: default avatarClaudiu Beznea <claudiu.beznea@microchip.com>
Link: https://lore.kernel.org/r/20230530093913.1656095-9-claudiu.beznea@microchip.com


Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent bd46cd0b

drivers/clk/xilinx/clk-xlnx-clock-wizard.c

+5 −0
Original line number Diff line number Diff line
@@ -648,6 +648,11 @@ static int clk_wzrd_probe(struct platform_device *pdev) 
	}



	clkout_name = devm_kasprintf(&pdev->dev, GFP_KERNEL, "%s_out0", dev_name(&pdev->dev));

	if (!clkout_name) {

		ret = -ENOMEM;

		goto err_disable_clk;

	}



	if (nr_outputs == 1) {

		clk_wzrd->clkout[0] = clk_wzrd_register_divider

				(&pdev->dev, clkout_name,