[asterisk-bugs] [Asterisk 0010817]: [patch] app_queue: interface_exists_global not set return value...

[noreply at bugs.digium.com]

The following issue has been RESOLVED. 
====================================================================== 
http://bugs.digium.com/view.php?id=10817 
====================================================================== 
Reported By:                snar
Assigned To:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   10817
Category:                   Applications/app_queue
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     resolved
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 83636 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             09-24-2007 10:16 CDT
Last Modified:              09-24-2007 13:46 CDT
====================================================================== 
Summary:                    [patch] app_queue: interface_exists_global not set
return value...
Description: 

Function interface_exists_global traverses all queues in order to find, 
is the given interface exists in any of the queues... 
return value, ret, initialized as ret=0 at line 876, then we gone into 
cycle, and, if we found matching interface 
                if ((mem = ao2_find(q->members, &tmpmem, OBJ_POINTER))) {
                        ao2_ref(mem, -1);
                        ao2_unlock(q);
                        queue_unref(q);
                        break;
                }                
we do not change ret before return... 
So, interface is never found by the function.
====================================================================== 

---------------------------------------------------------------------- 
 putnopvut - 09-24-07 13:46  
---------------------------------------------------------------------- 
Your note made me start doubting myself, so I set up some tests to be
certain that I did not oversimplify your patch. After running the tests, I
can say with confidence that everything is working as it should. I
appreciate your concern, though.

Closing... 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
09-24-07 13:46  putnopvut      Status                   feedback => resolved
09-24-07 13:46  putnopvut      Resolution               reopened => fixed   
09-24-07 13:46  putnopvut      Assigned To               => putnopvut       
09-24-07 13:46  putnopvut      Note Added: 0071039                          
======================================================================