From: Wei Yongjun <yongjun_...@trendmicro.com.cn> 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 <yongjun_...@trendmicro.com.cn> --- 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))); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
