Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261918AbTIPOvm (ORCPT ); Tue, 16 Sep 2003 10:51:42 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261920AbTIPOvm (ORCPT ); Tue, 16 Sep 2003 10:51:42 -0400 Received: from kinesis.swishmail.com ([209.10.110.86]:17925 "HELO kinesis.swishmail.com") by vger.kernel.org with SMTP id S261918AbTIPOvl (ORCPT ); Tue, 16 Sep 2003 10:51:41 -0400 Message-ID: <3F672396.10906@techsource.com> Date: Tue, 16 Sep 2003 10:52:06 -0400 From: Timothy Miller User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20020823 Netscape/7.0 X-Accept-Language: en-us, en MIME-Version: 1.0 To: =?ISO-8859-1?Q?J=F6rn_Engel?= CC: David Yu Chen , linux-kernel@vger.kernel.org, mc@cs.stanford.edu, David Woodhouse , linux-mtd@lists.infradead.org Subject: Re: [CHECKER] 32 Memory Leaks on Error Paths References: <200309160435.h8G4ZkQM009953@elaine4.Stanford.EDU> <20030916065553.GA12329@wohnheim.fh-wedel.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 814 Lines: 22 >> 276: /* OK, it's not open. Create cache info for it */ >>START --> >> 277: mtdblk = kmalloc(sizeof(struct mtdblk_dev), GFP_KERNEL); >> 278: if (!mtdblks) >>END --> >> 279: return -ENOMEM; > > Invalid. This is quite an obvious false positive, at least if your > algorithm checks for possible value ranges. Wait... one is "mtdblk", and the other is "mtdblks". One has an extra 's' on it. Unless there is some kind of aliasing going on, they would appear to be different variables. Naturally, I didn't check the original code, so I could be full of it. :) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/