DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=37852>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=37852





------- Additional Comments From [EMAIL PROTECTED]  2005-12-09 15:20 -------
(In reply to comment #0)

It's better to test for AllRoles into top-level 'if'. (do not skip checks for 
null principal)

Should be:

Class org.apache.catalina.realm.RealmBase, lines from 735:

if (constraint.getAllRoles() && !denyfromall)
  status = true;
if (log.isDebugEnabled())
  log.debug("  Checking roles " + principal);
if (roles.length == 0 && !constraint.getAllRoles()) {

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to