Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dfbb9cea authored by Michał Mirosław's avatar Michał Mirosław Committed by Greg Kroah-Hartman
Browse files

regulator: plug of_node leak in regulator_register()'s error path 



commit d3c731564e09b6c2ebefcd1344743a91a237d6dc upstream.

By calling device_initialize() earlier and noting that kfree(NULL) is
ok, we can save a bit of code in error handling and plug of_node leak.
Fixed commit already did part of the work.

Fixes: 9177514ce349 ("regulator: fix memory leak on error path of regulator_register()")
Signed-off-by: default avatarMichał Mirosław <mirq-linux@rere.qmqm.pl>
Reviewed-by: default avatarVladimir Zapolskiy <vz@mleia.com>
Acked-by: default avatarVladimir Zapolskiy <vz@mleia.com>
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/r/f5035b1b4d40745e66bacd571bbbb5e4644d21a1.1597195321.git.mirq-linux@rere.qmqm.pl


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 047a71ff

drivers/regulator/core.c

+4 −9
Original line number Diff line number Diff line
@@ -5065,6 +5065,7 @@ regulator_register(const struct regulator_desc *regulator_desc, 
		ret = -ENOMEM;

		goto rinse;

	}

	device_initialize(&rdev->dev);



	/*

	 * Duplicate the config so the driver could override it after
@@ -5072,9 +5073,8 @@ regulator_register(const struct regulator_desc *regulator_desc, 
	 */

	config = kmemdup(cfg, sizeof(*cfg), GFP_KERNEL);

	if (config == NULL) {

		kfree(rdev);

		ret = -ENOMEM;

		goto rinse;

		goto clean;

	}



	init_data = regulator_of_get_init_data(dev, regulator_desc, config,
@@ -5086,10 +5086,8 @@ regulator_register(const struct regulator_desc *regulator_desc, 
	 * from a gpio extender or something else.

	 */

	if (PTR_ERR(init_data) == -EPROBE_DEFER) {

		kfree(config);

		kfree(rdev);

		ret = -EPROBE_DEFER;

		goto rinse;

		goto clean;

	}



	/*
@@ -5142,7 +5140,6 @@ regulator_register(const struct regulator_desc *regulator_desc, 
	}



	/* register with sysfs */

	device_initialize(&rdev->dev);

	rdev->dev.class = &regulator_class;

	rdev->dev.parent = dev;

	dev_set_name(&rdev->dev, "regulator.%lu",
@@ -5220,13 +5217,11 @@ regulator_register(const struct regulator_desc *regulator_desc, 
	mutex_lock(&regulator_list_mutex);

	regulator_ena_gpio_free(rdev);

	mutex_unlock(&regulator_list_mutex);

	put_device(&rdev->dev);

	rdev = NULL;

clean:

	if (dangling_of_gpiod)

		gpiod_put(config->ena_gpiod);

	kfree(rdev);

	kfree(config);

	put_device(&rdev->dev);

rinse:

	if (dangling_cfg_gpiod)

		gpiod_put(cfg->ena_gpiod);