]> xenbits.xensource.com Git - libvirt.git/commitdiff
net: leaseshelper: Don't crash if DNSMASQ doesn't provide lease expiry
authorPeter Krempa <pkrempa@redhat.com>
Mon, 16 Jun 2014 14:03:01 +0000 (16:03 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Tue, 17 Jun 2014 06:55:50 +0000 (08:55 +0200)
The value is provided via environment and causes a crash if not defined.

src/network/leaseshelper.c

index bf1842b87fb6ac61a9efa4c557100f0f157799bb..99c47a9b7b3624e0253a2bd4a223b2e982711a4a 100644 (file)
@@ -181,7 +181,8 @@ main(int argc, char **argv)
         goto cleanup;
 
     /* Removed extraneous trailing space in DNSMASQ_LEASE_EXPIRES (dnsmasq < 2.52) */
-    if (exptime[strlen(exptime) - 1] == ' ')
+    if (exptime &&
+        exptime[strlen(exptime) - 1] == ' ')
         exptime[strlen(exptime) - 1] = '\0';
 
     /* Check if it is an IPv6 lease */