Re: [asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2015-01-05 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4302/ --- (Updated Jan. 5, 2015, 4:49 p.m.) Status -- This change has been mark

Re: [asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2015-01-05 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4302/#review14073 --- Ship it! Ship It! - Mark Michelson On Jan. 2, 2015, 4:46 p.

Re: [asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2015-01-02 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4302/ --- (Updated Jan. 2, 2015, 10:46 a.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2015-01-02 Thread Scott Griepentrog
> On Dec. 31, 2014, 3:46 p.m., Matt Jordan wrote: > > /branches/13/main/bridge_basic.c, lines 2398-2410 > > > > > > I'm not sure we should be clearing the reference here. > > > > Clearly, we need to clear

Re: [asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2014-12-31 Thread Matt Jordan
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4302/#review14062 --- /branches/13/main/bridge_basic.c

[asterisk-dev] [Code Review] 4302: bridge: channel ref leak after failed blond transfer

2014-12-30 Thread Scott Griepentrog
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4302/ --- Review request for Asterisk Developers. Bugs: ASTERISK-24513 https://i