Re: [PATCH] tests: qmp-cmd-test: fix memory leak

2020-07-15 Thread Auger Eric
Hi Li, On 7/14/20 7:15 PM, Li Qiang wrote: > Fixes: 5b88849e7b9("tests/qmp-cmd-test: Add > qmp/object-add-failure-modes" Thank you for fixing this. Adding a commit message generally is welcome such as for example: properly free each test response and separate qtest_qmp() calls with spare lines,

Re: [PATCH] tests: qmp-cmd-test: fix memory leak

2020-07-14 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200714171531.83723-1-liq...@163.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/b

[PATCH] tests: qmp-cmd-test: fix memory leak

2020-07-14 Thread Li Qiang
Fixes: 5b88849e7b9("tests/qmp-cmd-test: Add qmp/object-add-failure-modes" Signed-off-by: Li Qiang --- tests/qtest/qmp-cmd-test.c | 13 + 1 file changed, 13 insertions(+) diff --git a/tests/qtest/qmp-cmd-test.c b/tests/qtest/qmp-cmd-test.c index c68f99f659..f7b1aa7fdc 100644 --- a/te