[asterisk-dev] [Code Review] 2743: quickfix against ast_log infinite recursion when out of memory

wdoekes reviewboard at asterisk.org
Mon Aug 5 12:11:21 CDT 2013


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

Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

Dirty quickfix to tackle ast_log() killing the asterisk when we're out of memory.

I don't like this fix at all. But I like the problem even less.

If anyone has a better idea, do come forward. (Aside from fixing the problem that caused me to run out of memory in the first place. But that's for another day.)


Diffs
-----

  /trunk/include/asterisk/utils.h 396181 
  /trunk/main/pbx.c 396181 
  /trunk/main/utils.c 396181 

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


Testing
-------

When MALLOC_DEBUG is disabled,

CLI> core eat memory

did what it's supposed to do: without the patch, asterisk crashed (stack full). With it, it didn't.


Thanks,

wdoekes

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


More information about the asterisk-dev mailing list