From e957e60d5c599e8655d87cb7210392928566ac11 Mon Sep 17 00:00:00 2001 From: Arjen Stolk Date: Wed, 28 Dec 2016 11:05:50 +0100 Subject: [PATCH] mailcap: prevent segfault on test entry with %s When a mailcap entry with a testcommand containing %s is encountered, it is ignored. The entry was freed, but not set to NULL, leading to a segfault later on. --- src/mime/backend/mailcap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mime/backend/mailcap.c b/src/mime/backend/mailcap.c index 5abe9416..4e7a9224 100644 --- a/src/mime/backend/mailcap.c +++ b/src/mime/backend/mailcap.c @@ -301,6 +301,7 @@ parse_optional_fields(struct mailcap_entry *entry, unsigned char *line) for (field = entry->testcommand; *field; field++) if (*field == '%' && *(field+1) == 's') { mem_free(entry->testcommand); + entry->testcommand = NULL; return 0; }