[libvirt] [PATCH] Create directory for lease files if it's missing

Guido Günther agx at sigxcpu.org
Tue Jul 16 14:04:56 UTC 2013


On Thu, Jul 11, 2013 at 07:33:52PM +0200, Guido Günther wrote:
> If we don't autostart a network it's not being created.
> 
> Debian Bug http://bugs.debian.org/715200

Ping?
 -- Guido

> ---
>  src/nwfilter/nwfilter_dhcpsnoop.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/src/nwfilter/nwfilter_dhcpsnoop.c b/src/nwfilter/nwfilter_dhcpsnoop.c
> index 0965f63..3e9f046 100644
> --- a/src/nwfilter/nwfilter_dhcpsnoop.c
> +++ b/src/nwfilter/nwfilter_dhcpsnoop.c
> @@ -74,8 +74,9 @@
>  
>  #ifdef HAVE_LIBPCAP
>  
> -# define LEASEFILE LOCALSTATEDIR "/run/libvirt/network/nwfilter.leases"
> -# define TMPLEASEFILE LOCALSTATEDIR "/run/libvirt/network/nwfilter.ltmp"
> +# define LEASEFILE_DIR LOCALSTATEDIR "/run/libvirt/network/"
> +# define LEASEFILE LEASEFILE_DIR "nwfilter.leases"
> +# define TMPLEASEFILE LEASEFILE_DIR "nwfilter.ltmp"
>  
>  struct virNWFilterSnoopState {
>      /* lease file */
> @@ -1881,6 +1882,11 @@ virNWFilterSnoopLeaseFileRefresh(void)
>  {
>      int tfd;
>  
> +    if (virFileMakePathWithMode(LEASEFILE_DIR, 0700) < 0) {
> +        virReportError(errno, _("mkdir(\"%s\")"), LEASEFILE_DIR);
> +        return;
> +    }
> +
>      if (unlink(TMPLEASEFILE) < 0 && errno != ENOENT)
>          virReportSystemError(errno, _("unlink(\"%s\")"), TMPLEASEFILE);
>  
> -- 
> 1.8.3.2
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
> 




More information about the libvir-list mailing list