Re: [libvirt] Release of libvirt-0.9.10

2012-02-14 Thread Jiri Denemark
On Tue, Feb 14, 2012 at 14:17:18 +0800, Jun Koi wrote: On Tue, Feb 14, 2012 at 1:15 PM, Daniel Veillard veill...@redhat.com wrote:  As planned I made the release yesterday, it is tagged in git and available at:   ftp://libvirt.org/libvirt/ both as tarball and signed rpms for Fedora 16.

Re: [libvirt] [Libvirt-announce] Release of libvirt-0.9.10

2012-02-14 Thread Justin Clift
On 14/02/2012, at 4:15 PM, Daniel Veillard wrote: As planned I made the release yesterday, it is tagged in git and available at: ftp://libvirt.org/libvirt/ both as tarball and signed rpms for Fedora 16. Ouch. This fails to compile on OSX (unlike rc1), with this: CCLD

Re: [libvirt] Release of libvirt-0.9.10

2012-02-14 Thread Jun Koi
On Tue, Feb 14, 2012 at 4:01 PM, Jiri Denemark jdene...@redhat.com wrote: On Tue, Feb 14, 2012 at 14:17:18 +0800, Jun Koi wrote: On Tue, Feb 14, 2012 at 1:15 PM, Daniel Veillard veill...@redhat.com wrote:  As planned I made the release yesterday, it is tagged in git and available at:  

[libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread ajia
From: Alex Jia a...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return value. For details, please see the following link: RHBZ:

Re: [libvirt] [Libvirt-announce] Release of libvirt-0.9.10

2012-02-14 Thread Daniel Veillard
On Tue, Feb 14, 2012 at 07:18:32PM +1100, Justin Clift wrote: On 14/02/2012, at 4:15 PM, Daniel Veillard wrote: As planned I made the release yesterday, it is tagged in git and available at: ftp://libvirt.org/libvirt/ both as tarball and signed rpms for Fedora 16. Ouch. This fails

[libvirt] [PATCH 3/3] virsh: New command cmdChangeMedia

2012-02-14 Thread Osier Yang
One could use it to eject, insert, or update media of the CDROM or floppy drive. See the documents for more details. --- tools/virsh.c | 142 +++ tools/virsh.pod | 33 - 2 files changed, 172 insertions(+), 3 deletions(-) diff

[libvirt] [PATCH 0/3 v3] New command for changing media

2012-02-14 Thread Osier Yang
[PATCH 0/3] introduces two new helper functions: vshFindDisk is to find the disk XML node in xml doc with the type (indicates it's normal disk or changeable disk). VshPrepareDiskXML is to prepare the disk XML for disk changing commands' use, such as detach-disk. The purpose of this patch is to

[libvirt] [PATCH 1/3] virsh: Two new helper functions for disk device changes

2012-02-14 Thread Osier Yang
vshFindDisk is to find the disk node in xml doc with given source path or target of disk device, and type (indicates disk type, normal disk or changeable disk). vshPrepareDiskXML is to make changes on the disk node (e.g. create and insert the new source node for inserting media of CDROM drive).

[libvirt] [PATCH 2/3] virsh: Use vshFindDisk and vshPrepareDiskXML in cmdDetachDisk

2012-02-14 Thread Osier Yang
The first use of the two new helper functions. --- tools/virsh.c | 65 1 files changed, 10 insertions(+), 55 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 15578c7..3d94b9b 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@

[libvirt] RFC: Post-install behavior of libvirt(-client)

2012-02-14 Thread Martin Kletzander
Hello everyone, I have a small bug #786770 [1] assigned and I discussed this with few people from our team but I'm still not sure what should be the proper fix for this bug. To sumarize the bug, the problem described and discussed there [1] is that when there's a fresh installation of RHEL system

Re: [libvirt] [Libvirt-announce] Release of libvirt-0.9.10

2012-02-14 Thread Justin Clift
On 14/02/2012, at 7:31 PM, Daniel Veillard wrote: On Tue, Feb 14, 2012 at 07:18:32PM +1100, Justin Clift wrote: On 14/02/2012, at 4:15 PM, Daniel Veillard wrote: As planned I made the release yesterday, it is tagged in git and available at: ftp://libvirt.org/libvirt/ both as tarball and

Re: [libvirt] [PATCH 0/4 v2] New command for changing media

2012-02-14 Thread Paolo Bonzini
On 02/10/2012 05:35 AM, Osier Yang wrote: ping? On 2012年02月03日 17:23, Osier Yang wrote: [PATCH 0/4] introduces two new helper functions: vshFindDisk is to find the disk XML node in xml doc with the flags (indicates it's normal disk or changeable disk). VshPrepareDiskXML is to prepare the disk

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Peter Krempa
On 02/14/2012 09:31 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return value. For details, please see the following link:

[libvirt] [PATCH] storage: Allow runtime detection of scrub

2012-02-14 Thread Michal Privoznik
Currently, if scrub (used for wiping algorithms) is not present at compile time, we don't support any other wiping algorithms than zeroing, even if it was installed later. Switch to runtime detection instead. --- configure.ac | 30 --

[libvirt] Fail to import available VM image

2012-02-14 Thread Jun Koi
hi, i am trying to import an available KVM image to use with libvirt. i am not sure if this is the problem of libvirt or not, so bear in me if that is not the case. i run the following command, and got error: # virt-install -n winxp -r 800 -f img.winxp --accelerate --vnc --noautoconsole -v

[libvirt] [PATCH] correct security_require_confined default value

2012-02-14 Thread ajia
From: Alex Jia a...@redhat.com * src/qemu/qemu.conf: set security_require_confined default value to 0. Signed-off-by: Alex Jia a...@redhat.com --- src/qemu/qemu.conf |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu/qemu.conf b/src/qemu/qemu.conf index

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Alex Jia
On 02/14/2012 06:08 PM, Peter Krempa wrote: On 02/14/2012 09:31 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Peter Krempa
On 02/14/2012 11:48 AM, Alex Jia wrote: On 02/14/2012 06:08 PM, Peter Krempa wrote: On 02/14/2012 09:31 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Alex Jia
On 02/14/2012 06:17 PM, Jun Koi wrote: hi, i am trying to import an available KVM image to use with libvirt. i am not sure if this is the problem of libvirt or not, so bear in me if that is not the case. i run the following command, and got error: # virt-install -n winxp -r 800 -f img.winxp

[libvirt] [PATCHv2] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread ajia
From: Alex Jia a...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return value. For details, please see the following link: RHBZ:

Re: [libvirt] [PATCH 0/4 v2] New command for changing media

2012-02-14 Thread Osier Yang
On 02/14/2012 05:00 PM, Paolo Bonzini wrote: On 02/10/2012 05:35 AM, Osier Yang wrote: ping? On 2012年02月03日 17:23, Osier Yang wrote: [PATCH 0/4] introduces two new helper functions: vshFindDisk is to find the disk XML node in xml doc with the flags (indicates it's normal disk or changeable

Re: [libvirt] [PATCH] correct security_require_confined default value

2012-02-14 Thread Jiri Denemark
On Tue, Feb 14, 2012 at 18:38:24 +0800, a...@redhat.com wrote: From: Alex Jia a...@redhat.com * src/qemu/qemu.conf: set security_require_confined default value to 0. Signed-off-by: Alex Jia a...@redhat.com --- src/qemu/qemu.conf |2 +- 1 files changed, 1 insertions(+), 1

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Eric Blake
On 02/14/2012 03:08 AM, Peter Krempa wrote: On 02/14/2012 09:31 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return

Re: [libvirt] [PATCHv2] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Eric Blake
On 02/14/2012 04:19 AM, a...@redhat.com wrote: From: Alex Jia a...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return value. For details, please see the following

Re: [libvirt] [PATCH] correct security_require_confined default value

2012-02-14 Thread Eric Blake
On 02/14/2012 06:10 AM, Jiri Denemark wrote: On Tue, Feb 14, 2012 at 18:38:24 +0800, a...@redhat.com wrote: From: Alex Jia a...@redhat.com * src/qemu/qemu.conf: set security_require_confined default value to 0. Signed-off-by: Alex Jia a...@redhat.com --- src/qemu/qemu.conf |2 +- 1

Re: [libvirt] [PATCHv2] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Peter Krempa
On 02/14/2012 12:19 PM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix memory leaks and improve codes return value. For details, please see the following link:

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Peter Krempa
On 02/14/2012 02:11 PM, Eric Blake wrote: On 02/14/2012 03:08 AM, Peter Krempa wrote: On 02/14/2012 09:31 AM, a...@redhat.com wrote: From: Alex Jiaa...@redhat.com Detected by valgrind. Leaks are introduced in commit 17c7795. * python/libvirt-override.c (libvirt_virNodeGetMemoryStats): fix

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Jun Koi
On Tue, Feb 14, 2012 at 6:57 PM, Alex Jia a...@redhat.com wrote: On 02/14/2012 06:17 PM, Jun Koi wrote: hi, i am trying to import an available KVM image to use with libvirt. i am not sure if this is the problem of libvirt or not, so bear in me if that is not the case. i run the following

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Alex Jia
Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I want to know whether it works for you if you explicitly specify --connect qemu:///system with virt-install? I think a root reason probably is your disk image location, the log said it can't find storage volume, if you

Re: [libvirt] [PATCH] python: Avoid memory leaks on libvirt_virNodeGetMemoryStats

2012-02-14 Thread Alex Jia
Hi Eric, Thanks for your comment on v1 and v2 patch, I will update v3 based on v1 according to your advise. Regards, Alex - Original Message - From: Eric Blake ebl...@redhat.com To: Peter Krempa pkre...@redhat.com Cc: a...@redhat.com, libvir-list@redhat.com Sent: Tuesday, February 14,

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Jun Koi
On Tue, Feb 14, 2012 at 11:47 PM, Alex Jia a...@redhat.com wrote: Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I want to know whether it works for you if you explicitly specify --connect qemu:///system with virt-install? I think a root reason probably is your

Re: [libvirt] [PATCH v2] qemu: Implement DomainPMSuspendForDuration

2012-02-14 Thread Michal Privoznik
On 10.02.2012 22:25, Eric Blake wrote: On 02/10/2012 04:25 AM, Michal Privoznik wrote: via user agent. --- diff to v1: -allow mem and hybrid targets iff system_wakeup monitor command is presented Before I review too much of this, remember that there is still a pending discussion that

[libvirt] [PATCH 0/2 v3] Implement DomainPMSuspendForDuration for qemu

2012-02-14 Thread Michal Privoznik
This is basically v3 for: https://www.redhat.com/archives/libvir-list/2012-February/msg00533.html This time, I've split it into 2 patches so it can be reviewed easier. Michal Privoznik (2): qemu: Set capabilities based on supported monitor commands qemu: Implement DomainPMSuspendForDuration

[libvirt] [PATCH 1/2 v3] qemu: Set capabilities based on supported monitor commands

2012-02-14 Thread Michal Privoznik
In the future (my next patch in fact) we may want to make decisions depending on qemu having a monitor command or not. Therefore, we want to set qemuCaps flag instead of querying on the monitor each time we are about to make that decision. --- src/qemu/qemu_capabilities.c |1 +

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Laine Stump
On 02/14/2012 11:01 AM, Jun Koi wrote: On Tue, Feb 14, 2012 at 11:47 PM, Alex Jia a...@redhat.com wrote: Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I want to know whether it works for you if you explicitly specify --connect qemu:///system with virt-install? I

[libvirt] [working patch] Re: virsh bash completion file

2012-02-14 Thread Serge Hallyn
Ok, here is a patch which works for me. (I do see that I'll need to change some of the files edited to deal with the pre-autoconf tree, but I wanted to get it out there) I considered adding a name field to the virDriver entries, but the WITH_REMOTE combinations would get trick, and the cheap way

Re: [libvirt] [PATCH 1/2 v3] qemu: Set capabilities based on supported monitor commands

2012-02-14 Thread Eric Blake
On 02/14/2012 09:45 AM, Michal Privoznik wrote: In the future (my next patch in fact) we may want to make decisions depending on qemu having a monitor command or not. Love that wording :) Therefore, we want to set qemuCaps flag instead of querying on the monitor each time we are about to

Re: [libvirt] [PATCH 2/2 v3] qemu: Implement DomainPMSuspendForDuration

2012-02-14 Thread Eric Blake
On 02/14/2012 09:45 AM, Michal Privoznik wrote: via user agent. Allow targets mem hybrid iff system_wakeup monitor command is available. --- src/qemu/qemu_agent.c | 31 + src/qemu/qemu_agent.h |2 + src/qemu/qemu_driver.c | 87

[libvirt] [PATCHv3 0/3] Xen: Fix clock handling

2012-02-14 Thread Philipp Hahn
Before version 3.1 xen only implemented clock/@offset='utc' and 'localtime'. With the introduction of managed domains in 3.1 xend keeps track of the rtc_timeoffset, even over reboots. This translates to  libvirts clock/@offset='variable' variant. Be advised that only HV domains have a RTC. In

Re: [libvirt] [PATCHv2 0/3] Xen: Fix clock handling

2012-02-14 Thread Philipp Hahn
Hello Eric, On Monday 13 February 2012 18:43:21 Eric Blake wrote: 2. Instead of implementing a second variant of offset='variable' it would be enough to just add an attribute reset='true' (or this_is_not_from_a_buggy_libvirt_and_I_really_need_the_reset='true') to dumpxml, when a driver

[libvirt] [PATCHv3 1/3] Support clock=variable relative to localtime

2012-02-14 Thread Philipp Hahn
Since Xen 3.1 the clock=variable semantic is supported. In addition to qemu/kvm Xen also knows about a variant where the offset is relative to 'localtime' instead of 'utc'. Extends the libvirt structure with a flag 'basis' to specify, if the offset is relative to 'localtime' or 'utc'. Extends

[libvirt] [PATCHv3 2/3] Xen: Fix clock handling

2012-02-14 Thread Philipp Hahn
XenD-3.1 introduced managed domains. HV-domains have rtc_timeoffset (hgd24f37b31030 from 2007-04-03), which tracks the offset between the hypervisors clock and the domains RTC, and is persisted by XenD. In combination with localtime=1 this had a bug until XenD-3.4 (hg5d701be7c37b from 2009-04-01)

[libvirt] [PATCH] Don't add SPICE TLS channels when TLS is disabled

2012-02-14 Thread Christophe Fergeau
It's possible to disable SPICE TLS in qemu.conf. When this happens, libvirt ignores any SPICE TLS port or x509 directory that may have been set when it builds the qemu command line to use. However, it's not ignoring the secure channels that may have been set and adds tls-channel arguments to qemu

Re: [libvirt] [PATCHv3 1/3] Support clock=variable relative to localtime

2012-02-14 Thread Philipp Hahn
Hi, forgot one line to commit On Monday 06 February 2012 14:59:16 Philipp Hahn wrote: diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index aa4b32d..5508fc9 100644 ... @@ -7467,10 +7471,52 @@ static virDomainDefPtr virDomainDefParseXML(virCapsPtr caps, def-clock.offset =

[libvirt] [PATCHv4 1/3] Support clock=variable relative to localtime

2012-02-14 Thread Philipp Hahn
Since Xen 3.1 the clock=variable semantic is supported. In addition to qemu/kvm Xen also knows about a variant where the offset is relative to 'localtime' instead of 'utc'. Extends the libvirt structure with a flag 'basis' to specify, if the offset is relative to 'localtime' or 'utc'. Extends

[libvirt] [PATCHv4 2/3] Xen: Fix clock handling

2012-02-14 Thread Philipp Hahn
XenD-3.1 introduced managed domains. HV-domains have rtc_timeoffset (hgd24f37b31030 from 2007-04-03), which tracks the offset between the hypervisors clock and the domains RTC, and is persisted by XenD. In combination with localtime=1 this had a bug until XenD-3.4 (hg5d701be7c37b from 2009-04-01)

[libvirt] [PATCHv4 0/3] Xen: Fix clock handling

2012-02-14 Thread Philipp Hahn
Before version 3.1 xen only implemented clock/@offset='utc' and 'localtime'. With the introduction of managed domains in 3.1 xend keeps track of the rtc_timeoffset, even over reboots. This translates to  libvirts clock/@offset='variable' variant. Be advised that only HV domains have a RTC. In

[libvirt] [PATCH] snapshot: fix snapshot deletion

2012-02-14 Thread Eric Blake
Bug introduced in commit 35abced. On an inactive domain, $ virsh snapshot-create-as dom snap $ virsh snapshot-create dom $ virsh snapshot-create dom $ virsh snapshot-delete --children dom snap could crash libvirtd, due to a use-after-free that results when the callback freed the current element

Re: [libvirt] [PATCH] Don't add SPICE TLS channels when TLS is disabled

2012-02-14 Thread Eric Blake
On 02/14/2012 11:04 AM, Christophe Fergeau wrote: It's possible to disable SPICE TLS in qemu.conf. When this happens, libvirt ignores any SPICE TLS port or x509 directory that may have been set when it builds the qemu command line to use. However, it's not ignoring the secure channels that may

Re: [libvirt] [PATCH] storage: Allow runtime detection of scrub

2012-02-14 Thread Eric Blake
On 02/14/2012 03:12 AM, Michal Privoznik wrote: Currently, if scrub (used for wiping algorithms) is not present at compile time, we don't support any other wiping algorithms than zeroing, even if it was installed later. Switch to runtime detection instead. --- configure.ac |

Re: [libvirt] RFC: Post-install behavior of libvirt(-client)

2012-02-14 Thread Eric Blake
On 02/14/2012 01:37 AM, Martin Kletzander wrote: Hello everyone, I have a small bug #786770 [1] assigned and I discussed this with few people from our team but I'm still not sure what should be the proper fix for this bug. Thinking about this a little more, I see two more unconsistencies

Re: [libvirt] [PATCH v2] RFC Libvirt + Openvswitch

2012-02-14 Thread Eric Blake
On 02/13/2012 10:56 PM, Laine Stump wrote: But because an interfaceid is auto-generated by the parser when one isn't given, and a common parser function for virtualport is used by the domain and network parsers, this will result in an interfaceid being filled in when the network is defined,

[libvirt] [PATCH] daemon: plug memory leak

2012-02-14 Thread Eric Blake
Leak introduced in commit bb2eddc6. * daemon/remote.c (remoteDispatchAuthPolkit): Also free pkout on success. --- daemon/remote.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/daemon/remote.c b/daemon/remote.c index 1cea942..9c61306 100644 --- a/daemon/remote.c +++

[libvirt] [PATCH] daemon: fix logic bug with virAsprintf

2012-02-14 Thread Eric Blake
Regression introduced in commit 7033c5f2, due to improper conversion from snprintf to virAsprintf. * daemon/remote.c (remoteDispatchAuthList): Check return value correctly. --- This one's embarrassing. I think I broke polkit authorization in 0.9.10. :( daemon/remote.c |2 +- 1 files

Re: [libvirt] [PATCH v2] RFC Libvirt + Openvswitch

2012-02-14 Thread Laine Stump
On 02/14/2012 05:33 PM, Eric Blake wrote: On 02/13/2012 10:56 PM, Laine Stump wrote: But because an interfaceid is auto-generated by the parser when one isn't given, and a common parser function for virtualport is used by the domain and network parsers, this will result in an interfaceid

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Alex Jia
On 02/15/2012 01:01 AM, Laine Stump wrote: On 02/14/2012 11:01 AM, Jun Koi wrote: On Tue, Feb 14, 2012 at 11:47 PM, Alex Jiaa...@redhat.com wrote: Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I want to know whether it works for you if you explicitly specify

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Jun Koi
On Wed, Feb 15, 2012 at 1:01 AM, Laine Stump la...@laine.org wrote: On 02/14/2012 11:01 AM, Jun Koi wrote: On Tue, Feb 14, 2012 at 11:47 PM, Alex Jia a...@redhat.com wrote: Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I want to know whether it works for you if

Re: [libvirt] Fail to import available VM image

2012-02-14 Thread Alex Jia
On 02/15/2012 10:53 AM, Jun Koi wrote: On Wed, Feb 15, 2012 at 1:01 AM, Laine Stumpla...@laine.org wrote: On 02/14/2012 11:01 AM, Jun Koi wrote: On Tue, Feb 14, 2012 at 11:47 PM, Alex Jiaa...@redhat.com wrote: Hi Jun, I assume you haven't changed libvirt default URI, it may be a issue, I

[libvirt] question about guest status

2012-02-14 Thread Wen Congyang
Hi all: If the guest runs on xen, we can know the guest OS paniced, and the guest status is paniced. But if the guest runs on qemu/kvm, we need the same feature. And the feature to know guest-is-panic is a very very good feature for enterprise. We have some ways to know that guest has paniced:

Re: [libvirt] [PATCH] daemon: fix logic bug with virAsprintf

2012-02-14 Thread Wen Congyang
At 02/15/2012 07:03 AM, Eric Blake Wrote: Regression introduced in commit 7033c5f2, due to improper conversion from snprintf to virAsprintf. * daemon/remote.c (remoteDispatchAuthList): Check return value correctly. --- This one's embarrassing. I think I broke polkit authorization in