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

thedavidfactor at gmail.com thedavidfactor at gmail.com
Mon Nov 30 12:00:15 CST 2009


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



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

    If there's a better way to handle this, I'm open to suggestions, I thought that using the unassigned child_stderr to be the simplest solution


- thedavidfactor


On 2009-11-30 11:58:47, thedavidfactor wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/436/
> -----------------------------------------------------------
> 
> (Updated 2009-11-30 11:58:47)
> 
> 
> 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