[asterisk-commits] tilghman: trunk r206567 - in /trunk: apps/ contrib/scripts/

[SVN commits to the Asterisk project]

Author: tilghman
Date: Tue Jul 14 15:14:45 2009
New Revision: 206567

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=206567
Log:
Document all meetme realtime fields, and in the process, make some field lengths more consistent.
(closes issue #15493)
 Reported by: lasko
 Patches: 
       meetme.diff uploaded by lasko (license 833)

Modified:
    trunk/apps/app_meetme.c
    trunk/contrib/scripts/meetme.sql

Modified: trunk/apps/app_meetme.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/apps/app_meetme.c?view=diff&rev=206567&r1=206566&r2=206567
==============================================================================
--- trunk/apps/app_meetme.c (original)
+++ trunk/apps/app_meetme.c Tue Jul 14 15:14:45 2009
@@ -658,7 +658,7 @@
 
 #define MAX_CONFNUM 80
 #define MAX_PIN     80
-#define OPTIONS_LEN 32
+#define OPTIONS_LEN 100
 
 enum announcetypes {
 	CONF_HASJOIN,
@@ -1892,7 +1892,7 @@
 	struct timeval now;
 	struct ast_tm tm;
 	struct ast_variable *var, *orig_var;
-	char bookid[8]; 
+	char bookid[51];
 
 	if (!extendby) {
 		return 0;
@@ -3433,13 +3433,13 @@
 		int maxusers = 0;
 		struct timeval now;
 		char recordingfilename[256] = "";
-		char recordingformat[10] = "";
+		char recordingformat[11] = "";
 		char currenttime[19] = "";
 		char eatime[19] = "";
-		char bookid[19] = "";
+		char bookid[51] = "";
 		char recordingtmp[AST_MAX_EXTENSION] = "";
-		char useropts[OPTIONS_LEN]; /* Used for RealTime conferences */
-		char adminopts[OPTIONS_LEN];
+		char useropts[OPTIONS_LEN + 1]; /* Used for RealTime conferences */
+		char adminopts[OPTIONS_LEN + 1];
 		struct ast_tm tm, etm;
 		struct timeval endtime = { .tv_sec = 0 };
 		const char *var2;
@@ -3503,11 +3503,11 @@
 			} else if (!strcasecmp(var->name, "bookId")) {
 				ast_copy_string(bookid, var->value, sizeof(bookid));
 			} else if (!strcasecmp(var->name, "opts")) {
-				ast_copy_string(useropts, var->value, sizeof(char[OPTIONS_LEN]));
+				ast_copy_string(useropts, var->value, sizeof(char[OPTIONS_LEN + 1]));
 			} else if (!strcasecmp(var->name, "maxusers")) {
 				maxusers = atoi(var->value);
 			} else if (!strcasecmp(var->name, "adminopts")) {
-				ast_copy_string(adminopts, var->value, sizeof(char[OPTIONS_LEN]));
+				ast_copy_string(adminopts, var->value, sizeof(char[OPTIONS_LEN + 1]));
 			} else if (!strcasecmp(var->name, "recordingfilename")) {
 				ast_copy_string(recordingfilename, var->value, sizeof(recordingfilename));
 			} else if (!strcasecmp(var->name, "recordingformat")) {

Modified: trunk/contrib/scripts/meetme.sql
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/contrib/scripts/meetme.sql?view=diff&rev=206567&r1=206566&r2=206567
==============================================================================
--- trunk/contrib/scripts/meetme.sql (original)
+++ trunk/contrib/scripts/meetme.sql Tue Jul 14 15:14:45 2009
@@ -4,17 +4,27 @@
 
 CREATE TABLE meetme (
 	confno char(80) DEFAULT '0' NOT NULL,
+	-- Web booking id for the conference
+	bookId char(50) NULL,
 	-- Must set schedule=yes in meetme.conf to use starttime and endtime
 	starttime datetime NULL,
 	endtime datetime NULL,
 	-- PIN to enter the conference, if any
-	pin char(20) NULL,
+	pin char(30) NULL,
+	-- Options to associate with normal users of the conference
+	opts char(100) NULL,
 	-- PIN to enter the conference as an administrator, if any
-	adminpin char(20) NULL,
+	adminpin char(30) NULL,
+	-- Options to associate with administrator users of the conference
+	adminopts char(100) NULL,
 	-- Current count of conference participants
 	members integer DEFAULT 0 NOT NULL,
 	-- Maximum conference participants allowed concurrently
 	maxusers integer DEFAULT 0 NOT NULL,
+	-- Recording of the conference, if any
+	recordingfilename char(255) NULL,
+	-- File format of the conference recording, if any
+	recordingformat char(10) NULL,
 	PRIMARY KEY (confno, starttime)
 );