diff options
author | lpsolit%gmail.com <> | 2005-10-28 19:33:18 +0200 |
---|---|---|
committer | lpsolit%gmail.com <> | 2005-10-28 19:33:18 +0200 |
commit | cf3aa532ea51a41b02c8ea73db254d01c03280ba (patch) | |
tree | 52f60354f60ea59baf6a244edea719b32ef50e17 /Bugzilla/Product.pm | |
parent | e942748a69920c86a7eaba517c9a74223da52edb (diff) | |
download | bugzilla-cf3aa532ea51a41b02c8ea73db254d01c03280ba.tar.gz bugzilla-cf3aa532ea51a41b02c8ea73db254d01c03280ba.tar.xz |
Bug 314088: Several Bugzilla::Foo->new crash when passing a string instead of a valid ID as a param - Patch by Frédéric Buclin <LpSolit@gmail.com> r=kiko a=justdave
Diffstat (limited to 'Bugzilla/Product.pm')
-rw-r--r-- | Bugzilla/Product.pm | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Bugzilla/Product.pm b/Bugzilla/Product.pm index 2bc9da52d..5405b1651 100644 --- a/Bugzilla/Product.pm +++ b/Bugzilla/Product.pm @@ -63,7 +63,10 @@ sub _init { my $id = $param unless (ref $param eq 'HASH'); my $product; - if (defined $id && detaint_natural($id)) { + if (defined $id) { + detaint_natural($id) + || ThrowCodeError('param_must_be_numeric', + {function => 'Bugzilla::Product::_init'}); $product = $dbh->selectrow_hashref(qq{ SELECT $columns FROM products |