Changeset 61ad0cb2

Timestamp:

May 5, 2014, 6:44:45 AM (10 years ago)

Author:

Anders Kaseorg <andersk@mit.edu>

Parents:

9ae6095 (diff), 94b9ee0 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge 94b9ee074ae76ef2e585605ccf56f030d02378a6 into 9ae6095ed62f67742f9d4dc1008180e8d23e7b12

Files:

• functions.c (modified) (2 diffs)
• util.c (modified) (2 diffs)
• variable.c (modified) (1 diff)
• zephyr.c (modified) (5 diffs)

functions.c

@@ -1227 +1227 @@
   time_t now;
   va_list ap;
-  va_start(ap, fmt);
 
   if (!owl_global_is_debug_fast(&g))
@@ -1242 +1241 @@
           (int) getpid(), tmpbuff, now - owl_global_get_starttime(&g));
   g_free(tmpbuff);
+
+  va_start(ap, fmt);
   vfprintf(file, fmt, ap);
+  va_end(ap);
+
   putc('\n', file);
   fflush(file);
-
-  va_end(ap);
 }
 

util.c

@@ -468 +468 @@
   if (fstat(fileno(old), &st) != 0) {
     owl_function_error("Cannot stat %s: %s", filename, strerror(errno));
+    fclose(old);
     return -1;
   }
@@ -473 +474 @@
   /* resolve symlinks, because link() fails on symlinks, at least on AFS */
   actual_filename = owl_util_recursive_resolve_link(filename);
-  if (actual_filename == NULL)
+  if (actual_filename == NULL) {
+    fclose(old);
     return -1; /* resolving the symlink failed, but we already logged this error */
+  }
 
   newfile = g_strdup_printf("%s.new", actual_filename);

variable.c

@@ -1038 +1038 @@
 CALLER_OWN char *owl_variable_bool_get_tostring_default(const owl_variable *v, void *dummy)
 {
-  bool val = owl_variable_get_bool(v);
-  if (val == 0) {
-    return g_strdup("off");
-  } else if (val == 1) {
-    return g_strdup("on");
-  } else {
-    return g_strdup("<invalid>");
-  }
+  return g_strdup(owl_variable_get_bool(v) ? "on" : "off");
 }
 

zephyr.c

@@ -283 +283 @@
 #ifdef HAVE_LIBZEPHYR
   FILE *file;
+  int fopen_errno;
   char *tmp, *start;
   char *buffer = NULL;
@@ -289 +290 @@
   int subSize = 1024;
   int count;
-  struct stat statbuff;
 
   subsfile = owl_zephyr_dotfile(".zephyr.subs", filename);
 
-  if (stat(subsfile, &statbuff) != 0) {
-    g_free(subsfile);
-    if (error_on_nofile == 1)
+  count = 0;
+  file = fopen(subsfile, "r");
+  fopen_errno = errno;
+  g_free(subsfile);
+  if (!file) {
+    if (error_on_nofile == 1 || fopen_errno != ENOENT)
       return -1;
     return 0;
   }
-
-  ZResetAuthentication();
-  count = 0;
-  file = fopen(subsfile, "r");
-  g_free(subsfile);
-  if (!file)
-    return -1;
 
   subs = g_new(ZSubscription_t, subSize);
@@ -348 +344 @@
     g_free(buffer);
 
+  ZResetAuthentication();
   return owl_zephyr_loadsubs_helper(subs, count);
 #else
@@ -412 +409 @@
   char *buffer = NULL;
   int count;
-  struct stat statbuff;
 
   subs = g_new(ZSubscription_t, numSubs);
   subsfile = owl_zephyr_dotfile(".anyone", filename);
 
-  if (stat(subsfile, &statbuff) == -1) {
-    g_free(subs);
-    g_free(subsfile);
-    return 0;
-  }
-
-  ZResetAuthentication();
   count = 0;
   file = fopen(subsfile, "r");
@@ -445 +434 @@
     fclose(file);
   } else {
+    g_free(subs);
     return 0;
   }
   g_free(buffer);
 
+  ZResetAuthentication();
   return owl_zephyr_loadsubs_helper(subs, count);
 #else