[asterisk-dev] [Code Review] Fix for parking lot settings not being respected with unit test

Jeff Peeler jpeeler at digium.com
Wed Mar 10 14:16:12 CST 2010


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

(Updated 2010-03-10 14:16:12.750464)


Review request for Asterisk Developers.


Changes
-------

Ensure that test channel is hung up in failure cases.


Summary
-------

This verifies a change in parking to ensure the channel variable PARKINGLOT is respected. Kind of overkill for an obvious change, but many more parking scenarios can be added later.


This addresses bug 16592.
    https://issues.asterisk.org/view.php?id=16592


Diffs (updated)
-----

  /trunk/main/features.c 251632 

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


Testing
-------

Verified proposed fix works and that all created channels are properly destroyed. It seems though that I do actually have a ref count problem with the dynamic lot used for testing, which I haven't been able to track down yet. I went ahead and posted the review so the rest of the test can be examined.


Thanks,

Jeff




More information about the asterisk-dev mailing list