]> git.cworth.org Git - obsolete/notmuch-old/commitdiff
cli: config: do not overwrite symlinks when saving config file
authorJani Nikula <jani@nikula.org>
Sun, 7 Apr 2013 17:15:03 +0000 (20:15 +0300)
committerDavid Bremner <bremner@debian.org>
Sun, 14 Apr 2013 21:38:07 +0000 (18:38 -0300)
Use realpath to canonicalize the config path before writing.

Previously 'notmuch setup' and 'notmuch config set' overwrote the
config file even if it was a symbolic link.

notmuch-config.c
test/config

index 66a1cdfbf2bf8fd438bc752984528e8a00ad2bcc..d9c2eb3fe65957917bef62bfab61dbe74da32a8a 100644 (file)
@@ -444,7 +444,7 @@ int
 notmuch_config_save (notmuch_config_t *config)
 {
     size_t length;
-    char *data;
+    char *data, *filename;
     GError *error = NULL;
 
     data = g_key_file_to_data (config->key_file, &length, NULL);
@@ -453,14 +453,30 @@ notmuch_config_save (notmuch_config_t *config)
        return 1;
     }
 
-    if (! g_file_set_contents (config->filename, data, length, &error)) {
-       fprintf (stderr, "Error saving configuration to %s: %s\n",
-                config->filename, error->message);
+    /* Try not to overwrite symlinks. */
+    filename = realpath (config->filename, NULL);
+    if (! filename) {
+       fprintf (stderr, "Error canonicalizing %s: %s\n", config->filename,
+                strerror (errno));
+       g_free (data);
+       return 1;
+    }
+
+    if (! g_file_set_contents (filename, data, length, &error)) {
+       if (strcmp (filename, config->filename) != 0) {
+           fprintf (stderr, "Error saving configuration to %s (-> %s): %s\n",
+                    config->filename, filename, error->message);
+       } else {
+           fprintf (stderr, "Error saving configuration to %s: %s\n",
+                    filename, error->message);
+       }
        g_error_free (error);
+       free (filename);
        g_free (data);
        return 1;
     }
 
+    free (filename);
     g_free (data);
     return 0;
 }
index 344eced36c9781b7d26a4006a7f4efb5a9be4d1b..ca4cf330eb57c2f9b014cb02acd1e42d2a64ac3e 100755 (executable)
@@ -78,7 +78,6 @@ test_expect_equal "$(notmuch --config=alt-config-link config get user.name)" \
     "Link Name"
 
 test_begin_subtest "Writing config file through symlink follows symlink"
-test_subtest_known_broken
 test_expect_equal "$(readlink alt-config-link)" "alt-config"
 
 test_done