| Message ID | fa27aca12c46d2934baa594c05b7b3d7672e0be6.1665423686.git.me@ttaylorr.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | grep: tolerate NULL argument to free_grep_expr() | expand |
diff --git a/t/t4202-log.sh b/t/t4202-log.sh index cc15cb4ff6..e3ec5f5661 100755 --- a/t/t4202-log.sh +++ b/t/t4202-log.sh @@ -297,6 +297,12 @@ test_expect_success 'log --invert-grep --grep -i' ' fi ' +test_expect_failure 'log --invert-grep (no --grep)' ' + git log --pretty="tformat:%s" >expect && + git log --invert-grep --pretty="tformat:%s" >actual && + test_cmp expect actual +' + test_expect_success 'log --grep option parsing' ' echo second >expect && git log -1 --pretty="tformat:%s" --grep sec >actual &&
When `--invert-grep` is given without a pattern, `git log` behaves as normal. But as of f41fb662f5 (revisions API: have release_revisions() release "grep_filter", 2022-04-13), this doesn't quite work because we try to dereference the NULL `grep_expr` pointer in `free_pattern_expr()`, leading to a sgefault. The subsequent patch will explain the bug, provide a fix, and update this test to expect success. Signed-off-by: Taylor Blau <me@ttaylorr.com> --- t/t4202-log.sh | 6 ++++++ 1 file changed, 6 insertions(+)