threads / patch / 62663

patchbundle-url: always close child_out before leaving the function

Subject: [PATCH] bundle-url: always close child_out before leaving the function

## tl;dr

One message between Dec 18, 2024 and Dec 18, 2024. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

AreaZR via GitGitGadget· Dec 18, 2024, 03:32 UTC · lore
From: Seija Kijin <doremylover123@gmail.com>
Otherwise, child_out may always be left open
Signed-off-by: Seija Kijin <doremylover123@gmail.com>
---
    bundle-url: always close child_out before leaving the function
    
    Otherwise, child_out may always be left open
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1398%2FAreaZR%2Fbug-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1398/AreaZR/bug-v1
Pull-Request: https://github.com/git/git/pull/1398
 bundle-uri.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Show changes to bundle-uri.c +2 −2
diff --git a/bundle-uri.c b/bundle-uri.c
index cdf9e4f9e1d..23dfc1999b5 100644
--- a/bundle-uri.c
+++ b/bundle-uri.c
@@ -337,10 +337,10 @@ static int download_https_uri_to_file(const char *file, const char *uri)
 cleanup:
 	if (child_in)
 		fclose(child_in);
-	if (finish_command(&cp))
-		return 1;
 	if (child_out)
 		fclose(child_out);
+	if (finish_command(&cp))
+		return 1;
 	return result;
 }
 

base-commit: d882f382b3d939d90cfa58d17b17802338f05d66
-- 
gitgitgadget

← back to recent threads