File 0003-fast-import-tighten-parsing-of-boolean-com... of Package git.9802

Overview Repositories Revisions Requests Users Attributes Meta

File 0003-fast-import-tighten-parsing-of-boolean-command-line-.patch of Package git.9802

From 83d63e196e67e77482876a8c91e1999f4c748c91 Mon Sep 17 00:00:00 2001
From: Jeff King <peff@peff.net>
Date: Thu, 29 Aug 2019 11:25:45 -0400
Subject: [PATCH 03/28] fast-import: tighten parsing of boolean command line
 options

We parse options like "--max-pack-size=" using skip_prefix(), which
makes sense to get at the bytes after the "=". However, we also parse
"--quiet" and "--stats" with skip_prefix(), which allows things like
"--quiet-nonsense" to behave like "--quiet".

This was a mistaken conversion in 0f6927c229 (fast-import: put option
parsing code in separate functions, 2009-12-04). Let's tighten this to
an exact match, which was the original intent.

Signed-off-by: Jeff King <peff@peff.net>
---
 fast-import.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fast-import.c b/fast-import.c
index b70ac025e0..f5a82470bd 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -3313,9 +3313,9 @@ static int parse_one_option(const char *option)
 		option_active_branches(option);
 	} else if (skip_prefix(option, "export-pack-edges=", &option)) {
 		option_export_pack_edges(option);
-	} else if (starts_with(option, "quiet")) {
+	} else if (!strcmp(option, "quiet")) {
 		show_stats = 0;
-	} else if (starts_with(option, "stats")) {
+	} else if (!strcmp(option, "stats")) {
 		show_stats = 1;
 	} else {
 		return 0;
-- 
2.24.0

Places

File 0003-fast-import-tighten-parsing-of-boolean-command-line-.patch of Package git.9802

Places