fix missing authentication on virsh shell commands

* src/virsh.c: fix missing authentication when command are issued
  from the shell, patch by Matthias Bolte
Daniel

diff --git a/ChangeLog b/ChangeLog
index ae5678f83ff505a9b468c279aae375c0ee52fa08..1ee016d8039786f75a76033de3951bd80fcf6816 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Mon Mar 23 17:33:17 CET 2009 Daniel Veillard <veillard@redhat.com>
+
+       * src/virsh.c: fix missing authentication when command are issued
+         from the shell, patch by Matthias Bolte
+
 Fri Mar 20 21:43:57 CET 2009 Daniel Veillard <veillard@redhat.com>
 
        * src/util.c: flag unused parameters, by Maximilian Wilhelm

diff --git a/src/virsh.c b/src/virsh.c
index 9a7b0ed6733f8e5f6b51696b334e9c19ba160279..abbd93b85bb62b345a824e4b535731e0ee9d59a3 100644 (file)
--- a/src/virsh.c
+++ b/src/virsh.c
@@ -468,13 +468,14 @@ cmdConnect(vshControl *ctl, const vshCmd *cmd)
     ctl->name = vshStrdup(ctl, vshCommandOptString(cmd, "name", NULL));
 
     if (!ro) {
-        ctl->conn = virConnectOpen(ctl->name);
         ctl->readonly = 0;
     } else {
-        ctl->conn = virConnectOpenReadOnly(ctl->name);
         ctl->readonly = 1;
     }
 
+    ctl->conn = virConnectOpenAuth(ctl->name, virConnectAuthPtrDefault,
+                                   ctl->readonly ? VIR_CONNECT_RO : 0);
+
     if (!ctl->conn)
         vshError(ctl, FALSE, "%s", _("Failed to connect to the hypervisor"));
 
@@ -2315,7 +2316,7 @@ cmdMigrate (vshControl *ctl, const vshCmd *cmd)
         flags |= VIR_MIGRATE_LIVE;
 
     /* Temporarily connect to the destination host. */
-    dconn = virConnectOpen (desturi);
+    dconn = virConnectOpenAuth (desturi, virConnectAuthPtrDefault, 0);
     if (!dconn) goto done;
 
     /* Migrate. */