linux-kernel - Re: [Patch]Documentation/vm/slabinfo.c: clean up this code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 6 Oct 2007 09:21:25 +0800
From:	WANG Cong <xiyou.wangcong@...il.com>
To:	Christoph Lameter <clameter@....com>
Cc:	WANG Cong <xiyou.wangcong@...il.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...l.org>, linux-mm@...ck.org
Subject: Re: [Patch]Documentation/vm/slabinfo.c: clean up this code

On Fri, Oct 05, 2007 at 12:17:41PM -0700, Christoph Lameter wrote:
>On Fri, 5 Oct 2007, WANG Cong wrote:
>
>> 
>> This patch does the following cleanups for Documentation/vm/slabinfo.c:
>> 
>> 	- Fix two memory leaks;
>
>For user space code? Memory will be released as soon as the program 
>terminates.

Yes, it's of course in user space. But there really exists memory leaks,
since strdup(3) uses malloc(3) to allocate memory for new string, we
should use free(3) to free the memory, or this memory will be lost.

Regards. ;)


WANG Cong

-
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/