[asterisk-dev] [Code Review] fix for double close of file descriptors raised in issue 16305

thedavidfactor at gmail.com thedavidfactor at gmail.com
Tue Dec 1 13:17:17 CST 2009


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



/trunk/apps/app_externalivr.c
<https://reviewboard.asterisk.org/r/436/#comment2947>

    is this an ok way of doing this? I'm checking eivr_errors_fd because it could be null, then checking to see if the ready_fd is eivr_errors_fd.


- thedavidfactor


On 2009-12-01 13:14:17, thedavidfactor wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/436/
> -----------------------------------------------------------
> 
> (Updated 2009-12-01 13:14:17)
> 
> 
> Review request for Asterisk Developers and Chris Tooley.
> 
> 
> Summary
> -------
> 
> a reported raised an issue where EIVR was closing FDs used by AGI when under heavy load. This modification was designed to prevent EIVR from closing an FD if the FD had already been closed.
> 
> 
> This addresses bug 16305.
>     https://issues.asterisk.org/view.php?id=16305
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_externalivr.c 230584 
> 
> Diff: https://reviewboard.asterisk.org/r/436/diff
> 
> 
> Testing
> -------
> 
> reporter attached patch that had been tested to fix issue, I have tested this code and it doesn't seem to break any existing functionality
> 
> 
> Thanks,
> 
> thedavidfactor
> 
>




More information about the asterisk-dev mailing list