aboutsummaryrefslogtreecommitdiff
path: root/alias.c
diff options
context:
space:
mode:
authorTaylor Blau <me@ttaylorr.com>2022-10-06 17:43:37 -0400
committerTaylor Blau <me@ttaylorr.com>2022-10-06 17:43:37 -0400
commitac8a1db86793292dfc7055eeef2fe90459e771aa (patch)
treeb2bfe2c51a837c8efb73b7b52c749d9127e6b646 /alias.c
parentef374dd9b8eecaf84458677865732b89948c19be (diff)
parentbe85cfc4dbef9ffa177770f8607fd4a85445565e (diff)
downloadgit-ac8a1db86793292dfc7055eeef2fe90459e771aa.tar.xz
Sync with 2.34.5
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Diffstat (limited to 'alias.c')
-rw-r--r--alias.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/alias.c b/alias.c
index c471538020..00abde0817 100644
--- a/alias.c
+++ b/alias.c
@@ -46,14 +46,16 @@ void list_aliases(struct string_list *list)
#define SPLIT_CMDLINE_BAD_ENDING 1
#define SPLIT_CMDLINE_UNCLOSED_QUOTE 2
+#define SPLIT_CMDLINE_ARGC_OVERFLOW 3
static const char *split_cmdline_errors[] = {
N_("cmdline ends with \\"),
- N_("unclosed quote")
+ N_("unclosed quote"),
+ N_("too many arguments"),
};
int split_cmdline(char *cmdline, const char ***argv)
{
- int src, dst, count = 0, size = 16;
+ size_t src, dst, count = 0, size = 16;
char quoted = 0;
ALLOC_ARRAY(*argv, size);
@@ -96,6 +98,11 @@ int split_cmdline(char *cmdline, const char ***argv)
return -SPLIT_CMDLINE_UNCLOSED_QUOTE;
}
+ if (count >= INT_MAX) {
+ FREE_AND_NULL(*argv);
+ return -SPLIT_CMDLINE_ARGC_OVERFLOW;
+ }
+
ALLOC_GROW(*argv, count + 1, size);
(*argv)[count] = NULL;