Don't force Xen connections to readonly when non-root

diff --git a/ChangeLog b/ChangeLog
index f294dbfbc1a105cc1764db6d8c68a15b8a1d259f..d9a012e1dec7d55a0e8cf9e976794772365277b6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Tue Mar 11 10:21:53 EDT 2008 Daniel P. Berrange <berrange@redhat.com>
+
+       * src/virsh.c: Don't force connection to readonly as non-root
+       since with auth support we can have full connections.
+
 Tue Mar 11 10:17:53 EDT 2008 Daniel P. Berrange <berrange@redhat.com>
 
        * qemud/qemud.c: Don't run cleanup code when parent process is

diff --git a/src/virsh.c b/src/virsh.c
index c6bac127b72227d59c87b89c2f7047d11d501e24..cf0f12c50e9db5121545ee9cd99352707a811ba0 100644 (file)
--- a/src/virsh.c
+++ b/src/virsh.c
@@ -6045,13 +6045,6 @@ vshInit(vshControl * ctl)
     /* set up the library error handler */
     virSetErrorFunc(NULL, virshErrorHandler);
 
-#ifndef __MINGW32__
-    /* Force a non-root, Xen connection to readonly */
-    if ((ctl->name == NULL ||
-         !strcasecmp(ctl->name, "xen")) && ctl->uid != 0)
-         ctl->readonly = 1;
-#endif
-
     ctl->conn = virConnectOpenAuth(ctl->name,
                                    virConnectAuthPtrDefault,
                                    ctl->readonly ? VIR_CONNECT_RO : 0);