Mercurial > hg > dhcpcd

changeset 12:8abfec17fbd6 draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Clean some style and stop using %m (use %s + strerror (errno) instead)
author Roy Marples <roy@marples.name>
date Mon, 04 Dec 2006 12:56:30 +0000
parents 7e30983769ae
children 6288393e4e1f
files client.c common.c configure.c dhcpcd.c
diffstat 4 files changed, 23 insertions(+), 21 deletions(-) [+]
line wrap: on
line diff
--- a/client.c	Mon Dec 04 12:40:14 2006 +0000
+++ b/client.c	Mon Dec 04 12:56:30 2006 +0000
@@ -105,7 +105,8 @@
       fd = open ("/dev/urandom", 0);
       if (fd < 0 || read (fd,  &seed, sizeof(seed)) < 0)
 	{
-	  logger (LOG_WARNING, "Could not load seed from /dev/urandom: %m");
+	  logger (LOG_WARNING, "Could not load seed from /dev/urandom: %s",
+		  strerror (errno));
 	  seed = time (0);
 	}
       if (fd >= 0)
--- a/common.c	Mon Dec 04 12:40:14 2006 +0000
+++ b/common.c	Mon Dec 04 12:56:30 2006 +0000
@@ -20,6 +20,7 @@
  */
 
 #include <sys/time.h>
+#include <errno.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ -43,7 +44,7 @@
 
   if (clock_gettime(CLOCK_MONOTONIC, &tp) == -1)
     {
-      logger (LOG_ERR, "Unable to get uptime: %m");
+      logger (LOG_ERR, "Unable to get uptime: %s", strerror (errno));
       return -1;
     }
 
--- a/configure.c	Mon Dec 04 12:40:14 2006 +0000
+++ b/configure.c	Mon Dec 04 12:56:30 2006 +0000
@@ -47,7 +47,7 @@
 
 static char *cleanmetas (char *cstr)
 {
-  if (!cstr)
+  if (! cstr)
     return "";
   
   register char *c = cstr;
@@ -62,13 +62,13 @@
 
 void exec_script (char *script, char *infofile, char *arg)
 {
-  if (!script || !infofile || !arg)
+  if (! script || ! infofile || ! arg)
     return;
 
   struct stat buf;
-  if (stat (script, &buf))
+  if (stat (script, &buf) < 0)
     {
-      if (strcmp (script, DEFAULT_SCRIPT))
+      if (strcmp (script, DEFAULT_SCRIPT) != 0)
 	logger (LOG_ERR, "`%s': %s", script, strerror (ENOENT));
       return;
     }
@@ -88,8 +88,8 @@
   if ((pid = fork ()) == 0)
     {
       if (execv (script, argc))
-	logger (LOG_ERR, "error executing \"%s %s %s\": %m",
-		argc[0], argc[1], argc[2]);
+	logger (LOG_ERR, "error executing \"%s %s %s\": %s",
+		argc[0], argc[1], argc[2], strerror (errno));
       exit (0);
     }
   else if (pid == -1)
@@ -103,19 +103,19 @@
   char resolvconf[PATH_MAX];
   address_t *address;
 
-  if (!stat ("/sbin/resolvconf", &buf))
+  if (stat ("/sbin/resolvconf", &buf) == 0)
     {
       logger (LOG_DEBUG, "sending DNS information to resolvconf");
       snprintf (resolvconf, PATH_MAX, "/sbin/resolvconf -a %s", ifname);
       f = popen (resolvconf, "w");
 
-      if (!f)
-	logger (LOG_ERR, "popen: %m");
+      if (! f)
+	logger (LOG_ERR, "popen: %s", strerror (errno));
     }
   else
     {
       if (! (f = fopen(RESOLVFILE, "w")))
-	logger (LOG_ERR, "fopen `%s': %m", RESOLVFILE);
+	logger (LOG_ERR, "fopen `%s': %s", RESOLVFILE, strerror (errno));
     }
 
   if (f) 
@@ -150,7 +150,7 @@
 {
   struct stat buf;
 
-  if (stat ("/sbin/resolvconf", &buf))
+  if (stat ("/sbin/resolvconf", &buf) < 0)
     return;
 
   logger (LOG_DEBUG, "removing information from resolvconf");
@@ -171,8 +171,8 @@
   if ((pid = fork ()) == 0)
     {
       if (execve (argc[0], argc, NULL))
-	logger (LOG_ERR, "error executing \"%s %s %s\": %m",
-		argc[0], argc[1], argc[2]);
+	logger (LOG_ERR, "error executing \"%s %s %s\": %s",
+		argc[0], argc[1], argc[2], strerror (errno));
       exit (0);
     }
   else if (pid == -1)
@@ -187,7 +187,7 @@
 
   if (! (f = fopen(NTPFILE, "w")))
     {
-      logger (LOG_ERR, "fopen `%s': %m", NTPFILE);
+      logger (LOG_ERR, "fopen `%s': %s", NTPFILE, strerror (errno));
       return -1;
     }
 	  
@@ -215,7 +215,7 @@
 
   if (! (f = fopen(NISFILE, "w")))
     {
-      logger (LOG_ERR, "fopen `%s': %m", NISFILE);
+      logger (LOG_ERR, "fopen `%s': %s", NISFILE, strerror (errno));
       return -1;
     }
 
@@ -248,7 +248,7 @@
 
   if ((f = fopen (iface->infofile, "w")) == NULL)
     {
-      logger (LOG_ERR, "fopen `%s': %m", iface->infofile);
+      logger (LOG_ERR, "fopen `%s': %s", iface->infofile, strerror (errno));
       return -1;
     }
 
@@ -356,7 +356,7 @@
   char *dname = NULL;
   int dnamel = 0;
  
-  if (!options || !iface || !dhcp)
+  if (! options || ! iface || ! dhcp)
     return -1;
 
   /* Remove old routes
--- a/dhcpcd.c	Mon Dec 04 12:40:14 2006 +0000
+++ b/dhcpcd.c	Mon Dec 04 12:56:30 2006 +0000
@@ -355,14 +355,14 @@
   if (mkdir (CONFIGDIR, S_IRUSR |S_IWUSR |S_IXUSR | S_IRGRP | S_IXGRP
 	     | S_IROTH | S_IXOTH) && errno != EEXIST )
     {
-      logger (LOG_ERR, "mkdir(\"%s\",0): %m\n", CONFIGDIR);
+      logger (LOG_ERR, "mkdir(\"%s\",0): %s\n", CONFIGDIR, strerror (errno));
       exit (EXIT_FAILURE);
     }
 
   if (mkdir (ETCDIR, S_IRUSR | S_IWUSR | S_IXUSR | S_IRGRP | S_IXGRP
 	     | S_IROTH | S_IXOTH) && errno != EEXIST )
     {
-      logger (LOG_ERR, "mkdir(\"%s\",0): %m\n", ETCDIR);
+      logger (LOG_ERR, "mkdir(\"%s\",0): %s\n", ETCDIR, strerror (errno));
       exit (EXIT_FAILURE);
     }