[PATCH 09/20] util/crypto: add information about the payload part
Daniel Kahn Gillmor
dkg at fifthhorseman.net
Fri Jun 29 19:05:01 PDT 2018
On Sun 2018-06-24 22:15:46 -0300, David Bremner wrote:
> Daniel Kahn Gillmor <dkg at fifthhorseman.net> writes:
>
>> @@ -270,7 +274,7 @@ _notmuch_message_crypto_potential_payload (_notmuch_message_crypto_t *msg_crypto
>> if (parent && GMIME_IS_MULTIPART_ENCRYPTED (parent) && childnum == GMIME_MULTIPART_ENCRYPTED_VERSION) {
>> const char *enc_type = g_mime_object_get_content_type_parameter (parent, "protocol");
>> GMimeContentType *ct = g_mime_object_get_content_type (payload);
>> - if (ct) {
>> + if (ct && enc_type) {
>> const char *part_type = g_mime_content_type_get_mime_type (ct);
>> if (part_type && strcmp (part_type, enc_type) == 0)
>> return NOTMUCH_STATUS_SUCCESS;
>
> I couldn't really understand why enc_type was defined earlier than it
> was used here. Is this a bug fix?
whoops, you're right. I've moved this particular edit to an earlier
patch in the series. good catch, thanks!
--dkg
More information about the notmuch
mailing list