aboutsummaryrefslogtreecommitdiffstats
path: root/quips.cgi
diff options
context:
space:
mode:
authorFrédéric Buclin <LpSolit@gmail.com>2012-11-12 18:46:48 +0100
committerFrédéric Buclin <LpSolit@gmail.com>2012-11-12 18:46:48 +0100
commit2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461 (patch)
tree0267c67f18e8099384bc7ae15f58a4459ed71f63 /quips.cgi
parent34a9d58c4ab24390a21595edc08442651cfbe312 (diff)
downloadbugs-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar
bugs-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar.gz
bugs-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar.bz2
bugs-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.tar.xz
bugs-2a6f7d4663dfc5cb00c1bb8b8d8139869a22d461.zip
Bug 676844: Replace ThrowCodeError() by ThrowUserError() when the error is not triggered by the code itself
r=glob a=LpSolit
Diffstat (limited to 'quips.cgi')
-rwxr-xr-xquips.cgi3
1 files changed, 1 insertions, 2 deletions
diff --git a/quips.cgi b/quips.cgi
index 4ed40a547..bbe53c86e 100755
--- a/quips.cgi
+++ b/quips.cgi
@@ -123,8 +123,7 @@ if ($action eq "delete") {
action => "delete",
object => "quips"});
my $quipid = $cgi->param("quipid");
- ThrowCodeError("need_quipid") unless $quipid =~ /(\d+)/;
- $quipid = $1;
+ detaint_natural($quipid) || ThrowUserError("need_quipid");
check_hash_token($token, ['quips', $quipid]);
($vars->{'deleted_quip'}) = $dbh->selectrow_array(