diff options
| author |  Olivier Marin <dkr@freesurf.fr> | 2008-07-21 20:30:36 +0200 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2008-07-21 21:20:04 -0700 |
| commit | d5d745f90b929310f3893bb8b3766cc1745140c8 (patch) | |
| tree | de67ef03b447f77d5484f45b2288b73dede8383b /t | |
| parent | mailinfo: better parse email adresses containg parentheses (diff) | |
| download | tgif-d5d745f90b929310f3893bb8b3766cc1745140c8.tar.xz | |
parse-options: fix segmentation fault when a required value is missing
p->argc represent the number of arguments that have not been parsed yet,
_including_ the one we are currently parsing. If it is not greater than
one then there is no more argument.
Signed-off-by: Olivier Marin <dkr@freesurf.fr>
Acked-by: Pierre Habouzit <madcoder@debian.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t0040-parse-options.sh | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/t/t0040-parse-options.sh b/t/t0040-parse-options.sh index 6309aed451..03dbe00102 100755 --- a/t/t0040-parse-options.sh +++ b/t/t0040-parse-options.sh @@ -78,6 +78,13 @@ test_expect_success 'long options' ' test_cmp expect output ' +test_expect_success 'missing required value' ' + test-parse-options -s; + test $? = 129 && + test-parse-options --string; + test $? = 129 +' + cat > expect << EOF boolean: 1 integer: 13 |