Browse Source

Merge branch 'bug3135' into maint-0.2.1

Nick Mathewson 14 years ago
parent
commit
d1c7f65a8e
2 changed files with 7 additions and 1 deletions
  1. 6 0
      changes/bug3135
  2. 1 1
      src/or/config.c

+ 6 - 0
changes/bug3135

@@ -0,0 +1,6 @@
+  o Minor bugfixes
+    - Do not crash when our configuration file becomes unreadable
+      (usually due to a permissions change) between when we start
+      up and when a controller calls SAVECONF.  Fixes bug 3135;
+      bugfix on 0.0.9pre6.
+

+ 1 - 1
src/or/config.c

@@ -4625,7 +4625,7 @@ write_configuration_file(const char *fname, or_options_t *options)
   switch (file_status(fname)) {
     case FN_FILE:
       old_val = read_file_to_str(fname, 0, NULL);
-      if (strcmpstart(old_val, GENERATED_FILE_PREFIX)) {
+      if (!old_val || strcmpstart(old_val, GENERATED_FILE_PREFIX)) {
         rename_old = 1;
       }
       tor_free(old_val);