[rt-users] bug in Queue_Overlay.pm (3.0.2pre6)

Mark D. Nagel mnagel at willingminds.com
Wed Jun 4 12:26:31 EDT 2003


There was a typo in the ACL check for modifying queue watchers - here's 
a patch:

--- Queue_Overlay.pm.orig       Wed Jun  4 09:15:06 2003
+++ Queue_Overlay.pm    Wed Jun  4 09:15:13 2003
@@ -558,7 +558,7 @@
     # and the current user  doesn't have 'ModifyTicket'
     # bail
     else {
-        unless ( $self->CurrentUserHasRight('ModifyQueueWatcher') ) {
+        unless ( $self->CurrentUserHasRight('ModifyQueueWatchers') ) {
             return ( 0, $self->loc("Permission Denied") );
         }
     }

I think I've seen one or two other reports like this where the Right 
name has a typo. It might be useful to issue a pre-query before the 
'honkin-big SQL query' in Principal_Overlay::HasRight to check whether 
the given Right is in ACL.RightName and issue an alert in the logs 
indicating the nature of the internal error. It would probably be OK to 
run that query once at startup time to populate a hash and then check 
the hash in HasRight to improve performance.

Regards,
Mark

-- 
Mark D. Nagel, CCIE #3177 <mnagel at willingminds.com>
Principal Consultant, Willing Minds LLC
tel/fax: 949-623-9853, web: http://www.willingminds.com/





More information about the rt-users mailing list