summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2013-02-01 16:28:27 +0300
committerMatthew Garrett <matthew.garrett@nebula.com>2013-02-27 08:30:43 -0500
commit34cf1df34aba1a6274deeef92ad6d02d5f52dab0 (patch)
treeea9f8a7db5b603671e245e5e5b53334ef4e09afc
parentfabf85e3ca15d5b94058f391dac8df870cdd427a (diff)
downloadop-kernel-dev-34cf1df34aba1a6274deeef92ad6d02d5f52dab0.zip
op-kernel-dev-34cf1df34aba1a6274deeef92ad6d02d5f52dab0.tar.gz
sony-laptop: leak in error handling sony_nc_lid_resume_setup()
We need to decrement "i" first because the current "i" was not allocated succesfully. Also we should go free the way down to zero to avoid a leak. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Matthew Garrett <matthew.garrett@nebula.com>
-rw-r--r--drivers/platform/x86/sony-laptop.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c
index 438c7fa..9557414 100644
--- a/drivers/platform/x86/sony-laptop.c
+++ b/drivers/platform/x86/sony-laptop.c
@@ -2351,7 +2351,7 @@ static int sony_nc_lid_resume_setup(struct platform_device *pd)
return 0;
liderror:
- for (; i > 0; i--)
+ for (i--; i >= 0; i--)
device_remove_file(&pd->dev, &lid_ctl->attrs[i]);
kfree(lid_ctl);
OpenPOWER on IntegriCloud