diff options
| author | travis%sedsystems.ca <> | 2005-03-10 01:18:03 +0100 |
|---|---|---|
| committer | travis%sedsystems.ca <> | 2005-03-10 01:18:03 +0100 |
| commit | 672d71471ee2ddae6005a9259789899c41264823 (patch) | |
| tree | aac906cd42b2873fca45baeff2a1100b321b70ac /attachment.cgi | |
| parent | 82bc36a9ccc7a9a0808c2d4c0e8b7d44eb1e63dd (diff) | |
| download | bugzilla-672d71471ee2ddae6005a9259789899c41264823.tar.gz bugzilla-672d71471ee2ddae6005a9259789899c41264823.tar.xz | |
Bug 282574 : use the new "auth_failure" error message for all authentication failures
Patch by Frederic Buclin <LpSolit@gmail.com> r=travis, wurblzap a=myk
Diffstat (limited to 'attachment.cgi')
| -rwxr-xr-x | attachment.cgi | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/attachment.cgi b/attachment.cgi index 64e30f64a..5e4c520f5 100755 --- a/attachment.cgi +++ b/attachment.cgi @@ -204,9 +204,10 @@ sub validateID # Make sure the user is authorized to access this attachment's bug. ($bugid, my $isprivate) = FetchSQLData(); ValidateBugID($bugid); - if (($isprivate > 0 ) && Param("insidergroup") && - !(UserInGroup(Param("insidergroup")))) { - ThrowUserError("attachment_access_denied"); + if ($isprivate && Param("insidergroup")) { + UserInGroup(Param("insidergroup")) + || ThrowUserError("auth_failure", {action => "access", + object => "attachment"}); } # XXX shim code, kill $::FORM |