From 7cdf0a56708eea46726b9d0a3c17327371c48a94 Mon Sep 17 00:00:00 2001 From: Daniel Kahn Gillmor Date: Mon, 1 Feb 2016 15:39:49 +1900 Subject: [PATCH] [PATCH v3 04/16] Provide _notmuch_crypto_{set,get}_gpg_path --- 74/d19acc30eb14ae5997bf370d62ace0d421e52c | 237 ++++++++++++++++++++++ 1 file changed, 237 insertions(+) create mode 100644 74/d19acc30eb14ae5997bf370d62ace0d421e52c diff --git a/74/d19acc30eb14ae5997bf370d62ace0d421e52c b/74/d19acc30eb14ae5997bf370d62ace0d421e52c new file mode 100644 index 000000000..7e1beb98d --- /dev/null +++ b/74/d19acc30eb14ae5997bf370d62ace0d421e52c @@ -0,0 +1,237 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by arlo.cworth.org (Postfix) with ESMTP id 800D16DE1781 + for ; Sun, 31 Jan 2016 12:40:12 -0800 (PST) +X-Virus-Scanned: Debian amavisd-new at cworth.org +X-Spam-Flag: NO +X-Spam-Score: 0 +X-Spam-Level: +X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none] + autolearn=disabled +Received: from arlo.cworth.org ([127.0.0.1]) + by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id qgCMc9u7yGpG for ; + Sun, 31 Jan 2016 12:40:10 -0800 (PST) +Received: from che.mayfirst.org (che.mayfirst.org [209.234.253.108]) + by arlo.cworth.org (Postfix) with ESMTP id E20866DE0022 + for ; Sun, 31 Jan 2016 12:40:08 -0800 (PST) +Received: from fifthhorseman.net (ip-64-134-185-108.public.wayport.net + [64.134.185.108]) + by che.mayfirst.org (Postfix) with ESMTPSA id 77452F991 + for ; Sun, 31 Jan 2016 15:40:05 -0500 (EST) +Received: by fifthhorseman.net (Postfix, from userid 1000) + id 38F1720544; Sun, 31 Jan 2016 15:40:06 -0500 (EST) +From: Daniel Kahn Gillmor +To: Notmuch Mail +Subject: [PATCH v3 04/16] Provide _notmuch_crypto_{set,get}_gpg_path +Date: Sun, 31 Jan 2016 15:39:49 -0500 +Message-Id: <1454272801-23623-5-git-send-email-dkg@fifthhorseman.net> +X-Mailer: git-send-email 2.7.0.rc3 +In-Reply-To: <1454272801-23623-1-git-send-email-dkg@fifthhorseman.net> +References: <1454272801-23623-1-git-send-email-dkg@fifthhorseman.net> +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.20 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Sun, 31 Jan 2016 20:40:12 -0000 + +Use functions to access the gpg_path for a _notmuch_crypto_t object. +This lets us return sensible defaults based on the state of the user's +machine. +--- + notmuch-reply.c | 13 ++++++++++--- + notmuch-show.c | 12 ++++++++++-- + util/crypto.c | 48 +++++++++++++++++++++++++++++++++++++++++++++++- + util/crypto.h | 8 +++++++- + 4 files changed, 74 insertions(+), 7 deletions(-) + +diff --git a/notmuch-reply.c b/notmuch-reply.c +index eccfb32..793e6f9 100644 +--- a/notmuch-reply.c ++++ b/notmuch-reply.c +@@ -790,13 +790,15 @@ notmuch_reply_command (notmuch_config_t *config, int argc, char *argv[]) + .crypto = { + .verify = FALSE, + .decrypt = FALSE, +- .gpgpath = NULL ++ .gpg_path = NULL + } + }; + int format = FORMAT_DEFAULT; + int reply_all = TRUE; + struct sprinter *sp = NULL; +- ++ notmuch_status_t status; ++ const char *gpg_path = NULL; ++ + notmuch_opt_desc_t options[] = { + { NOTMUCH_OPT_KEYWORD, &format, "format", 'f', + (notmuch_keyword_t []){ { "default", FORMAT_DEFAULT }, +@@ -845,7 +847,12 @@ notmuch_reply_command (notmuch_config_t *config, int argc, char *argv[]) + return EXIT_FAILURE; + } + +- params.crypto.gpgpath = notmuch_config_get_crypto_gpg_path (config); ++ gpg_path = notmuch_config_get_crypto_gpg_path (config); ++ status = _notmuch_crypto_set_gpg_path (&(params.crypto), gpg_path); ++ if (status != NOTMUCH_STATUS_SUCCESS) { ++ fprintf (stderr, "Error: could not set gpg_path to '%s'.\n", gpg_path); ++ return EXIT_FAILURE; ++ } + + if (notmuch_database_open (notmuch_config_get_database_path (config), + NOTMUCH_DATABASE_MODE_READ_ONLY, ¬much)) +diff --git a/notmuch-show.c b/notmuch-show.c +index 3c91ece..096fd49 100644 +--- a/notmuch-show.c ++++ b/notmuch-show.c +@@ -1006,13 +1006,15 @@ notmuch_show_command (notmuch_config_t *config, int argc, char *argv[]) + .crypto = { + .verify = FALSE, + .decrypt = FALSE, +- .gpgpath = NULL ++ .gpg_path = NULL + }, + .include_html = FALSE + }; + int format_sel = NOTMUCH_FORMAT_NOT_SPECIFIED; + int exclude = EXCLUDE_TRUE; + int entire_thread = ENTIRE_THREAD_DEFAULT; ++ notmuch_status_t status; ++ const char *gpg_path = NULL; + + notmuch_opt_desc_t options[] = { + { NOTMUCH_OPT_KEYWORD, &format_sel, "format", 'f', +@@ -1130,7 +1132,13 @@ notmuch_show_command (notmuch_config_t *config, int argc, char *argv[]) + return EXIT_FAILURE; + } + +- params.crypto.gpgpath = notmuch_config_get_crypto_gpg_path (config); ++ ++ gpg_path = notmuch_config_get_crypto_gpg_path (config); ++ status = _notmuch_crypto_set_gpg_path (&(params.crypto), gpg_path); ++ if (status != NOTMUCH_STATUS_SUCCESS) { ++ fprintf (stderr, "Error: could not set gpg_path to '%s'.\n", gpg_path); ++ return EXIT_FAILURE; ++ } + + if (notmuch_database_open (notmuch_config_get_database_path (config), + NOTMUCH_DATABASE_MODE_READ_ONLY, ¬much)) +diff --git a/util/crypto.c b/util/crypto.c +index 151a352..c89ce8a 100644 +--- a/util/crypto.c ++++ b/util/crypto.c +@@ -21,7 +21,9 @@ + + #include "notmuch.h" + #include "crypto.h" ++#include "search-path.h" + #include ++#include + + #define ARRAY_SIZE(arr) (sizeof (arr) / sizeof (arr[0])) + +@@ -38,7 +40,7 @@ get_gpg_context (_notmuch_crypto_t *crypto, GMimeCryptoContext **ctx) + } + + /* TODO: GMimePasswordRequestFunc */ +- crypto->gpgctx = g_mime_gpg_context_new (NULL, crypto->gpgpath ? crypto->gpgpath : "gpg"); ++ crypto->gpgctx = g_mime_gpg_context_new (NULL, _notmuch_crypto_get_gpg_path(crypto)); + if (! crypto->gpgctx) { + return NOTMUCH_STATUS_FAILED_CRYPTO_CONTEXT_CREATION; + } +@@ -120,6 +122,47 @@ _notmuch_crypto_get_gmime_ctx_for_protocol (_notmuch_crypto_t *crypto, + return NOTMUCH_STATUS_UNKNOWN_CRYPTO_PROTOCOL; + } + ++const char* ++_notmuch_crypto_get_gpg_path (const _notmuch_crypto_t *crypto) ++{ ++ if (crypto->gpg_path) ++ return crypto->gpg_path; ++ ++#define try_gpg_path(z) if (test_for_executable(z)) return z ++ try_gpg_path("gpg2"); ++ try_gpg_path("gpg"); ++#undef try_gpg_path ++ return NULL; ++} ++ ++notmuch_status_t ++_notmuch_crypto_set_gpg_path (_notmuch_crypto_t *crypto, const char* gpg_path) ++{ ++ /* return success if this matches what is already configured */ ++ if ((!gpg_path && !crypto->gpg_path) || ++ (gpg_path && crypto->gpg_path && 0 == strcmp(gpg_path, crypto->gpg_path))) ++ return NOTMUCH_STATUS_SUCCESS; ++ ++ if (!gpg_path && !test_for_executable(gpg_path)) ++ return NOTMUCH_STATUS_FILE_ERROR; ++ ++ /* clear any existing gpgctx, since things are changing */ ++ if (crypto->gpgctx) { ++ g_object_unref (crypto->gpgctx); ++ crypto->gpgctx = NULL; ++ } ++ ++ if (crypto->gpg_path) { ++ talloc_free(crypto->gpg_path); ++ crypto->gpg_path = NULL; ++ } ++ ++ if (gpg_path) ++ crypto->gpg_path = talloc_strdup (NULL, gpg_path); ++ ++ return NOTMUCH_STATUS_SUCCESS; ++} ++ + void + _notmuch_crypto_cleanup (_notmuch_crypto_t *crypto) + { +@@ -132,4 +175,7 @@ _notmuch_crypto_cleanup (_notmuch_crypto_t *crypto) + g_object_unref (crypto->pkcs7ctx); + crypto->pkcs7ctx = NULL; + } ++ ++ talloc_free (crypto->gpg_path); ++ crypto->gpg_path = NULL; + } +diff --git a/util/crypto.h b/util/crypto.h +index 7cb0a39..70fc8ef 100644 +--- a/util/crypto.h ++++ b/util/crypto.h +@@ -11,7 +11,7 @@ typedef struct _notmuch_crypto { + GMimeCryptoContext* pkcs7ctx; + notmuch_bool_t verify; + notmuch_bool_t decrypt; +- const char *gpgpath; ++ char *gpg_path; + } _notmuch_crypto_t; + + +@@ -20,6 +20,12 @@ _notmuch_crypto_get_gmime_ctx_for_protocol (_notmuch_crypto_t *crypto, + const char *protocol, + GMimeCryptoContext **ctx); + ++notmuch_status_t ++_notmuch_crypto_set_gpg_path (_notmuch_crypto_t *crypto, const char *gpg_path); ++ ++const char * ++_notmuch_crypto_get_gpg_path (const _notmuch_crypto_t *crypto); ++ + void + _notmuch_crypto_cleanup (_notmuch_crypto_t *crypto); + +-- +2.7.0.rc3 + -- 2.26.2