Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-17 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4160/ --- (Updated Nov. 17, 2014, 9:56 a.m.) Status -- This change has been mar

Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-13 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4160/#review13744 --- Ship it! Ship It! - Mark Michelson On Nov. 12, 2014, 8:28 p

Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-12 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4160/ --- (Updated Nov. 12, 2014, 3:28 p.m.) Review request for Asterisk Developers.

Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-11 Thread Mark Michelson
> On Nov. 11, 2014, 6:43 p.m., Mark Michelson wrote: > > If I understand the purpose of p->refer correctly, it's supposed to be > > details relating to a specific REFER (or REFER-esque in some cases) > > transaction. I think that any in-dialog places where p->refer may be > > allocated, the pr

Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-11 Thread Corey Farrell
> On Nov. 11, 2014, 1:43 p.m., Mark Michelson wrote: > > If I understand the purpose of p->refer correctly, it's supposed to be > > details relating to a specific REFER (or REFER-esque in some cases) > > transaction. I think that any in-dialog places where p->refer may be > > allocated, the pr

Re: [asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-11 Thread Mark Michelson
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4160/#review13722 --- If I understand the purpose of p->refer correctly, it's suppose

[asterisk-dev] [Code Review] 4160: chan_sip: Fix theoretical leak of p->refer

2014-11-09 Thread Corey Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4160/ --- Review request for Asterisk Developers. Bugs: ASTERISK-15242 https://i