[libvirt] [PATCH RFC 2/4] qemu_agent: ignore requests echoed back by guest

Joao Martins joao.m.martins at oracle.com
Fri Mar 3 11:38:46 UTC 2017



On 03/02/2017 06:10 PM, Jim Fehlig wrote:
> On 02/08/2017 09:44 AM, Joao Martins wrote:
>> Signed-off-by: Joao Martins <joao.m.martins at oracle.com>
> 
> Since it wasn't needed before, I think it is helpful for the commit message to 
> describe why the change is needed now.

True, my apologies. I should have included at least the sentence I had on the
cover letter regarding this commit (same for patch 3).

>> ---
>>  src/util/virqemuagent.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/util/virqemuagent.c b/src/util/virqemuagent.c
>> index caabae0..ffb3489 100644
>> --- a/src/util/virqemuagent.c
>> +++ b/src/util/virqemuagent.c
>> @@ -333,7 +333,8 @@ qemuAgentIOProcessLine(qemuAgentPtr mon,
>>          goto cleanup;
>>      }
>>
>> -    if (virJSONValueObjectHasKey(obj, "QMP") == 1) {
>> +    if (virJSONValueObjectHasKey(obj, "QMP") == 1 ||
>> +        virJSONValueObjectHasKey(obj, "execute") == 1) {
> 
> And a brief comment in the code, e.g. along the lines of the commit summary.

/nods




More information about the libvir-list mailing list