From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46964 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PlPMC-00038m-6m for qemu-devel@nongnu.org; Fri, 04 Feb 2011 12:22:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PlPMB-0004sZ-2y for qemu-devel@nongnu.org; Fri, 04 Feb 2011 12:22:04 -0500 Received: from mail-iw0-f173.google.com ([209.85.214.173]:61806) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PlPMA-0004sJ-UH for qemu-devel@nongnu.org; Fri, 04 Feb 2011 12:22:03 -0500 Received: by iwc10 with SMTP id 10so2519042iwc.4 for ; Fri, 04 Feb 2011 09:22:01 -0800 (PST) Message-ID: <4D4C35B2.9010801@codemonkey.ws> Date: Fri, 04 Feb 2011 11:21:54 -0600 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PING 0.14] Missing patches (mostly fixes) References: <4D412CF0.30706@redhat.com> <4D49B04F.80201@mail.berlios.de> <4D4BF160.9040500@codemonkey.ws> <4D4C34E6.7070008@mail.berlios.de> In-Reply-To: <4D4C34E6.7070008@mail.berlios.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Weil Cc: Kevin Wolf , Anthony Liguori , "Michael S. Tsirkin" , qemu-devel , Riku Voipio , Alexander Graf , Markus Armbruster , Blue Swirl , Gerd Hoffmann , Paolo Bonzini On 02/04/2011 11:18 AM, Stefan Weil wrote: > Am 04.02.2011 16:27, schrieb Markus Armbruster: >> Anthony Liguori writes: >> >>> On 02/02/2011 01:28 PM, Stefan Weil wrote: >> [...] >>>> [PATCH 1/3] tests: Fix two memory leaks >>>> (http://patchwork.ozlabs.org/patch/79945/) >> >>>> [PATCH 2/3] check-qdict: Fix possible crash >>>> (http://patchwork.ozlabs.org/patch/79946/) >>> >>> Luiz >> >> I wouldn't bother with the second one for 0.14. Yes, we're reading >> lines from a file with %s, but it's a fixed file with known contents, no >> long lines, and we're reading it in a test program only developers ever >> use. >> >> As to the first one, Luiz has never touched that file. Neither have I, >> and it's not obvious to me why it should go into 0.14. >> >> [...] > > Even if the current code does not result in a real bug at the moment, > it should get fixed: > > * Using tools like cppcheck (or others) to find bugs is good, > because it finds bugs which are important. > Sorting out "unimportant" bugs from the results wastes time > which could be invested better, and this waste of time lasts > forever until the "unimportant" bug will be fixed. The sooner > you fix it, the better it is. No, this is not a good use of time. I've said multiple times in the past, I'm not interested in implementing work arounds for false positives in static analysis tools. We have enough real problems to fix, we don't need to waste cycles on psuedo problems. Regards, Anthony Liguori > > * Code gets copied, and maybe the copy of code with some weakness > can expose a real problem. > > Therefore I think that both patches should be applied > at least to qemu master (as they are really simple patches, > applying them to 0.14 would be cheap and good as well). > > Regards, > Stefan Weil > >