[asterisk-commits] russell: trunk r54970 - in /trunk: ./ apps/app_meetme.c

[asterisk-commits at lists.digium.com]

Author: russell
Date: Fri Feb 16 15:13:51 2007
New Revision: 54970

URL: http://svn.digium.com/view/asterisk?view=rev&rev=54970
Log:
Merged revisions 54969 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

................
r54969 | russell | 2007-02-16 15:12:18 -0600 (Fri, 16 Feb 2007) | 13 lines

Merged revisions 54955 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r54955 | russell | 2007-02-16 14:56:58 -0600 (Fri, 16 Feb 2007) | 5 lines

For conferences that are configured in meetme.conf, check the configuration
file every time someone joins the conference instead of only when the
conference is first created.  This is to ensure that changes to the pin
numbers in the config file are always honored.  (issue #9073)

........

................

Modified:
    trunk/   (props changed)
    trunk/apps/app_meetme.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/apps/app_meetme.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_meetme.c?view=diff&rev=54970&r1=54969&r2=54970
==============================================================================
--- trunk/apps/app_meetme.c (original)
+++ trunk/apps/app_meetme.c Fri Feb 16 15:13:51 2007
@@ -631,7 +631,8 @@
  * \return A pointer to the conference struct, or NULL if it wasn't found and
  *         make or dynamic were not set.
  */
-static struct ast_conference *build_conf(char *confno, char *pin, char *pinadmin, int make, int dynamic, int refcount)
+static struct ast_conference *build_conf(const char *confno, const char *pin, 
+	const char *pinadmin, int make, int dynamic, int refcount)
 {
 	struct ast_conference *cnf;
 	struct zt_confinfo ztc;
@@ -2093,6 +2094,55 @@
 	return ret;
 }
 
+static struct ast_conference *build_conf_from_config(struct ast_conference *conf, const char *confno, 
+	int make, int dynamic, int refcount)
+{
+	struct ast_config *cfg;
+	struct ast_variable *var;
+
+	if (!(cfg = ast_config_load(CONFIG_FILE_NAME))) {
+		ast_log(LOG_WARNING, "No %s file :(\n", CONFIG_FILE_NAME);
+		return NULL;
+	}
+
+	var = ast_variable_browse(cfg, "rooms");
+	for (; var; var = var->next) {
+		char *parse;
+		AST_DECLARE_APP_ARGS(args,
+			AST_APP_ARG(confno);
+			AST_APP_ARG(pin);
+			AST_APP_ARG(pinadmin);
+		);
+
+		if (strcasecmp(var->name, "conf"))
+			continue;
+
+	 	parse = ast_strdupa(var->value);
+		AST_NONSTANDARD_APP_ARGS(args, parse, ',');
+
+		if (strcasecmp(args.confno, confno))
+			continue;
+
+		if (!conf) {
+			conf = build_conf(args.confno, S_OR(args.pin, ""), 
+				S_OR(args.pinadmin, ""), make, dynamic, refcount);
+			break;
+		}
+
+		ast_copy_string(conf->pin, S_OR(args.pin, ""), sizeof(conf->pin));
+		ast_copy_string(conf->pinadmin, S_OR(args.pinadmin, ""), sizeof(conf->pinadmin));
+
+		break;
+	}
+
+	if (!var && !conf)
+		ast_log(LOG_DEBUG, "%s isn't a valid conference\n", confno);
+
+	ast_config_destroy(cfg);
+
+	return conf;
+}
+
 static struct ast_conference *find_conf_realtime(struct ast_channel *chan, char *confno, int make, int dynamic,
 						 char *dynamic_pin, size_t pin_buf_len, int refcount, struct ast_flags *confflags)
 {
@@ -2149,19 +2199,10 @@
 	return cnf;
 }
 
-
 static struct ast_conference *find_conf(struct ast_channel *chan, char *confno, int make, int dynamic,
 					char *dynamic_pin, size_t pin_buf_len, int refcount, struct ast_flags *confflags)
 {
-	struct ast_config *cfg;
-	struct ast_variable *var;
 	struct ast_conference *cnf;
-	char *parse;
-	AST_DECLARE_APP_ARGS(args,
-		AST_APP_ARG(confno);
-		AST_APP_ARG(pin);
-		AST_APP_ARG(pinadmin);
-	);
 
 	/* Check first in the conference list */
 	AST_LIST_LOCK(&confs);
@@ -2190,34 +2231,7 @@
 				cnf = build_conf(confno, "", "", make, dynamic, refcount);
 			}
 		} else {
-			/* Check the config */
-			cfg = ast_config_load(CONFIG_FILE_NAME);
-			if (!cfg) {
-				ast_log(LOG_WARNING, "No %s file :(\n", CONFIG_FILE_NAME);
-				return NULL;
-			}
-			for (var = ast_variable_browse(cfg, "rooms"); var; var = var->next) {
-				if (strcasecmp(var->name, "conf"))
-					continue;
-				
-				if (!(parse = ast_strdupa(var->value)))
-					return NULL;
-				
-				AST_NONSTANDARD_APP_ARGS(args, parse, ',');
-				if (!strcasecmp(args.confno, confno)) {
-					/* Bingo it's a valid conference */
-					cnf = build_conf(args.confno,
-							S_OR(args.pin, ""),
-							S_OR(args.pinadmin, ""),
-							make, dynamic, refcount);
-					break;
-				}
-			}
-			if (!var) {
-				if (option_debug)
-					ast_log(LOG_DEBUG, "%s isn't a valid conference\n", confno);
-			}
-			ast_config_destroy(cfg);
+			cnf = build_conf_from_config(NULL, confno, make, dynamic, refcount);
 		}
 	} else if (dynamic_pin) {
 		/* Correct for the user selecting 'D' instead of 'd' to have
@@ -2225,6 +2239,10 @@
 		   with a pin. */
 		if (dynamic_pin[0] == 'q')
 			dynamic_pin[0] = '\0';
+	} else if (!cnf->isdynamic) {
+		/* If the conference exists, check the config again, just in case
+		 * the pin in the file has changed. */
+		build_conf_from_config(cnf, confno, 0, 0, 0);
 	}
 
 	if (cnf) {