[libvirt] [PATCH] virt-admin: Properly fix the default session daemon URI to admin server
Erik Skultety
eskultet at redhat.com
Tue Aug 9 14:10:32 UTC 2016
On 09/08/16 16:04, Jiri Denemark wrote:
> On Tue, Aug 09, 2016 at 15:43:57 +0200, Erik Skultety wrote:
>> Commit 30ce2f0e tried to fix the issue with an incorrect session URI to admin
>> server but it messed up the checks:
>>
>> if (geteuid == 0 && VIR_STRDUP(*uristr, "libvirtd:///system") < 0)
>> return -1;
>> else if (VIR_STRDUP(*uristr, "libvirtd:///session") < 0)
>> return -1;
>>
>> So if a client executed with root privileges tries to connect, its euid is
>> checked (true) and the correct URI is successfully copied to @uristr (false),
>> therefore the 'else' branch is taken and @uristr is replaced by the session URI
>> which for root results in:
>> Failed to connect socket to '/root/.cache/libvirt/libvirt-admin-sock':
>> No such file or directory
>>
>> Signed-off-by: Erik Skultety <eskultet at redhat.com>
>> ---
>> src/libvirt-admin.c | 8 +++++---
>> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> ACK
>
> Jirka
Thanks, pushed.
Erik
More information about the libvir-list
mailing list