[asterisk-dev] [Code Review] Chan_dahdi reload memory leak due to variables

Matthew Nicholson reviewboard at asterisk.org
Tue Apr 12 13:11:07 CDT 2011


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



branches/1.6.2/channels/chan_dahdi.c
<https://reviewboard.asterisk.org/r/1170/#comment6921>

    whitespace


- Matthew


On 2011-04-12 13:02:19, jrose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1170/
> -----------------------------------------------------------
> 
> (Updated 2011-04-12 13:02:19)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> reload chan_dahdi would cause variables to spill over from the previous config into the current config as well as add any variables in the new config (if left the same, full on duplication)
> This patch purges variables from pvts when they are reconfigured as well as from the configuration pvt once the configuration has been loaded.
> 
> 
> This addresses bug 17450.
>     https://issues.asterisk.org/view.php?id=17450
> 
> 
> Diffs
> -----
> 
>   branches/1.6.2/channels/chan_dahdi.c 313276 
> 
> Diff: https://reviewboard.asterisk.org/r/1170/diff
> 
> 
> Testing
> -------
> 
> !touch /etc/asterisk/chan_dahdi.conf
> reload chan_dahdi
> memory show summary / memory show allocations chan_dahdi.c
> 
> Performed with a variety of chan_dahdi.conf changes (and lack of changes)
> 
> 
> Thanks,
> 
> jrose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110412/801ce4ed/attachment.htm>


More information about the asterisk-dev mailing list