summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMaxim Levitsky <maximlevitsky@gmail.com>2011-01-09 01:25:06 +0200
committerDavid Woodhouse <David.Woodhouse@intel.com>2011-03-11 14:19:44 +0000
commitbd637f6f22235b4613f9ab6555e8088a455c1ed4 (patch)
tree3892841ec4fd7bbf0ceffad2e594830b554d0f34
parent82013d988fc03a1b908b2b0360a1e34f6152fda6 (diff)
downloadop-kernel-dev-bd637f6f22235b4613f9ab6555e8088a455c1ed4.zip
op-kernel-dev-bd637f6f22235b4613f9ab6555e8088a455c1ed4.tar.gz
mtd: mtd_blkdevs: fix double free on error path
This one liner patch fixes double free that will occur if add_mtd_blktrans_dev fails. On failure it frees the input argument, but all its users also free it on error which is natural thing to do. Thus don't free it. All credit for finding that bug belongs to reporters of the bug in the android bugzilla http://code.google.com/p/android/issues/detail?id=13761 Commit message tweaked by Artem. Signed-off-by: Maxim Levitsky <maximlevitsky@gmail.com> Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com> Signed-off-by: David Woodhouse <David.Woodhouse@intel.com> Cc: stable@kernel.org
-rw-r--r--drivers/mtd/mtd_blkdevs.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/mtd/mtd_blkdevs.c b/drivers/mtd/mtd_blkdevs.c
index cb20c67..e0a2373 100644
--- a/drivers/mtd/mtd_blkdevs.c
+++ b/drivers/mtd/mtd_blkdevs.c
@@ -413,7 +413,6 @@ error3:
error2:
list_del(&new->list);
error1:
- kfree(new);
return ret;
}
OpenPOWER on IntegriCloud