diff options
author | Jani Nikula <jani@nikula.org> | 2017-10-16 18:40:44 +0300 |
---|---|---|
committer | David Bremner <david@tethera.net> | 2017-11-05 15:41:13 -0400 |
commit | cd3f5e1a934b9952ca718e0c5bc1471b3884020c (patch) | |
tree | 88b27329063cf3cc8dfe872b4263bdd061faf039 | |
parent | 949c27144e0b9294267511993a109c29d319a23d (diff) |
cli/crypto: fix segfault on failed gmime2 crypto context creation
Commit 1fdc08d0ffab ("cli/crypto: treat failure to create a crypto
context as fatal.") started treating crypto context creation failures
"as fatal", returning NULL from _mime_node_create().
Unfortunately, we do not have NULL checks for _mime_node_create()
failures. The only caller, mime_node_child(), could check and return
NULL (as it's documented to do on errors) but none of the several call
sites have NULL checks either. And none of them really have a trivial
but feasible and graceful way of recovery.
So while the right thing to do would be to handle NULL returns
properly all over the place, and we have other scenarios that do
return NULL from above mentioned functions, the crypto context
creation failure is something that does seem to show up regularly in
some scenarios, revert back to the functionality before commit
1fdc08d0ffab as an interim fix.
-rw-r--r-- | mime-node.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mime-node.c b/mime-node.c index bb0870d0..16091735 100644 --- a/mime-node.c +++ b/mime-node.c @@ -247,7 +247,7 @@ _mime_node_create (mime_node_t *parent, GMimeObject *part) const char *protocol = g_mime_content_type_get_parameter (content_type, "protocol"); cryptoctx = notmuch_crypto_get_context (node->ctx->crypto, protocol); if (!cryptoctx) - return NULL; + return node; } #endif |