diff options
author | Hugo Seabrook <hugo.seabrook@gmail.com> | 2013-02-16 23:21:49 +0100 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2013-02-16 23:21:49 +0100 |
commit | 91a5a96b0b9890808fe888b60ba552cb49d874a8 (patch) | |
tree | bd33b67b550f6113920ad7a338bb00dd4cf9929d /extensions/MoreBugUrl/lib/PHP.pm | |
parent | f45ef9337770947f5bb36829c0427e14775ffe54 (diff) | |
download | bugzilla-91a5a96b0b9890808fe888b60ba552cb49d874a8.tar.gz bugzilla-91a5a96b0b9890808fe888b60ba552cb49d874a8.tar.xz |
Bug 825758: Most Bugzilla::BugUrl::* modules incorrectly validate the domain name
r/a=LpSolit
Diffstat (limited to 'extensions/MoreBugUrl/lib/PHP.pm')
-rw-r--r-- | extensions/MoreBugUrl/lib/PHP.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extensions/MoreBugUrl/lib/PHP.pm b/extensions/MoreBugUrl/lib/PHP.pm index 1521cae1a..ea090e865 100644 --- a/extensions/MoreBugUrl/lib/PHP.pm +++ b/extensions/MoreBugUrl/lib/PHP.pm @@ -20,8 +20,8 @@ sub should_handle { # PHP Bug URLs have only one form: # https://bugs.php.net/bug.php?id=1234 - return ($uri->authority =~ /^bugs.php.net$/i - and $uri->path =~ m|/bug.php$| + return (lc($uri->authority) eq 'bugs.php.net' + and $uri->path =~ m|/bug\.php$| and $uri->query_param('id') =~ /^\d+$/) ? 1 : 0; } |