|
From: | Emanuele |
Subject: | Re: [Qemu-devel] [PATCH v2 00/34] Qtest driver framework |
Date: | Thu, 9 Aug 2018 12:27:37 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 08/09/2018 11:44 AM, Paolo Bonzini wrote:
On 09/08/2018 11:20, Emanuele wrote:Why this? Shouldn't it be: if (g_strcmp0(old_path, path)) { qtest_end(); /* handles global_qtest = NULL */ g_free(old_path); /* handles NULL */ old_path = path; global_qtest = qtest_start(path); } else ....Yes, of course. Though I'd have thought that my version has "just" a memory leak. Even better, this could call qos_invalidate_command_line. Can you post the fixes to a separate commit on github?
https://github.com/esposem/qemu/commit/4389e83522c06ce7001382284153d59105692061I also added g_free(old_path); in qos-test main function, since last old_path is not free'd. This commit is part of qgraph-v3, where I am also applying Laurent's suggestions.
Emanuele
[Prev in Thread] | Current Thread | [Next in Thread] |