diff options
author | Frédéric Buclin <LpSolit@gmail.com> | 2012-11-12 18:46:48 +0100 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-11-12 18:46:48 +0100 |
commit | 2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461 (patch) | |
tree | 0267c67f18e8099384bc7ae15f58a4459ed71f63 /page.cgi | |
parent | 34a9d58c4ab24390a21595edc08442651cfbe312 (diff) | |
download | bugzilla-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar.gz bugzilla-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar.xz |
Bug 676844: Replace ThrowCodeError() by ThrowUserError() when the error is not triggered by the code itself
r=glob a=LpSolit
Diffstat (limited to 'page.cgi')
-rwxr-xr-x | page.cgi | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -54,13 +54,13 @@ if ($id) { # Be careful not to allow directory traversal. if ($id =~ /\.\./) { # two dots in a row is bad - ThrowCodeError("bad_page_cgi_id", { "page_id" => $id }); + ThrowUserError("bad_page_cgi_id", { "page_id" => $id }); } # Split into name and ctype. $id =~ /^([\w\-\/\.]+)\.(\w+)$/; if (!$2) { # if this regexp fails to match completely, something bad came in - ThrowCodeError("bad_page_cgi_id", { "page_id" => $id }); + ThrowUserError("bad_page_cgi_id", { "page_id" => $id }); } my %vars = ( |