From: Wei Yongjun <[email protected]> In the error handling case of kdb_defcmd2() s->command has been set to NULL but the save_command remain no free, this will cause memory leak.
spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun <[email protected]> --- kernel/debug/kdb/kdb_main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c index 31df170..0cbe79f 100644 --- a/kernel/debug/kdb/kdb_main.c +++ b/kernel/debug/kdb/kdb_main.c @@ -651,6 +651,7 @@ static int kdb_defcmd2(const char *cmdstr, const char *argv0) kdb_printf("Could not allocate new kdb_defcmd table for %s\n", cmdstr); s->usable = 0; + kfree(save_command); return KDB_NOTIMP; } memcpy(s->command, save_command, s->count * sizeof(*(s->command))); ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ Kgdb-bugreport mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
