Re: [asterisk-dev] [Code Review] 4445: app_chanspy, channel: fix frame leaks

2015-02-26 Thread Kevin Harwell

---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4445/
---

(Updated Feb. 26, 2015, 11:06 a.m.)


Status
--

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
---

Committed in revision 432362


Bugs: ASTERISK-24828
https://issues.asterisk.org/jira/browse/ASTERISK-24828


Repository: Asterisk


Description
---

During testing it was noticed that Asterisk was leaking frames. After doing 
some digging a couple spots where a frame leak could occur were found. This 
patch plugs those leaks.


Diffs
-

  branches/11/main/channel.c 432173 
  branches/11/apps/app_chanspy.c 432173 

Diff: https://reviewboard.asterisk.org/r/4445/diff/


Testing
---

After applying the patch testing was done again to make sure the leaks no 
longer occurred.


Thanks,

Kevin Harwell

-- 
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Re: [asterisk-dev] [Code Review] 4445: app_chanspy, channel: fix frame leaks

2015-02-25 Thread rmudgett

---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4445/#review14548
---

Ship it!


Ship It!

- rmudgett


On Feb. 24, 2015, 5:05 p.m., Kevin Harwell wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviewboard.asterisk.org/r/4445/
 ---
 
 (Updated Feb. 24, 2015, 5:05 p.m.)
 
 
 Review request for Asterisk Developers.
 
 
 Bugs: ASTERISK-24828
 https://issues.asterisk.org/jira/browse/ASTERISK-24828
 
 
 Repository: Asterisk
 
 
 Description
 ---
 
 During testing it was noticed that Asterisk was leaking frames. After doing 
 some digging a couple spots where a frame leak could occur were found. This 
 patch plugs those leaks.
 
 
 Diffs
 -
 
   branches/11/main/channel.c 432173 
   branches/11/apps/app_chanspy.c 432173 
 
 Diff: https://reviewboard.asterisk.org/r/4445/diff/
 
 
 Testing
 ---
 
 After applying the patch testing was done again to make sure the leaks no 
 longer occurred.
 
 
 Thanks,
 
 Kevin Harwell
 


-- 
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev