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


Reply via email to