On Sun, 04 Dec 2016, Ioan-Adrian Ratiu <a...@adirat.com> wrote:
> Config files are currently read using glib's g_key_file_load_from_file
> function which is very inconvenient because it's limited by design to read
> only from "regular data files" in a filesystem. Because of this limitation
> notmuch can't read configs from pipes, fifos, sockets, stdin, etc. Not even
> "notmuch --config=/dev/stdin" works:
>
> Error reading configuration file /dev/stdin: Not a regular file
>
> So replace g_key_file_load_from_file with g_key_file_load_from_data which
> gives us much more freedom to read configs from multiple sources.
>
> This also helps the more security sensitive users: If someone has private
> information in the config file, it can be encrypted on disk, then decrypted
> in RAM and passed through a pipe directly to notmuch without the use of
> intermediate plain text files.
>
> Signed-off-by: Ioan-Adrian Ratiu <a...@adirat.com>
> ---
>  notmuch-config.c | 60 
> ++++++++++++++++++++++++++++++++++++++++++++------------
>  1 file changed, 47 insertions(+), 13 deletions(-)
>
> diff --git a/notmuch-config.c b/notmuch-config.c
> index bd52790..1ea897a 100644
> --- a/notmuch-config.c
> +++ b/notmuch-config.c
> @@ -205,33 +205,67 @@ get_username_from_passwd_file (void *ctx)
>  static notmuch_bool_t
>  get_config_from_file (notmuch_config_t *config, notmuch_bool_t create_new)
>  {
> +    // #define BUF_SIZE 200 // test line (use debug code or strace(1) to 
> verify)
> +    #define BUF_SIZE 4096
> +    char *config_str;
> +    int config_len = 0;
> +    int config_bufsize = BUF_SIZE;
> +    size_t len;
>      GError *error = NULL;
> -    notmuch_bool_t ret = FALSE;
>  
> -    if (g_key_file_load_from_file (config->key_file, config->filename,
> -                                G_KEY_FILE_KEEP_COMMENTS, &error))
> -     return TRUE;
> -
> -    if (error->domain == G_FILE_ERROR && error->code == G_FILE_ERROR_NOENT) {
> +    FILE *fp = fopen(config->filename, "r");
> +    if (fp == NULL) {
>       /* If create_new is true, then the caller is prepared for a
>        * default configuration file in the case of FILE NOT FOUND.
>        */
>       if (create_new) {
>           config->is_new = TRUE;
> -         ret = TRUE;
> +         return TRUE;
>       } else {
> -         fprintf (stderr, "Configuration file %s not found.\n"
> +         fprintf (stderr, "Error opening config file '%s': %s\n"
>                    "Try running 'notmuch setup' to create a configuration.\n",
> -                  config->filename);
> +                  config->filename, strerror(errno));
> +         return FALSE;
>       }
> -    } else {
> -     fprintf (stderr, "Error reading configuration file %s: %s\n",
> -              config->filename, error->message);
>      }
>  
> +    config_str = talloc_zero_array (config, char, config_bufsize);
> +    if (config_str == NULL) {
> +     fprintf (stderr, "Error reading '%s': Out of memory\n", 
> config->filename);
> +     return FALSE;
> +    }
> +
> +    while ((len = fread (config_str + config_len, 1,
> +                      config_bufsize - config_len, fp)) > 0) {
> +     config_len += len;
> +     if (config_len == config_bufsize) {
> +         config_bufsize += BUF_SIZE;
> +         config_str = talloc_realloc (config, config_str, char, 
> config_bufsize);
> +         if (config_str == NULL) {
> +             fprintf (stderr, "Error reading '%s': Failed to reallocate 
> memory\n",
> +                      config->filename);
> +             return FALSE;
> +         }
> +     }
> +    }
> +
> +    if (ferror (fp)) {
> +     fprintf (stderr, "Error reading '%s': I/O error\n", config->filename);
> +     return FALSE;
> +    }
> +
> +    fclose(fp);
> +
> +    if (g_key_file_load_from_data (config->key_file, config_str, 
> strlen(config_str),
> +                                G_KEY_FILE_KEEP_COMMENTS, &error))

I think you should free config_str regardless of the
g_key_file_load_from_data() return value.

> +     return TRUE;
> +
> +    fprintf (stderr, "Error parsing config file '%s': %s\n",
> +          config->filename, error->message);
> +
>      g_error_free (error);
>  
> -    return ret;
> +    return FALSE;
>  }
>  
>  /* Open the named notmuch configuration file. If the filename is NULL,
> -- 
> 2.10.2
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch
_______________________________________________
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch

Reply via email to