[Libvir] [PATCH] add error message for the case of the vcpupin command fails.
Richard W.M. Jones
rjones at redhat.com
Tue Mar 4 11:03:25 UTC 2008
OK, I'm going to qualify that a bit. Is it always safe to access
arg->data? I think probably it is, but I'm not sure so I modified the
patch slightly just to print an error message like this:
$ src/virsh -c test:///default vcpupin test AAA 0,1
error: vcpupin: Invalid or missing vCPU number.
Rich.
PS. Please try to attach your patches instead of pasting them inline.
--
Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones
virt-top is 'top' for virtual machines. Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://et.redhat.com/~rjones/virt-top
-------------- next part --------------
Index: src/virsh.c
===================================================================
RCS file: /data/cvs/libvirt/src/virsh.c,v
retrieving revision 1.134
diff -u -r1.134 virsh.c
--- src/virsh.c 27 Feb 2008 16:14:44 -0000 1.134
+++ src/virsh.c 4 Mar 2008 11:05:56 -0000
@@ -1723,6 +1723,7 @@
vcpu = vshCommandOptInt(cmd, "vcpu", &vcpufound);
if (!vcpufound) {
+ vshError(ctl, FALSE, _("vcpupin: Invalid or missing vCPU number."));
virDomainFree(dom);
return FALSE;
}
More information about the libvir-list
mailing list