[asterisk-dev] [Code Review] 4125: app_queue: fix a couple leaks to struct call_queue in set_member_value

[Corey Farrell]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4125/
-----------------------------------------------------------

(Updated Oct. 30, 2014, 6:53 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 426805


Bugs: ASTERISK-24466
    https://issues.asterisk.org/jira/browse/ASTERISK-24466


Repository: Asterisk


Description
-------

set_member_value has a couple leaks to references in the variable q found through testsuite tests/queues/set_penalty.

This change also removes the REF_DEBUG_ONLY_QUEUES compiler declaration, this is no longer possible with the updated REF_DEBUG code.


Diffs
-----

  /branches/11/apps/app_queue.c 426569 

Diff: https://reviewboard.asterisk.org/r/4125/diff/


Testing
-------

All tests/queues/set_penalty no longer leaks any references (verifies first added queue_unref).

I'm unsure if the second added queue_unref has been tested, but seems like it is needed.


Thanks,

Corey Farrell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141030/70290810/attachment-0001.html>