Re: virsh qemu-monitor-command broken with 0.9.5

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

 



On 09/21/2011 06:18 AM, Jason Krieg wrote:
Hi,

the virsh qemu-monitor-command is not working with libvirt 0.9.5

with git commit 85d2810823a31634b12145d6c196930b40425370

*opts_seen moved into the  != VSH_OT_ARGV if statement

so now opts_seen is only set if not VSH_OT_ARGV




diff -Nurp libvirt-0.9.5/tools/virsh.c libvirt-0.9.5.new/tools/virsh.c
--- libvirt-0.9.5/tools/virsh.c	2011-09-20 05:00:45.000000000 +0200
+++ libvirt-0.9.5.new/tools/virsh.c	2011-09-21 13:56:17.000000000 +0200
@@ -13916,8 +13916,8 @@ vshCmddefGetData(const vshCmdDef *cmd, u
      opt =&cmd->opts[i];
      if (opt->type != VSH_OT_ARGV) {
          *opts_need_arg&= ~(1<<  i);
-        *opts_seen |= 1<<  i;
      }
+    *opts_seen |= 1<<  i;
      return opt;

Thanks for the report, but reverting this change would break snapshot-create-as diskspec parsing.

I'll have to come up with a proper fix that works with both commands.

--
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]