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-Url: http://drtracing.org/?a=commitdiff_plain;h=9844ce0501b03843c44156d6e36afb80e847c974;p=deliverable%2Flinux.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 68d2518fadaa..78d8fde9fe8f 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;
 	}