<div dir="ltr"><div>Thanks Matt,</div><div><br></div><div>   I think I found what is going wrong , but Before that I want to confirm and Can you please verify what I am doing below is correct - </div><div><br></div><div>During Record using agi handle_recordfile and When there is a disconnect while in ast_waitstream , I can see the ast_closestream is initiated only channel.c during call cleanup </div>
<div>to close the stream opened for beep .</div><div><br></div><div>can I call the ast_closestream in agi handle_recordfile itself if I get return response as -1 to close the filestream opened for beep, I hope this shouldn't affect the flow </div>
<div><br></div><div>thanks,</div><div>bala</div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 21, 2014 at 10:55 PM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, May 21, 2014 at 7:33 PM, bala murugan <<a href="mailto:fightwithme@gmail.com">fightwithme@gmail.com</a>> wrote:<br>

><br>
><br>
><br>
> On Wed, May 21, 2014 at 8:23 PM, bala murugan <<a href="mailto:fightwithme@gmail.com">fightwithme@gmail.com</a>> wrote:<br>
>><br>
>> HI ,<br>
>><br>
>>     has any one noticed in res_agi.c handle_recordfile , if there is a<br>
>> call disconnect while beep is played back to caller the beep filestream is<br>
>> not getting closed and it leads to FileDescriptor Leak. This is still there<br>
>> in asterisk 12 . I have a Fix and Let me know I can submit the same .<br>
>><br>
>> thanks,<br>
><br>
<br>
</div></div>Nope, that one would be new as far as I'm aware of. I'd be interested<br>
in seeing that patch, as the play back of the beep sound file is<br>
handled by ast_streamfile, which is used rather extensively.<br>
<br>
If you'd like to submit a patch up stream to the project, please open<br>
an issue on the issue tracker [1], sign a license contributor<br>
agreement [2], and attach the patch in unified diff format to the<br>
issue. You can find more information on submitting a patch to Asterisk<br>
on the Asterisk wiki [3].<br>
<br>
[1] <a href="https://issues.asterisk.org" target="_blank">https://issues.asterisk.org</a><br>
<br>
[2] <a href="https://wiki.asterisk.org/wiki/display/AST/Digium+License+Agreement" target="_blank">https://wiki.asterisk.org/wiki/display/AST/Digium+License+Agreement</a><br>
<br>
[3] <a href="https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process" target="_blank">https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process</a><br>
<br>
Thanks!<br>
<br>
Matt<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Matthew Jordan<br>
Digium, Inc. | Engineering Manager<br>
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA<br>
Check us out at: <a href="http://digium.com" target="_blank">http://digium.com</a> & <a href="http://asterisk.org" target="_blank">http://asterisk.org</a><br>
<br>
--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>
</font></span></blockquote></div><br></div>