diff options
author | Jeff King <peff@peff.net> | 2022-10-10 20:42:38 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-10-10 22:16:56 -0700 |
commit | c4f94907902f18d80a78ee19983d0d02932154d7 (patch) | |
tree | 3326c34d8419294b1689fd83172e28af6b9e825d /fsmonitor.c | |
parent | 5aa9e3262e6ac99c1c06d7f0e0619b1ea94a9881 (diff) | |
download | git-c4f94907902f18d80a78ee19983d0d02932154d7.tar.gz |
fsmonitor: fix leak of warning message
The fsm_settings__get_incompatible_msg() function returns an allocated
string. So we can't pass its result directly to warning(); we must hold
on to the pointer and free it to avoid a leak.
The leak here is small and fixed size, but Coverity complained, and
presumably SANITIZE=leaks would eventually.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'fsmonitor.c')
-rw-r--r-- | fsmonitor.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fsmonitor.c b/fsmonitor.c index 540736b39f..08af00c738 100644 --- a/fsmonitor.c +++ b/fsmonitor.c @@ -309,8 +309,10 @@ void refresh_fsmonitor(struct index_state *istate) enum fsmonitor_reason reason = fsm_settings__get_reason(r); if (!warn_once && reason > FSMONITOR_REASON_OK) { + char *msg = fsm_settings__get_incompatible_msg(r, reason); warn_once = 1; - warning("%s", fsm_settings__get_incompatible_msg(r, reason)); + warning("%s", msg); + free(msg); } if (fsm_mode <= FSMONITOR_MODE_DISABLED || |