diff options
author | dkl%redhat.com <> | 2009-01-26 21:40:22 +0100 |
---|---|---|
committer | dkl%redhat.com <> | 2009-01-26 21:40:22 +0100 |
commit | c49af480dcb59aadfa1edb76f246c68917a59765 (patch) | |
tree | 86ee9438a99ae209ea0fd8bbc5eddc2376b30b1c /Bugzilla/WebService/User.pm | |
parent | 2319f5f6660df8ed65c44dc29a5b2f42f9d81629 (diff) | |
download | bugzilla-c49af480dcb59aadfa1edb76f246c68917a59765.tar.gz bugzilla-c49af480dcb59aadfa1edb76f246c68917a59765.tar.xz |
Bug 473646 - WebService methods should check list parameters for scalars and convert before use
Patch by Dave Lawrence <dkl@redhat.com> - r/a=mkanat
Diffstat (limited to 'Bugzilla/WebService/User.pm')
-rwxr-xr-x | Bugzilla/WebService/User.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Bugzilla/WebService/User.pm b/Bugzilla/WebService/User.pm index 6283f55a1..790a9da7c 100755 --- a/Bugzilla/WebService/User.pm +++ b/Bugzilla/WebService/User.pm @@ -28,7 +28,7 @@ use Bugzilla::Error; use Bugzilla::User; use Bugzilla::Util qw(trim); use Bugzilla::Token; -use Bugzilla::WebService::Util qw(filter); +use Bugzilla::WebService::Util qw(filter validate); # Don't need auth to login use constant LOGIN_EXEMPT => { @@ -131,7 +131,7 @@ sub create { # $call = $rpc->call( 'User.get', { ids => [1,2,3], # names => ['testusera@redhat.com', 'testuserb@redhat.com'] }); sub get { - my ($self, $params) = @_; + my ($self, $params) = validate(@_, 'names', 'ids'); my @user_objects; @user_objects = map { Bugzilla::User->check($_) } @{ $params->{names} } |