diff options
| author | Jeff King <peff@peff.net> | 2023-05-18 20:03:25 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2023-05-18 18:33:04 -0700 |
| commit | cfa120947e6337e7be2658f71a0132e337ee090a (patch) | |
| tree | 57aaeeece2975a292d793e6a6c04009ac753b585 | |
| parent | 20bd08aefb20168c6c227d2bfd1965761f9201ea (diff) | |
| download | git-cfa120947e6337e7be2658f71a0132e337ee090a.tar.xz | |
format-patch: free rev.message_id when exiting
We may allocate a message-id string via gen_message_id(), but we never
free it, causing a small leak. This can be demonstrated by running t9001
with a leak-checking build. The offending test is the one touched by
3ece9bf0f9 (send-email: clear the $message_id after validation,
2023-05-17), but the leak is much older than that. The test was simply
unlucky enough to trigger the leaking code path for the first time.
We can fix this by freeing the string at the end of the function. We can
also re-mark the test script as leak-free, effectively reverting
20bd08aefb (t9001: mark the script as no longer leak checker clean,
2023-05-17).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/log.c | 1 | ||||
| -rwxr-xr-x | t/t9001-send-email.sh | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/builtin/log.c b/builtin/log.c index 7d19578963..ab74760386 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -2415,6 +2415,7 @@ done: strbuf_release(&rdiff_title); strbuf_release(&sprefix); free(to_free); + free(rev.message_id); if (rev.ref_message_ids) string_list_clear(rev.ref_message_ids, 0); free(rev.ref_message_ids); diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index 2051103226..8d49eff91a 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @@ -4,7 +4,7 @@ test_description='git send-email' GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME -# no longer TEST_PASSES_SANITIZE_LEAK=true - format-patch --thread leaks +TEST_PASSES_SANITIZE_LEAK=true . ./test-lib.sh # May be altered later in the test |