[PATCH v2 3/4] add-patch: plug a leak handling the '/' command

From: Rubén Justo <rjusto@gmail.com>
Date: Apr 22, 2024, 22:54 UTC
Message-ID: <f35a22a9-bbb1-491c-9f1f-e8c77f18fb9f@gmail.com>
In-Reply-To: <69f86760-96ff-4c62-9649-4627652b7c19@gmail.com>

Plug a leak we have since d6cf873340 (built-in add -p: implement the '/' ("search regex") command, 2019-12-13).

This leak can be triggered with:

    $ printf "A\n\nB\n" >file
    $ git add file && git commit -m file
    $ printf "AA\n\nBB\n" >file
    $ printf "s\n/ .\n" >lines
    $ git add -p <lines

Signed-off-by: Rubén Justo <rjusto@gmail.com>
---
 add-patch.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/add-patch.c b/add-patch.c
index a06dd18985..0997d4af73 100644
--- a/add-patch.c
+++ b/add-patch.c
@@ -1646,6 +1646,7 @@ static int patch_update_file(struct add_p_state *s,
 				err(s, _("No hunk matches the given pattern"));
 				break;
 			}
+			regfree(&regex);
 			hunk_index = i;
 		} else if (s->answer.buf[0] == 's') {
 			size_t splittable_into = hunk->splittable_into;

-- 
2.45.0.rc0.4.gfff7e48949