From: Dan Carpenter <error27@gmail.com>
Date: Fri, 7 May 2010 09:06:21 +0000 (+0200)
Subject: backlight: 88pm860x_bl - potential memory leak
X-Git-Tag: v2.6.35-rc1~31^2~13
X-Git-Url: https://bbs.cooldavid.org/git/?a=commitdiff_plain;h=9844ce0501b03843c44156d6e36afb80e847c974;p=net-next-2.6.git

backlight: 88pm860x_bl - potential memory leak

I added a kfree() on the error path.  I don't really expect it to affect
anyone, but it's easy and makes the static checkers happy.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
---

diff --git a/drivers/video/backlight/88pm860x_bl.c b/drivers/video/backlight/88pm860x_bl.c
index 68d2518fada..78d8fde9fe8 100644
--- a/drivers/video/backlight/88pm860x_bl.c
+++ b/drivers/video/backlight/88pm860x_bl.c
@@ -222,6 +222,7 @@ static int pm860x_backlight_probe(struct platform_device *pdev)
 	data->port = __check_device(pdata, name);
 	if (data->port < 0) {
 		dev_err(&pdev->dev, "wrong platform data is assigned");
+		kfree(data);
 		return -EINVAL;
 	}