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

[Russell Bryant]

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



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

    Since these are set to -1 after close, shouldn't these checks change?


- Russell


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
> 
>