[Home]

Summary:ASTERISK-03566: [patch] app_meetme admin pin patch breaks parsing of meetme.conf
Reporter:kuj (kuj)Labels:
Date Opened:2005-02-22 15:50:00.000-0600Date Closed:2008-01-15 15:25:55.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) meetme.txt
Description:Today's patch to app_meetme.c (1.81) breaks the parsing of meetme.conf. While mochouinard's original patch for ASTERISK-2355 still "breaks up" the "conf => ..." entries at both "|" and "," seperators, looks like Mark's released CVS version does not use "," as field seperator in meetme.conf.

****** ADDITIONAL INFORMATION ******

Meetme.conf.sample documents "," as the only field seperator for the room, pin and admin pin. However, with latest patch a correct meetme.conf is ignoring "," as a seperator, resulting in "invalid conference number" messages.

Too trivial to need a disclaimer.
Comments:By: Mark Spencer (markster) 2005-02-22 23:10:55.000-0600

Sorry!  My mistake.

By: Digium Subversion (svnbot) 2008-01-15 15:25:55.000-0600

Repository: asterisk
Revision: 5067

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r5067 | markster | 2008-01-15 15:25:55 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix silly typo (bug ASTERISK-3566)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=5067