[asterisk-dev] [Code Review] Logger does not warn of failure to open logging channels

opticron reviewboard at asterisk.org
Mon Aug 8 16:10:45 CDT 2011


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

(Updated Aug. 8, 2011, 4:10 p.m.)


Review request for Asterisk Developers.


Changes
-------

Change message from warning to error.


Summary
-------

Currently, logger only prints an error message to stderr when it fails to open a logger channel where many users will not see it because the logger lock is held.  The alternative provided by this patch is to log the error to all attached consoles in the hopes that it will be easier to see.  Additionally, this patch prevents the failed logger channel from being added to the list where it would silently fail on each call to the Asterisk logger.


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


Diffs (updated)
-----

  trunk/main/logger.c 331040 

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


Testing
-------

Verified that the stderr log was the only thing printed for failed logger channels.  Verified that the patch provides warning to indirect consoles (not using the -c option).


Thanks,

opticron

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


More information about the asterisk-dev mailing list