]> git.ipfire.org Git - thirdparty/git.git/commit - config.c
config: fix a leak in git_config_copy_or_rename_section_in_file
authorRubén Justo <rjusto@gmail.com>
Sun, 11 Jun 2023 18:49:28 +0000 (20:49 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 12 Jun 2023 22:04:16 +0000 (15:04 -0700)
commit003c1f1171f12678fe7994b3e6b3f6b2f2b88de3
treefd335dc8d73d0f86e025856bb4e1a9aa421a5949
parentfe86abd7511a9a6862d5706c6fa1d9b57a63ba09
config: fix a leak in git_config_copy_or_rename_section_in_file

In 52d59cc645 (branch: add a --copy (-c) option to go with --move (-m),
2017-06-18) a new strbuf variable was introduced, but not released.

Thus, when copying a branch that has any configuration, we have a
leak.

   $ git branch foo
   $ git config branch.foo.some-key some_value
   $ git branch -c foo bar

   Direct leak of 65 byte(s) in 1 object(s) allocated from:
       ... in xrealloc wrapper.c
       ... in strbuf_grow strbuf.c
       ... in strbuf_vaddf strbuf.c
       ... in strbuf_addf strbuf.c
       ... in store_create_section config.c
       ... in git_config_copy_or_rename_section_in_file config.c
       ... in git_config_copy_section_in_file config.c
       ... in git_config_copy_section config.c
       ... in copy_or_rename_branch builtin/branch.c
       ... in cmd_branch builtin/branch.c
       ... in run_builtin git.c

Let's fix that leak.

Signed-off-by: Rubén Justo <rjusto@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
config.c