[libvirt] Fix umask on error path in qemudListenUnix

Hi, Attached patch makes sure we reset the umask in qemudListenUnix on the error path. O.k. to apply? -- Guido

On Wed, Mar 09, 2011 at 02:20:59PM +0100, Guido Günther wrote:
Hi, Attached patch makes sure we reset the umask in qemudListenUnix on the error path. O.k. to apply? -- Guido
From 10c0088ca1ee55a2e44802f1f0185d7a8be907ee Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Guido=20G=C3=BCnther?= <agx@sigxcpu.org> Date: Wed, 9 Mar 2011 14:19:56 +0100 Subject: [PATCH] Fix umask on error path
--- daemon/libvirtd.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c index 9a5a53e..1e41feb 100644 --- a/daemon/libvirtd.c +++ b/daemon/libvirtd.c @@ -603,6 +603,7 @@ static int qemudListenUnix(struct qemud_server *server, if (bind(sock->fd, &sock->addr.data.sa, sock->addr.len) < 0) { VIR_ERROR(_("Failed to bind socket to '%s': %s"), path, virStrerror(errno, ebuf, sizeof ebuf)); + umask(oldmask); goto cleanup; } umask(oldmask);
Ah, right, good catch, ACK BTW how did you find this out, that sounds really hard to track, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

On Thu, Mar 10, 2011 at 07:43:05PM +0800, Daniel Veillard wrote:
On Wed, Mar 09, 2011 at 02:20:59PM +0100, Guido Günther wrote:
Hi, Attached patch makes sure we reset the umask in qemudListenUnix on the error path. O.k. to apply? -- Guido
From 10c0088ca1ee55a2e44802f1f0185d7a8be907ee Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Guido=20G=C3=BCnther?= <agx@sigxcpu.org> Date: Wed, 9 Mar 2011 14:19:56 +0100 Subject: [PATCH] Fix umask on error path
--- daemon/libvirtd.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c index 9a5a53e..1e41feb 100644 --- a/daemon/libvirtd.c +++ b/daemon/libvirtd.c @@ -603,6 +603,7 @@ static int qemudListenUnix(struct qemud_server *server, if (bind(sock->fd, &sock->addr.data.sa, sock->addr.len) < 0) { VIR_ERROR(_("Failed to bind socket to '%s': %s"), path, virStrerror(errno, ebuf, sizeof ebuf)); + umask(oldmask); goto cleanup; } umask(oldmask);
Ah, right, good catch, ACK BTW how did you find this out, that sounds really hard to track, By pure accident while looking at the code ;) Pushed now. Thanks, -- Guido
participants (2)
-
Daniel Veillard
-
Guido Günther