On Thu, Jan 30, 2014 at 16:58:18 +0800, Osier Yang wrote:
> This fixes the wrong argument order.
>
> ---
> Pushed under trivial rule.
> ---
> src/qemu/qemu_hostdev.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/qemu/qemu_hostdev.c b/src/qemu/qemu_hostdev.c
> index 2b11d6c..1b16386 100644
> --- a/src/qemu/qemu_hostdev.c
> +++ b/src/qemu/qemu_hostdev.c
> @@ -1135,8 +1135,8 @@ qemuPrepareHostdevSCSIDevices(virQEMUDriverPtr driver,
> virReportError(VIR_ERR_OPERATION_INVALID,
> _("SCSI device %s is already in use by "
> "other domain(s) as '%s'"),
> - tmp_shareable ? "shareable" : "non-shareable",
> - virSCSIDeviceGetName(tmp));
> + virSCSIDeviceGetName(tmp),
> + tmp_shareable ? "shareable" : "non-shareable");
> goto error;
> }
While this fixes wrong argument order, it is still wrong because it's
untranslatable. The code should be rewritten as
if (tmp_shareable) {
virReportError(VIR_ERR_OPERATION_INVALID,
_("SCSI device shareable is already in use by "
"other domain(s) as '%s'"),
virSCSIDeviceGetName(tmp));
} else {
virReportError(VIR_ERR_OPERATION_INVALID,
_("SCSI device non-shareable is already in use by "
"other domain(s) as '%s'"),
virSCSIDeviceGetName(tmp));
}
Not to mention that the error message itself doesn't make a lot of sense
to me... Did you wanted to say something else, e.g.:
if (scsi_shareable && !tmp_shareable) {
virReportError(VIR_ERR_OPERATION_INVALID,
_("Shareable SCSI device '%s' is already in "
"use by other domain(s) as non-shareable "
"device '%s'"),
virSCSIDeviceGetName(scsi),
virSCSIDeviceGetName(tmp));
} else if (!scsi_shareable) {
virReportError(VIR_ERR_OPERATION_INVALID,
_("Non-shareable SCSI device '%s' is already in "
"use by other domain(s) as device '%s'"),
virSCSIDeviceGetName(scsi),
virSCSIDeviceGetName(tmp));
}
Jirka
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list