Bug#510235: [Pkg-libvirt-maintainers] Bug#510235: libvirt-bin: virt-manager unable to connect to libvirtd as r/w (full VM management) on a local connection

[Guido Günther]

On Wed, Dec 31, 2008 at 01:50:09PM +0100, Guido Günther wrote:
> tags 510235 + patch
> 
> On Wed, Dec 31, 2008 at 01:20:33PM +0100, Guido Günther wrote:
> > The check in virt-manager is bogus anyways since you don't use policykit
> > but that's a different issue. I'll fix that.
> Attached patch fixes the issue for me.
Attached now.
 -- Guido

>From dd095406b303dbd83663e2114440397e277ef322 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Guido=20G=C3=BCnther?= <a...@sigxcpu.org>
Date: Wed, 31 Dec 2008 13:32:31 +0100
Subject: [PATCH] update policy file location

Closes: #510235
---
 src/virtManager/connection.py |   12 ++++++++++--
 1 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/virtManager/connection.py b/src/virtManager/connection.py
index e3fe863..0ac7d66 100644
--- a/src/virtManager/connection.py
+++ b/src/virtManager/connection.py
@@ -37,7 +37,10 @@ from virtManager.domain import vmmDomain
 from virtManager.network import vmmNetwork
 from virtManager.netdev import vmmNetDevice
 
-LIBVIRT_POLICY_FILE = "/usr/share/PolicyKit/policy/libvirtd.policy"
+LIBVIRT_POLICY_FILES = [
+    "/usr/share/PolicyKit/policy/libvirtd.policy",
+    "/usr/share/PolicyKit/policy/org.libvirt.unix.policy"
+]
 
 def get_local_hostname():
     try:
@@ -125,7 +128,12 @@ class vmmConnection(gobject.GObject):
 
         self.readOnly = readOnly
         if not self.is_remote() and os.getuid() != 0 and self.uri != "qemu:///session":
-            if not os.path.exists(LIBVIRT_POLICY_FILE):
+            hasPolkit = False
+            for f in LIBVIRT_POLICY_FILES:
+                if os.path.exists(f):
+                    hasPolkit = True
+
+            if not hasPolkit:
                 self.readOnly = True
 
         self.state = self.STATE_DISCONNECTED
-- 
1.6.0.3