]> granicus.if.org Git - neomutt/commit
Actually fix gpgme segfault in create_recipient_set().
authorKevin McCarthy <kevin@8t8.us>
Mon, 17 Oct 2016 18:22:13 +0000 (11:22 -0700)
committerRichard Russon <rich@flatcap.org>
Fri, 28 Oct 2016 10:12:20 +0000 (11:12 +0100)
commitd94b68aa6932bf79f8b1a2ad4eff018ba002b69f
tree910561a4539aba89f12f99d8b9dc4971f09e02ab
parent0640ab75bf69424486e2b61ae7ac91f703be00a3
Actually fix gpgme segfault in create_recipient_set().

Changeset 6e44bfa16096 did not fix the segv.  (Sorry, I made the fix
based off a report on IRC but didn't trigger the segv myself: it was
caused by an out-of-tree patch).

The actual problem was that the rset was only resized on a successful
gpgme_get_key().  However, on error, the array still needs to be
NULL-terminated before calling free_recipient_set().

Move the resize so it always takes place.  This obviates the need for
the NULL check added in 6e44bfa16096.
crypt-gpgme.c