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

2014-01-28 Thread David Sharp
On Tue, Jan 28, 2014 at 2:01 PM, Johannes Sixt wrote: > Am 28.01.2014 22:21, schrieb David Sharp: >> @@ -738,9 +740,12 @@ int cmd_rev_parse(int argc, const char **argv, const >> char *prefix) >> continue; >> }

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

2014-01-28 Thread David Sharp
On Tue, Jan 28, 2014 at 1:43 PM, Junio C Hamano wrote: > David Sharp writes: > >> 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. > > When I sent

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; >> } >>

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

2014-01-28 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 | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff

[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