|
From: | Stefan Weil |
Subject: | Re: [Qemu-devel] [PING 0.14] Missing patches (mostly fixes) |
Date: | Fri, 04 Feb 2011 18:18:30 +0100 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.16) Gecko/20101226 Iceowl/1.0b1 Icedove/3.0.11 |
Am 04.02.2011 16:27, schrieb Markus Armbruster:
Anthony Liguori <address@hidden> 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/)LuizI 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. * 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
[Prev in Thread] | Current Thread | [Next in Thread] |