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

rmudgett reviewboard at asterisk.org
Fri Sep 6 17:36:10 CDT 2013


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

Review request for Asterisk Developers.


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/20130906/017868fb/attachment-0001.htm>


More information about the asterisk-dev mailing list