On 16/07/2020 12.33, Li Qiang wrote: > Markus Armbruster <arm...@redhat.com> 于2020年7月16日周四 下午5:52写道: >> >> Li Qiang <liq...@gmail.com> writes: >> >>> Markus Armbruster <arm...@redhat.com> 于2020年7月16日周四 下午1:59写道: >>>> >>>> Li Qiang <liq...@163.com> writes: >>>> >>>>> Properly free each test response to avoid memory leak and separate >>>>> qtest_qmp() calls with spare lines, in a consistent manner. >>>>> >>>>> Fixes: 5b88849e7b9("tests/qmp-cmd-test: Add >>>>> qmp/object-add-failure-modes" >>>> >>>> The patch also fixes leaks introduced in 442b09b83d and 9fc719b869, >>>> actually. At least it should, but the patch appears to be incomplete. >> >> 442b09b83d was fine, actually. >> >> 9fc719b869 wasn't, and your second patch hunk fixes it. Please add a >> "Fixes: 9fc719b869' line to the commit message. > > Hi Thomas, > > Could you do this minor adjustment? > Add also add Markus's r-b tag.
Sure, I'll add it! Thomas