diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2017-06-16 02:15:46 +0300 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-06-16 22:44:03 +0300 |
| commit | 6a83d902073803c4141e02c53decf8c03e35da27 (patch) | |
| tree | 209cf1639a325875240fce3e1f1cabbdb25878df /mailinfo.c | |
| parent | cf9f49ea4817b36e93e64909a918a912cc32c28b (diff) | |
coccinelle: make use of the "type" FREE_AND_NULL() rule
Apply the result of the just-added coccinelle rule. This manually
excludes a few occurrences, mostly things that resulted in many
FREE_AND_NULL() on one line, that'll be manually fixed in a subsequent
change.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'mailinfo.c')
| -rw-r--r-- | mailinfo.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mailinfo.c b/mailinfo.c index f92cb9f729..44f829edb1 100644 --- a/mailinfo.c +++ b/mailinfo.c @@ -919,8 +919,7 @@ again: /* we hit an end boundary */ /* pop the current boundary off the stack */ strbuf_release(*(mi->content_top)); - free(*(mi->content_top)); - *(mi->content_top) = NULL; + FREE_AND_NULL(*(mi->content_top)); /* technically won't happen as is_multipart_boundary() will fail first. But just in case.. |