Re: [PATCH] rev-parse: Check argc before using argv[i+1]

2014-01-28 Thread David Sharp
On Tue, Jan 28, 2014 at 11:12 AM, Junio C Hamano wrote: > David Sharp writes: > >> @@ -738,9 +740,11 @@ int cmd_rev_parse(int argc, const char **argv, const >> char *prefix) >> continue; >> } >> if (!strcmp(arg, "--resolve

Re: [PATCH] rev-parse: Check argc before using argv[i+1]

2014-01-28 Thread Junio C Hamano
David Sharp writes: > @@ -738,9 +740,11 @@ int cmd_rev_parse(int argc, const char **argv, const > char *prefix) > continue; > } > if (!strcmp(arg, "--resolve-git-dir")) { > - const char *gitdir

[PATCH] rev-parse: Check argc before using argv[i+1]

2014-01-27 Thread David Sharp
Without this patch, git-rev-parse --prefix, --default, or --resolve-git-dir, without a value argument, would result in a segfault. Instead, die() with a message. Signed-off-by: David Sharp --- builtin/rev-parse.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git