[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Wed, 6 Jun 2007 05:10:24 +0000
From: "young dave" <hidave.darkstar@...il.com>
To: "Andrew Morton" <akpm@...ux-foundation.org>
Cc: "Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
"Jaroslav Kysela" <perex@...e.cz>
Subject: Re: [patch -mm] alsa mixer_oss kfree fix
Hi, Andrew
> kfree(NULL) is legal, and is often used.
>
Apart from the null pointer, IMHO,there's two problem need to be
fixed, I'm not sure.
What's your opinion?
1. the label indent should be removed
2. similar code use different label, some use " __unlock" but others
use "__unalloc", "__unlock" seems to be misspelled.
Regards
dave
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists