diff options
| author | Jakub Narebski <jnareb@gmail.com> | 2012-02-28 19:41:47 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2012-02-28 11:45:31 -0800 |
| commit | 36612e4daf8b5b5eaf16315aa13c66925f878cd6 (patch) | |
| tree | 910d07215d5ce41fc0faf0656af6758a7248edaa /t/t9501-gitweb-standalone-http-status.sh | |
| parent | f174a2583c9f42315b60205890fa67a79a1f1669 (diff) | |
| download | git-36612e4daf8b5b5eaf16315aa13c66925f878cd6.tar.gz | |
gitweb: Handle invalid regexp in regexp searchjn/maint-gitweb-invalid-regexp
When using regexp search ('sr' parameter / $search_use_regexp variable
is true), check first that regexp is valid.
Without this patch we would get an error from Perl during search (if
searching is performed by gitweb), or highlighting matches substring
(if applicable), if user provided invalid regexp... which means broken
HTML, with error page (including HTTP headers) generated after gitweb
already produced some output.
Add test that illustrates such error: for example for regexp "*\.git"
we would get the following error:
Quantifier follows nothing in regex; marked by <-- HERE in m/* <-- HERE \.git/
at /var/www/cgi-bin/gitweb.cgi line 3084.
Reported-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t9501-gitweb-standalone-http-status.sh')
| -rwxr-xr-x | t/t9501-gitweb-standalone-http-status.sh | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/t/t9501-gitweb-standalone-http-status.sh b/t/t9501-gitweb-standalone-http-status.sh index 26102ee9b0..31076edc5b 100755 --- a/t/t9501-gitweb-standalone-http-status.sh +++ b/t/t9501-gitweb-standalone-http-status.sh @@ -134,4 +134,14 @@ our $maxload = undef; EOF +# ---------------------------------------------------------------------- +# invalid arguments + +test_expect_success 'invalid arguments: invalid regexp (in project search)' ' + gitweb_run "a=project_list;s=*\.git;sr=1" && + grep "Status: 400" gitweb.headers && + grep "400 - Invalid.*regexp" gitweb.body +' +test_debug 'cat gitweb.headers' + test_done |