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

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

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

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

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 previous

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 previous

[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