From 493fdae0464282fbac99f60d94bfaabf5559c9ff Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 22 Jun 2024 04:36:48 +0000 Subject: [PATCH] object-file: fix leak on conversion failure I'm not sure exactly how to trigger the leak, but it seems fairly obvious that the `content' buffer should be freed even if convert_object_file() fails. Noticed while working in this area on unrelated things. Signed-off-by: Eric Wong Acked-by: Derrick Stolee Signed-off-by: Junio C Hamano --- object-file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/object-file.c b/object-file.c index 610b1f465c..dd441627d6 100644 --- a/object-file.c +++ b/object-file.c @@ -1711,9 +1711,9 @@ static int oid_object_info_convert(struct repository *r, ret = convert_object_file(&outbuf, the_hash_algo, input_algo, content, size, type, !do_die); + free(content); if (ret == -1) return -1; - free(content); size = outbuf.len; content = strbuf_detach(&outbuf, NULL); }