[asterisk-dev] [Code Review] Fix possible memory leak with SIP realtime regs

[Terry Wilson]

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

(Updated Sept. 29, 2011, 5:30 p.m.)


Review request for Asterisk Developers and wdoekes.


Summary
-------

It was possible that if realtime sipregs are used, that the varregs variable returned by ast_load_realtime() would not be freed. This patch aims to correct that.


This addresses bug ASTERISK-17792.
    https://issues.asterisk.org/jira/browse/ASTERISK-17792


Diffs
-----

  /branches/1.8/channels/chan_sip.c 338223 

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


Testing
-------

It compiles. We return NULL in every case that ast_variables_destroy(varregs) is called, so there should be no chance of causing a problem with destroying the variable multiple times, etc.


Thanks,

Terry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110929/94534994/attachment-0001.htm>