[asterisk-dev] [Code Review] app_alarmreceiver new features and fixes

Pedro Kiefer reviewboard at asterisk.org
Wed Oct 17 11:18:00 CDT 2012


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



/trunk/apps/app_alarmreceiver.c
<https://reviewboard.asterisk.org/r/2088/#comment14072>

    write_event return 0 when successful, and we have to test first if elp is not null, before calling write_event.
    So this line should actually read:
    
    while ((elp != NULL) && (write_event(logfile, elp) == 0)) {


- Pedro


On Oct. 17, 2012, 8:39 a.m., KNK wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2088/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2012, 8:39 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Adds new formats to app_alarmreceiver, ALAW calls support and protection from broken panels and malicious callers
> 
> 
> This addresses bug ASTERISK-20289.
>     https://issues.asterisk.org/jira/browse/ASTERISK-20289
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_alarmreceiver.c 375136 
>   /trunk/configs/alarmreceiver.conf.sample 375136 
> 
> Diff: https://reviewboard.asterisk.org/r/2088/diff
> 
> 
> Testing
> -------
> 
> All new functionality was tested
> 
> 
> Thanks,
> 
> KNK
> 
>

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


More information about the asterisk-dev mailing list