]> git.ipfire.org Git - thirdparty/git.git/commit - grep.c
grep: report missing left operand of --and
authorRené Scharfe <l.s.r@web.de>
Wed, 30 Jun 2021 16:12:43 +0000 (18:12 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 30 Jun 2021 21:19:03 +0000 (14:19 -0700)
commitfe7fe62d8da0949d9b2bca34467b349bd294e91b
treefbac7dfc8393c1aae8245f860fc1ab102215530b
parent8b1a5f33d3ed427b0a6eaee595537805db6bc38c
grep: report missing left operand of --and

Git grep allows combining two patterns with --and.  It checks and
reports if the second pattern is missing when compiling the expression.
A missing first pattern, however, is only reported later at match time.
Thus no error is returned if no matching is done, e.g. because no file
matches the also given pathspec.

When that happens we get an expression tree with an GREP_NODE_AND node
and a NULL pointer to the missing left child.  free_pattern_expr()
tries to dereference it during the cleanup at the end, which results
in a segmentation fault.

Fix this by verifying the presence of the left operand at expression
compilation time.

Reported-by: Matthew Hughes <matthewhughes934@gmail.com>
Helped-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: René Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
grep.c
t/t7810-grep.sh