]> bbs.cooldavid.org Git - net-next-2.6.git/commit
git://bbs.cooldavid.org / net-next-2.6.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 0fffed2) | patch
Input: adxl34x - fix leak and use after free
authorDan Carpenter <error27@gmail.com>
Fri, 23 Jul 2010 06:38:45 +0000 (23:38 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 23 Jul 2010 06:39:16 +0000 (23:39 -0700)
commitf1cba532e8c1001a39650379aa7e04ad974d0592
tree9c663e053d1a9c35d260de1b00a141e6b9dd073ftree | snapshot (tar.bz2 tar.gz zip)
parent0fffed27f92d9d7a34de9fe017b7082b5958bb93commit | diff
Input: adxl34x - fix leak and use after free

These are a couple smatch issues.  In the original code, if only one of
the allocation fails we leak the other variable so we should goto
out_free_mem.

Also there was a use after free if debugging was enabled and so I moved
the kfree() down a line.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
drivers/input/misc/adxl34x.c diff | blob | blame | history
Clone of davem's net-next-2.6 tree