Review Board 1.7.16


app_confbridge: Repeatedly starting and stopping recording ref leaks the recording channel. (v11 version)

Review Request #4368 - Created Jan. 23, 2015 and submitted

rmudgett
11
ASTERISK-24719
Reviewers
asterisk-dev
Asterisk
Starting and stopping conference recording more than once causes the
recording channels to be leaked.  For v13 the channels also show up in the
CLI "core show channels" output.

* Reworked and simplified the recording channel code to use
ast_bridge_impart() instead of managing the recording thread in the
ConfBridge code.  The recording channel's ref handling easily falls into
place and other off nominal code paths get handled better as a result.

v13 version of patch: https://reviewboard.asterisk.org/r/4369/
With the test:
The confbridge testsuite tests still pass.
Manual testing no longer has the recording channel ref leak.
Review request changed
Updated (Jan. 27, 2015, 12:24 p.m.)
  • changed from pending to submitted
Committed in revision 431178

https://reviewboard.asterisk.org/ runs on a server provided by Digium, Inc. and uses bandwidth donated to the open source Asterisk community by API Digital Communications in Huntsville, AL USA.
Please report problems with this site to asteriskteam@digium.com.