Summary: | ASTERISK-17896: [patch] meetme cli cmd completion leaves conferences mutex locked | ||
Reporter: | zvision (zvision) | Labels: | |
Date Opened: | 2011-05-20 08:45:11 | Date Closed: | 2011-05-23 09:30:30 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_meetme |
Versions: | 1.6.2.18 | Frequency of Occurrence | |
Related Issues: | |||
Environment: | Attachments: | ( 0) app_meetme.diff | |
Description: | When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. | ||
Comments: | By: Richard Mudgett (rmudgett) 2011-05-20 15:58:24 The bot that automatically closes issues seems to be down. Committed to v1.4, v1.6.2, v1.8, and trunk. r320238 | rmudgett | 2011-05-20 15:53:30 -0500 (Fri, 20 May 2011) | 34 lines Changed paths: M /trunk M /trunk/apps/app_meetme.c Merged revisions 320237 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.8 ................ r320237 | rmudgett | 2011-05-20 15:49:03 -0500 (Fri, 20 May 2011) | 27 lines Merged revisions 320236 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.6.2 ................ r320236 | rmudgett | 2011-05-20 15:44:54 -0500 (Fri, 20 May 2011) | 20 lines Merged revisions 320235 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r320235 | rmudgett | 2011-05-20 15:38:22 -0500 (Fri, 20 May 2011) | 13 lines The meetme CLI command completion leaves conferences mutex locked. When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. Add missing unlock. (closes issue ASTERISK-17896) Reported by: zvision Patches: app_meetme.diff uploaded by zvision (license 798) ........ ................ ................ By: Digium Subversion (svnbot) 2011-05-23 09:30:19 Repository: asterisk Revision: 320235 U branches/1.4/apps/app_meetme.c ------------------------------------------------------------------------ r320235 | rmudgett | 2011-05-23 09:30:18 -0500 (Mon, 23 May 2011) | 13 lines The meetme CLI command completion leaves conferences mutex locked. When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. Add missing unlock. (closes issue ASTERISK-17896) Reported by: zvision Patches: app_meetme.diff uploaded by zvision (license 798) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=320235 By: Digium Subversion (svnbot) 2011-05-23 09:30:22 Repository: asterisk Revision: 320236 _U branches/1.6.2/ U branches/1.6.2/apps/app_meetme.c ------------------------------------------------------------------------ r320236 | rmudgett | 2011-05-23 09:30:22 -0500 (Mon, 23 May 2011) | 20 lines Merged revisions 320235 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r320235 | rmudgett | 2011-05-20 15:38:22 -0500 (Fri, 20 May 2011) | 13 lines The meetme CLI command completion leaves conferences mutex locked. When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. Add missing unlock. (closes issue ASTERISK-17896) Reported by: zvision Patches: app_meetme.diff uploaded by zvision (license 798) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=320236 By: Digium Subversion (svnbot) 2011-05-23 09:30:26 Repository: asterisk Revision: 320237 _U branches/1.8/ U branches/1.8/apps/app_meetme.c ------------------------------------------------------------------------ r320237 | rmudgett | 2011-05-23 09:30:25 -0500 (Mon, 23 May 2011) | 27 lines Merged revisions 320236 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.6.2 ................ r320236 | rmudgett | 2011-05-20 15:44:54 -0500 (Fri, 20 May 2011) | 20 lines Merged revisions 320235 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r320235 | rmudgett | 2011-05-20 15:38:22 -0500 (Fri, 20 May 2011) | 13 lines The meetme CLI command completion leaves conferences mutex locked. When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. Add missing unlock. (closes issue ASTERISK-17896) Reported by: zvision Patches: app_meetme.diff uploaded by zvision (license 798) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=320237 By: Digium Subversion (svnbot) 2011-05-23 09:30:29 Repository: asterisk Revision: 320238 _U trunk/ U trunk/apps/app_meetme.c ------------------------------------------------------------------------ r320238 | rmudgett | 2011-05-23 09:30:29 -0500 (Mon, 23 May 2011) | 34 lines Merged revisions 320237 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.8 ................ r320237 | rmudgett | 2011-05-20 15:49:03 -0500 (Fri, 20 May 2011) | 27 lines Merged revisions 320236 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.6.2 ................ r320236 | rmudgett | 2011-05-20 15:44:54 -0500 (Fri, 20 May 2011) | 20 lines Merged revisions 320235 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r320235 | rmudgett | 2011-05-20 15:38:22 -0500 (Fri, 20 May 2011) | 13 lines The meetme CLI command completion leaves conferences mutex locked. When issuing a meetme kick CLI command and an invalid (non-existent) conference number is specified, pressing Tab leaves the conferences mutex locked and, therefore, all conferences deadlock. Add missing unlock. (closes issue ASTERISK-17896) Reported by: zvision Patches: app_meetme.diff uploaded by zvision (license 798) ........ ................ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=320238 |