Revert "backlight: fix memory leak on obscure error path"
This reverts commit 9ea3c49896
.
'data' is being allocated by devm_kzalloc(), is it's actually
wrong to free() it in the error path, as it will be destroyed
properly later.
Reported-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
@@ -228,7 +228,6 @@ static int pm860x_backlight_probe(struct platform_device *pdev)
|
|||||||
data->port = pdata->flags;
|
data->port = pdata->flags;
|
||||||
if (data->port < 0) {
|
if (data->port < 0) {
|
||||||
dev_err(&pdev->dev, "wrong platform data is assigned");
|
dev_err(&pdev->dev, "wrong platform data is assigned");
|
||||||
kfree(data);
|
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user