Re: Fix umask on error path in qemudListenUnix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxx>
> 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@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]