[asterisk-dev] [Code Review] Keep bridge up if parking attempt fails

[Jeff Peeler]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/133/
-----------------------------------------------------------

Review request for Asterisk Developers.


Summary
-------

This corrects the unexpected behavior of having a bridged call drop just because of a failed parking attempt. Now, the bridge stays up if the parking attempt fails giving the user an opportunity to try again later or try another extension.

I pulled out the slot assignment functionality from park_call_full and put into park_space_reserve, enabling one to check and see if an attempt to enter the parking lot will succeed without doing so. The idea is to call park_space_reserve before doing any masquerades and bail out if the parking lot entry will fail.

The only thing I'm concerned about is if anywhere in park_call_full needs to be protected with the parkinglock.


This addresses bug 13494.
    http://bugs.digium.com/view.php?id=13494


Diffs
-----

  /branches/1.4/res/res_features.c 171433 

Diff: http://reviewboard.digium.com/r/133/diff


Testing
-------

Tested cases with both attended and blind transfer to the park extension as well as with one touch parking.


Thanks,

Jeff