From: Krzesimir Nowak Date: Wed, 11 Dec 2013 11:54:41 +0000 (+0100) Subject: gitweb: Move check-ref-format code into separate function X-Git-Tag: v1.9-rc0~57^2~3 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/c0bc2265ef74331c0314be4a3f148e784800c3de?ds=inline;hp=--cc gitweb: Move check-ref-format code into separate function This check will be used in more than one place later. Signed-off-by: Krzesimir Nowak Signed-off-by: Junio C Hamano --- c0bc2265ef74331c0314be4a3f148e784800c3de diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl index 68c77f6f8f..46bd6ac8c3 100755 --- a/gitweb/gitweb.perl +++ b/gitweb/gitweb.perl @@ -1452,6 +1452,16 @@ sub validate_pathname { return $input; } +sub is_valid_ref_format { + my $input = shift || return undef; + + # restrictions on ref name according to git-check-ref-format + if ($input =~ m!(/\.|\.\.|[\000-\040\177 ~^:?*\[]|/$)!) { + return undef; + } + return $input; +} + sub validate_refname { my $input = shift || return undef; @@ -1462,10 +1472,9 @@ sub validate_refname { # it must be correct pathname $input = validate_pathname($input) or return undef; - # restrictions on ref name according to git-check-ref-format - if ($input =~ m!(/\.|\.\.|[\000-\040\177 ~^:?*\[]|/$)!) { - return undef; - } + # check git-check-ref-format restrictions + is_valid_ref_format($input) + or return undef; return $input; }