[asterisk-dev] [Code Review] 4548: Restrict ast_log_safe use from utils.h to the core only.

Corey Farrell reviewboard at asterisk.org
Mon Mar 30 06:40:57 CDT 2015


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

(Updated March 30, 2015, 6:40 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers, rmudgett and Scott Griepentrog.


Changes
-------

Committed in revision 433772


Repository: Asterisk


Description
-------

When modules are distributed as binaries, they are sometimes built against the latest Asterisk in a branch, but need to be able to run against any version within that branch.  The recent addition of ast_log_safe can pose a problem in this situation.  The new function is used from utils.h, but the symbol is not available in older versions.  The solution to this is to have modules just use ast_log for OOM errors.  The core will still use ast_log_safe for OOM errors, so this will protect against infinite logger recursion.


Diffs
-----

  /branches/11/main/Makefile 433658 
  /branches/11/include/asterisk/utils.h 433658 

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


Testing
-------

make cdr main utils

Check object files with 'nm -g' to verify that main/*.o still used ast_log_safe.  Verified that cdr/*.o and utils/*.o did not use ast_log_safe.


Thanks,

Corey Farrell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150330/7303e42e/attachment.html>


More information about the asterisk-dev mailing list