Summary: | ASTERISK-10474: msg description file not closed properly after insufficiently long recording rejected | ||
Reporter: | Ryan Mitchell (brak2718) | Labels: | |
Date Opened: | 2007-10-08 21:45:59 | Date Closed: | 2007-10-10 08:56:49 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) vm1.4.12.1.patch | |
Description: | In leave_voicemail(), the tmp message description file does not get closed if the message is shorter than minmessage. The file is deleted, but the file should still be closed first. Eventually asterisk breaks by running into process max open files limit. ****** ADDITIONAL INFORMATION ****** Tested using ODBC voicemail storage. Trivial fix. | ||
Comments: | By: Ryan Mitchell (brak2718) 2007-10-08 21:50:17 Ubuntu 7.0.4, kernel 2.6.20-16-generic. By: Digium Subversion (svnbot) 2007-10-10 08:54:29 Repository: asterisk Revision: 85242 U branches/1.4/apps/app_voicemail.c ------------------------------------------------------------------------ r85242 | file | 2007-10-10 08:54:28 -0500 (Wed, 10 Oct 2007) | 6 lines Close voicemail message description file if duration did not meet the minimum, or else we will eventually run out of file descriptors. (closes issue ASTERISK-10474) Reported by: brak2718 Patches: vm1.4.12.1.patch uploaded by brak2718 (license 279) ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-10-10 08:56:49 Repository: asterisk Revision: 85254 _U trunk/ U trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r85254 | file | 2007-10-10 08:56:48 -0500 (Wed, 10 Oct 2007) | 14 lines Merged revisions 85242 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r85242 | file | 2007-10-10 11:14:56 -0300 (Wed, 10 Oct 2007) | 6 lines Close voicemail message description file if duration did not meet the minimum, or else we will eventually run out of file descriptors. (closes issue ASTERISK-10474) Reported by: brak2718 Patches: vm1.4.12.1.patch uploaded by brak2718 (license 279) ........ ------------------------------------------------------------------------ |