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)


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@206567 65c4cc65-6c06-0410-ace0-fbb531ad65f3

2 files changed, 20 insertions, 10 deletions

diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index a011d15e6..85f4a59e8 100644
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -658,7 +658,7 @@ static int rt_log_members;
 
 #define MAX_CONFNUM 80
 #define MAX_PIN     80
-#define OPTIONS_LEN 32
+#define OPTIONS_LEN 100
 
 enum announcetypes {
 	CONF_HASJOIN,
@@ -1892,7 +1892,7 @@ static int rt_extend_conf(const char *confno)
 	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 @@ static struct ast_conference *find_conf_realtime(struct ast_channel *chan, char
 		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 @@ static struct ast_conference *find_conf_realtime(struct ast_channel *chan, char
 			} 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")) {
diff --git a/contrib/scripts/meetme.sql b/contrib/scripts/meetme.sql
index 334d9ba57..cb96e630c 100644
--- a/contrib/scripts/meetme.sql
+++ b/contrib/scripts/meetme.sql
@@ -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)
 );