Fix writing to /etc/resolv.conf when resolvconf not present and add more debug statements

diff --git a/ChangeLog b/ChangeLog
index 4032b02683ea14aabeb96bee7e47434fb1ebe4d4..53be0849444b3ff6e89c46dbfa86f4cf752c0e76 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,5 @@
+Add more debug messages.
+Fix writing to resolv.conf when resolvconf not present.
 Include linux/if_addr.h for 2.6.19+ kernels, thanks to AlexExtreme.
 
 dhcpcd-3.0.2

diff --git a/client.c b/client.c
index 87f8ee3d5b104349ad5fcabb6a548d047c1475c2..c84c63d9ccd3200dce5f1906d5a164ddde30d1c0 100644 (file)
--- a/client.c
+++ b/client.c
@@ -82,9 +82,10 @@
 
 static int daemonise (char *pidfile)
 {
+  logger (LOG_DEBUG, "forking to background");
   if (daemon (0, 0) < 0)
     {
-      logger (LOG_ERR, "unable to daemonise: %s", strerror (errno));
+      logger (LOG_ERR, "daemon: %s", strerror (errno));
       return -1;
     }
 

diff --git a/configure.c b/configure.c
index 9121f5a5b0b868a7b24b6afb6015d5ffb611f592..11216d42fe10240f0eae697b80f3bcc637075ce7 100644 (file)
--- a/configure.c
+++ b/configure.c
@@ -96,13 +96,14 @@ void exec_script (char *script, char *infofile, char *arg)
     logger (LOG_ERR, "fork: %s", strerror (errno));
 }
 
-static int make_resolv (char *ifname, dhcp_t *dhcp, int wait)
+static int make_resolv (char *ifname, dhcp_t *dhcp)
 {
   FILE *f;
   struct stat buf;
   char resolvconf[PATH_MAX];
   address_t *address;
 
+  memset (&resolvconf, 0, sizeof (resolvconf));
   if (stat ("/sbin/resolvconf", &buf) == 0)
     {
       logger (LOG_DEBUG, "sending DNS information to resolvconf");
@@ -114,6 +115,7 @@ static int make_resolv (char *ifname, dhcp_t *dhcp, int wait)
     }
   else
     {
+      logger (LOG_DEBUG, "writing "RESOLVFILE);
       if (! (f = fopen(RESOLVFILE, "w")))
        logger (LOG_ERR, "fopen `%s': %s", RESOLVFILE, strerror (errno));
     }
@@ -185,6 +187,7 @@ static int make_ntp (char *ifname, dhcp_t *dhcp)
   address_t *address;
   char *a;
 
+  logger (LOG_DEBUG, "writing "NTPFILE);
   if (! (f = fopen(NTPFILE, "w")))
     {
       logger (LOG_ERR, "fopen `%s': %s", NTPFILE, strerror (errno));
@@ -213,6 +216,7 @@ static int make_nis (char *ifname, dhcp_t *dhcp)
   address_t *address;
   char prefix[256] = {0};
 
+  logger (LOG_DEBUG, "writing "NISFILE);
   if (! (f = fopen(NISFILE, "w")))
     {
       logger (LOG_ERR, "fopen `%s': %s", NISFILE, strerror (errno));
@@ -246,6 +250,7 @@ static int write_info(interface_t *iface, dhcp_t *dhcp)
   route_t *route;
   address_t *address;
 
+  logger (LOG_DEBUG, "writing %s", iface->infofile);
   if ((f = fopen (iface->infofile, "w")) == NULL)
     {
       logger (LOG_ERR, "fopen `%s': %s", iface->infofile, strerror (errno));
@@ -483,7 +488,9 @@ int configure (options_t *options, interface_t *iface, dhcp_t *dhcp)
     }
 
   if (options->dodns && dhcp->dnsservers)
-    make_resolv(iface->name, dhcp, (options->dohostname && !dhcp->hostname));
+    make_resolv(iface->name, dhcp);
+  else
+    logger (LOG_DEBUG, "no dns information to write");
 
   if (options->dontp && dhcp->ntpservers)
     make_ntp(iface->name, dhcp);
@@ -492,7 +499,7 @@ int configure (options_t *options, interface_t *iface, dhcp_t *dhcp)
     make_nis(iface->name, dhcp);
 
   /* Now we have made a resolv.conf we can obtain a hostname if we need one */
-  if (options->dohostname && !dhcp->hostname)
+  if (options->dohostname && ! dhcp->hostname)
     {
       he = gethostbyaddr (inet_ntoa (dhcp->address),
                          sizeof (struct in_addr), AF_INET);
@@ -509,14 +516,16 @@ int configure (options_t *options, interface_t *iface, dhcp_t *dhcp)
 
   gethostname (curhostname, sizeof (curhostname));
   
-  if (options->dohostname || !strlen (curhostname) 
-      || !strcmp (curhostname, "(none)") || !strcmp (curhostname, "localhost"))
+  if (options->dohostname
+      || strlen (curhostname) == 0
+      || strcmp (curhostname, "(none)") == 0
+      || strcmp (curhostname, "localhost") == 0)
     {
       if (dhcp->hostname)
        strcpy ((char *) newhostname, dhcp->hostname); 
 
       sethostname ((char *) newhostname, strlen ((char *) newhostname));
-      logger (LOG_INFO, "setting hostname to %s", newhostname);
+      logger (LOG_INFO, "setting hostname to `%s'", newhostname);
     }
 
   write_info (iface, dhcp);