Re: [Qemu-devel] [PATCH v9 03/27] qapi: Plug leaks in test-qmp-*

2015-11-05 Thread Eric Blake
On 11/04/2015 10:44 AM, Markus Armbruster wrote: > Eric Blake writes: > >> On 11/04/2015 01:19 AM, Markus Armbruster wrote: >>> Eric Blake writes: >>> Make valgrind happy with the current state of the tests, so that tests/test-qmp-input-strict.c | 10 ++ tests/test-qmp

Re: [Qemu-devel] [PATCH v9 03/27] qapi: Plug leaks in test-qmp-*

2015-11-04 Thread Markus Armbruster
Eric Blake writes: > On 11/04/2015 01:19 AM, Markus Armbruster wrote: >> Eric Blake writes: >> >>> Make valgrind happy with the current state of the tests, so that >>> it is easier to see if future patches introduce new memory problems >>> without being drowned in noise. Many of the leaks were

Re: [Qemu-devel] [PATCH v9 03/27] qapi: Plug leaks in test-qmp-*

2015-11-04 Thread Eric Blake
On 11/04/2015 01:19 AM, Markus Armbruster wrote: > Eric Blake writes: > >> Make valgrind happy with the current state of the tests, so that >> it is easier to see if future patches introduce new memory problems >> without being drowned in noise. Many of the leaks were due to >> calling a second

Re: [Qemu-devel] [PATCH v9 03/27] qapi: Plug leaks in test-qmp-*

2015-11-04 Thread Markus Armbruster
Eric Blake writes: > Make valgrind happy with the current state of the tests, so that > it is easier to see if future patches introduce new memory problems > without being drowned in noise. Many of the leaks were due to > calling a second init without tearing down the data from an earlier > visi

[Qemu-devel] [PATCH v9 03/27] qapi: Plug leaks in test-qmp-*

2015-11-03 Thread Eric Blake
Make valgrind happy with the current state of the tests, so that it is easier to see if future patches introduce new memory problems without being drowned in noise. Many of the leaks were due to calling a second init without tearing down the data from an earlier visit. But since teardown is alrea