Summary: | ASTERISK-25333: [patch]App meetme sometimes not close fd with pseudo module | ||
Reporter: | Afonso Zimmermann (afonso.zimmermann) | Labels: | patch |
Date Opened: | 2015-08-20 10:07:32 | Date Closed: | |
Priority: | Major | Regression? | |
Status: | Open/New | Components: | Applications/app_meetme |
Versions: | 13.5.0 | Frequency of Occurrence | Occasional |
Related Issues: | |||
Environment: | asterisk with no dahdi board, calls sendo to meetme. | Attachments: | ( 0) conf.diff |
Description: | At release a meetme call, sometimes meetme does not close fd with pseudo module. In 'conf_run' function, some uses of 'goto' C function, is used 'close(fd);' but not at all. I recent supported a scenario with dthis problem, and my solution to this issue is call 'close(fd);' after 'outrun' goto label. | ||
Comments: | By: Asterisk Team (asteriskteam) 2015-08-20 10:07:34.755-0500 Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution. A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report. Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process]. By: Rusty Newton (rnewton) 2015-08-21 08:50:33.498-0500 Would you like to provide a patch? It is always nice to have credit for your fixes! https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process By: Afonso Zimmermann (afonso.zimmermann) 2015-08-21 11:26:19.164-0500 My purpose when submit this patch is show how a have fixed this. But this need a review. I will submit this to gerrit to review. By: Rusty Newton (rnewton) 2015-08-24 09:05:16.566-0500 When you have submitted your patch to Gerrit please post the Gerrit review URL here. Thanks! |