projects
/
platform
/
kernel
/
linux-stable.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
d14b7a4
)
Revert "backlight: fix memory leak on obscure error path"
author
Jiri Kosina
<jkosina@suse.cz>
Mon, 30 Jul 2012 08:40:28 +0000
(10:40 +0200)
committer
Jiri Kosina
<jkosina@suse.cz>
Mon, 30 Jul 2012 08:40:28 +0000
(10:40 +0200)
This reverts commit
9ea3c498962bc63748e92c31c874169ab0726324
.
'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>
drivers/video/backlight/88pm860x_bl.c
patch
|
blob
|
history
diff --git
a/drivers/video/backlight/88pm860x_bl.c
b/drivers/video/backlight/88pm860x_bl.c
index
f75da87
..
f49181c
100644
(file)
--- a/
drivers/video/backlight/88pm860x_bl.c
+++ b/
drivers/video/backlight/88pm860x_bl.c
@@
-228,7
+228,6
@@
static int pm860x_backlight_probe(struct platform_device *pdev)
data->port = pdata->flags;
if (data->port < 0) {
dev_err(&pdev->dev, "wrong platform data is assigned");
- kfree(data);
return -EINVAL;
}