qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] bad error message in drive-mirror


From: Eric Blake
Subject: [Qemu-devel] bad error message in drive-mirror
Date: Mon, 25 Aug 2014 17:26:31 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.7.0

I just noticed today that passing "granularity":1 to QMP drive-mirror
results in this confusing error message:

Invalid parameter 'drive-virtio-disk0'

because the code mistakenly did
        error_set(errp, QERR_INVALID_PARAMETER, device);
instead of the correct
        error_set(errp, QERR_INVALID_PARAMETER, "granularity");

I ran out of time to patch it today, and at any rate, whoever writes the
patch should probably audit for other bad uses of
error_set(QERR_INVALID_PARAMETER).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]