Re: [asterisk-dev] [Code Review] 4262: bridge: channel ref leak on blond_nonfinal_enter

2014-12-19 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4262/ --- (Updated Dec. 19, 2014, 11:25 a.m.) Status -- This change has been

Re: [asterisk-dev] [Code Review] 4262: bridge: channel ref leak on blond_nonfinal_enter

2014-12-16 Thread Scott Griepentrog
On Dec. 12, 2014, 6:30 p.m., rmudgett wrote: /branches/12/main/bridge_basic.c, lines 2262-2264 https://reviewboard.asterisk.org/r/4262/diff/1/?file=69672#file69672line2262 Without looking further, the channel refs here seem to be mismatched. The recall_target is created and

Re: [asterisk-dev] [Code Review] 4262: bridge: channel ref leak on blond_nonfinal_enter

2014-12-16 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4262/ --- (Updated Dec. 16, 2014, 3:09 p.m.) Review request for Asterisk

Re: [asterisk-dev] [Code Review] 4262: bridge: channel ref leak on blond_nonfinal_enter

2014-12-16 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4262/#review13970 --- Ship it! Ship It! - Mark Michelson On Dec. 16, 2014, 9:09

Re: [asterisk-dev] [Code Review] 4262: bridge: channel ref leak on blond_nonfinal_enter

2014-12-12 Thread rmudgett
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4262/#review13959 --- /branches/12/main/bridge_basic.c