[asterisk-dev] [Code Review] 2832: Fix incorrect usages of ast_realloc().

[svnbot]

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

(Updated Sept. 10, 2013, 12:54 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 398757


Repository: Asterisk


Description
-------

There are several locations in the code base where this is done:
buf = ast_realloc(buf, new_size);

This is going to leak the original buf contents if the realloc fails.

Several of these incorrect usages should apply to v1.8 and v11.


Diffs
-----

  /branches/12/funcs/func_dialgroup.c 398615 
  /branches/12/main/asterisk.c 398615 
  /branches/12/main/cli.c 398615 
  /branches/12/main/event.c 398615 
  /branches/12/main/heap.c 398615 
  /branches/12/main/indications.c 398615 
  /branches/12/main/xmldoc.c 398615 
  /branches/12/res/res_musiconhold.c 398615 
  /branches/12/res/res_pjsip/pjsip_configuration.c 398615 

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


Testing
-------

Mostly compiled the code and code inspection.

* The CLI tab completion still works.
* MOH still plays.
* Online XML documentation still shows application syntax.


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130910/9e52364f/attachment.htm>