From 9de63004ecb863d7de1763231d4fb801084377fe Mon Sep 17 00:00:00 2001 From: Jameson Graef Rollins Date: Sun, 27 May 2012 11:45:45 +1700 Subject: [PATCH] [PATCH v5 6/7] cli: new crypto verify flag to handle verification --- d5/bc15e53498e481dae2c3b8cc4b617225e9e8f6 | 189 ++++++++++++++++++++++ 1 file changed, 189 insertions(+) create mode 100644 d5/bc15e53498e481dae2c3b8cc4b617225e9e8f6 diff --git a/d5/bc15e53498e481dae2c3b8cc4b617225e9e8f6 b/d5/bc15e53498e481dae2c3b8cc4b617225e9e8f6 new file mode 100644 index 000000000..79c599eb8 --- /dev/null +++ b/d5/bc15e53498e481dae2c3b8cc4b617225e9e8f6 @@ -0,0 +1,189 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by olra.theworths.org (Postfix) with ESMTP id 85A9C431FBC + for ; Sat, 26 May 2012 11:46:01 -0700 (PDT) +X-Virus-Scanned: Debian amavisd-new at olra.theworths.org +X-Spam-Flag: NO +X-Spam-Score: -2.3 +X-Spam-Level: +X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5 + tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled +Received: from olra.theworths.org ([127.0.0.1]) + by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id lurVCWQuOfUG for ; + Sat, 26 May 2012 11:45:59 -0700 (PDT) +Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu + [131.215.239.19]) + by olra.theworths.org (Postfix) with ESMTP id F2615429E2F + for ; Sat, 26 May 2012 11:45:55 -0700 (PDT) +Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1]) + by earth-doxen-postvirus (Postfix) with ESMTP id 2EDA866E00E5 + for ; Sat, 26 May 2012 11:45:55 -0700 (PDT) +X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new +Received: from finestructure.net (unknown [76.89.192.57]) + (Authenticated sender: jrollins) + by earth-doxen-submit (Postfix) with ESMTP id 1E92466E00C0 + for ; Sat, 26 May 2012 11:45:53 -0700 (PDT) +Received: by finestructure.net (Postfix, from userid 1000) + id 753A0B83; Sat, 26 May 2012 11:45:51 -0700 (PDT) +From: Jameson Graef Rollins +To: Notmuch Mail +Subject: [PATCH v5 6/7] cli: new crypto verify flag to handle verification +Date: Sat, 26 May 2012 11:45:45 -0700 +Message-Id: <1338057946-29209-7-git-send-email-jrollins@finestructure.net> +X-Mailer: git-send-email 1.7.10 +In-Reply-To: <1338057946-29209-6-git-send-email-jrollins@finestructure.net> +References: <1338057946-29209-1-git-send-email-jrollins@finestructure.net> + <1338057946-29209-2-git-send-email-jrollins@finestructure.net> + <1338057946-29209-3-git-send-email-jrollins@finestructure.net> + <1338057946-29209-4-git-send-email-jrollins@finestructure.net> + <1338057946-29209-5-git-send-email-jrollins@finestructure.net> + <1338057946-29209-6-git-send-email-jrollins@finestructure.net> +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.13 +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: Sat, 26 May 2012 18:46:01 -0000 + +Use this flag rather than depend on the existence of an initialized +gpgctx, to determine whether we should verify a multipart/signed. We +will be moving to create the ctx lazily, so we don't want to depend on +it being previously initialized if it's not needed. +--- + mime-node.c | 5 ++--- + notmuch-client.h | 8 ++++---- + notmuch-reply.c | 1 + + notmuch-show.c | 14 +++++++++++--- + 4 files changed, 18 insertions(+), 10 deletions(-) + +diff --git a/mime-node.c b/mime-node.c +index a838224..73e28c5 100644 +--- a/mime-node.c ++++ b/mime-node.c +@@ -183,8 +183,7 @@ _mime_node_create (mime_node_t *parent, GMimeObject *part) + } + + /* Handle PGP/MIME parts */ +- if (GMIME_IS_MULTIPART_ENCRYPTED (part) +- && node->ctx->crypto->gpgctx && node->ctx->crypto->decrypt) { ++ if (GMIME_IS_MULTIPART_ENCRYPTED (part) && node->ctx->crypto->decrypt) { + if (node->nchildren != 2) { + /* this violates RFC 3156 section 4, so we won't bother with it. */ + fprintf (stderr, "Error: %d part(s) for a multipart/encrypted " +@@ -218,7 +217,7 @@ _mime_node_create (mime_node_t *parent, GMimeObject *part) + (err ? err->message : "no error explanation given")); + } + } +- } else if (GMIME_IS_MULTIPART_SIGNED (part) && node->ctx->crypto->gpgctx) { ++ } else if (GMIME_IS_MULTIPART_SIGNED (part) && node->ctx->crypto->verify) { + if (node->nchildren != 2) { + /* this violates RFC 3156 section 5, so we won't bother with it. */ + fprintf (stderr, "Error: %d part(s) for a multipart/signed message " +diff --git a/notmuch-client.h b/notmuch-client.h +index 962c747..0f29a83 100644 +--- a/notmuch-client.h ++++ b/notmuch-client.h +@@ -79,6 +79,7 @@ typedef struct notmuch_show_format { + + typedef struct notmuch_crypto { + notmuch_crypto_context_t* gpgctx; ++ notmuch_bool_t verify; + notmuch_bool_t decrypt; + } notmuch_crypto_t; + +@@ -350,10 +351,9 @@ struct mime_node { + }; + + /* Construct a new MIME node pointing to the root message part of +- * message. If crypto->gpgctx is non-NULL, it will be used to verify +- * signatures on any child parts. If crypto->decrypt is true, then +- * crypto.gpgctx will additionally be used to decrypt any encrypted +- * child parts. ++ * message. If crypto->verify is true, signed child parts will be ++ * verified. If crypto->decrypt is true, encrypted child parts will be ++ * decrypted. + * + * Return value: + * +diff --git a/notmuch-reply.c b/notmuch-reply.c +index 6f368c9..1ab3db9 100644 +--- a/notmuch-reply.c ++++ b/notmuch-reply.c +@@ -707,6 +707,7 @@ notmuch_reply_command (void *ctx, int argc, char *argv[]) + notmuch_show_params_t params = { + .part = -1, + .crypto = { ++ .verify = FALSE, + .decrypt = FALSE + } + }; +diff --git a/notmuch-show.c b/notmuch-show.c +index fb5e9b6..3c06792 100644 +--- a/notmuch-show.c ++++ b/notmuch-show.c +@@ -987,11 +987,11 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[])) + .part = -1, + .omit_excluded = TRUE, + .crypto = { ++ .verify = FALSE, + .decrypt = FALSE + } + }; + int format_sel = NOTMUCH_FORMAT_NOT_SPECIFIED; +- notmuch_bool_t verify = FALSE; + int exclude = EXCLUDE_TRUE; + + notmuch_opt_desc_t options[] = { +@@ -1008,7 +1008,7 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[])) + { NOTMUCH_OPT_INT, ¶ms.part, "part", 'p', 0 }, + { NOTMUCH_OPT_BOOLEAN, ¶ms.entire_thread, "entire-thread", 't', 0 }, + { NOTMUCH_OPT_BOOLEAN, ¶ms.crypto.decrypt, "decrypt", 'd', 0 }, +- { NOTMUCH_OPT_BOOLEAN, &verify, "verify", 'v', 0 }, ++ { NOTMUCH_OPT_BOOLEAN, ¶ms.crypto.verify, "verify", 'v', 0 }, + { 0, 0, 0, 0, 0 } + }; + +@@ -1018,6 +1018,10 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[])) + return 1; + } + ++ /* decryption implies verification */ ++ if (params.crypto.decrypt) ++ params.crypto.verify = TRUE; ++ + if (format_sel == NOTMUCH_FORMAT_NOT_SPECIFIED) { + /* if part was requested and format was not specified, use format=raw */ + if (params.part >= 0) +@@ -1052,7 +1056,7 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[])) + break; + } + +- if (params.crypto.decrypt || verify) { ++ if (params.crypto.decrypt || params.crypto.verify) { + #ifdef GMIME_ATLEAST_26 + /* TODO: GMimePasswordRequestFunc */ + params.crypto.gpgctx = g_mime_gpg_context_new (NULL, "gpg"); +@@ -1063,6 +1067,10 @@ notmuch_show_command (void *ctx, unused (int argc), unused (char *argv[])) + if (params.crypto.gpgctx) { + g_mime_gpg_context_set_always_trust ((GMimeGpgContext*) params.crypto.gpgctx, FALSE); + } else { ++ /* If we fail to create the gpgctx set the verify and ++ * decrypt flags to FALSE so we don't try to do any ++ * further verification or decryption */ ++ params.crypto.verify = FALSE; + params.crypto.decrypt = FALSE; + fprintf (stderr, "Failed to construct gpg context.\n"); + } +-- +1.7.10 + -- 2.26.2