From: Christopher Rienzo <chris@rienzo.net>
Date: Thu, 19 Jul 2012 00:29:26 +0000 (+0000)
Subject: parse uuid and descriptor name in start_tone_detect API
X-Git-Tag: v1.2.0~116
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=338c1737522ebb86d44118bc2c17e28973b413b4;p=thirdparty%2Ffreeswitch.git

parse uuid and descriptor name in start_tone_detect API
---

diff --git a/src/mod/applications/mod_spandsp/mod_spandsp.c b/src/mod/applications/mod_spandsp/mod_spandsp.c
index 33a06a067a..7fd5fedfb4 100644
--- a/src/mod/applications/mod_spandsp/mod_spandsp.c
+++ b/src/mod/applications/mod_spandsp/mod_spandsp.c
@@ -266,19 +266,30 @@ SWITCH_STANDARD_API(start_tone_detect_api)
 {
 	switch_status_t status = SWITCH_STATUS_SUCCESS;
 	switch_core_session_t *psession = NULL;
+	char *puuid = NULL, *descriptor = NULL;
 
 	if (zstr(cmd)) {
-		stream->write_function(stream, "-ERR missing descriptor name\n");
+		stream->write_function(stream, "-ERR missing uuid\n");
 		return SWITCH_STATUS_SUCCESS;
 	}
 
-	if (!(psession = switch_core_session_locate(cmd))) {
-		stream->write_function(stream, "-ERR Cannot locate session\n");
-		return SWITCH_STATUS_SUCCESS;
+	puuid = strdup((char *)cmd);
+
+	if ((descriptor = strchr(puuid, ' '))) {
+		*descriptor++ = '\0';
+	}
+
+	if (zstr(descriptor)) {
+		stream->write_function(stream, "-ERR missing descriptor name\n");
+		goto end;
 	}
 
+	if (!(psession = switch_core_session_locate(puuid))) {
+		stream->write_function(stream, "-ERR Cannot locate session\n");
+		goto end;
+	}
 
-	status = callprogress_detector_start(psession, cmd);
+	status = callprogress_detector_start(psession, descriptor);
 
 	if (status == SWITCH_STATUS_SUCCESS) {
 		stream->write_function(stream, "+OK started\n");
@@ -288,10 +299,12 @@ SWITCH_STANDARD_API(start_tone_detect_api)
 
 	switch_core_session_rwunlock(psession);
 
-	return status;
-}
+ end:
 
+	switch_safe_free(puuid);
 
+	return status;
+}
 
 /**
  * Stop tone detector application
@@ -319,7 +332,7 @@ SWITCH_STANDARD_API(stop_tone_detect_api)
 	switch_core_session_t *psession = NULL;
 
 	if (zstr(cmd)) {
-		stream->write_function(stream, "-ERR missing descriptor name\n");
+		stream->write_function(stream, "-ERR missing session UUID\n");
 		return SWITCH_STATUS_SUCCESS;
 	}
 
@@ -732,10 +745,10 @@ SWITCH_MODULE_LOAD_FUNCTION(mod_spandsp_init)
 	if (mod_spandsp_dsp_load(module_interface, pool) != SWITCH_STATUS_SUCCESS) {
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "Couldn't load or process spandsp.conf, not adding tone_detect applications\n");
 	} else {
-		SWITCH_ADD_APP(app_interface, "start_tone_detect", "Start background tone detection with cadence", "", start_tone_detect_app, "[name]", SAF_NONE);
+		SWITCH_ADD_APP(app_interface, "start_tone_detect", "Start background tone detection with cadence", "", start_tone_detect_app, "<name>", SAF_NONE);
 		SWITCH_ADD_APP(app_interface, "stop_tone_detect", "Stop background tone detection with cadence", "", stop_tone_detect_app, "", SAF_NONE);
-		SWITCH_ADD_API(api_interface, "start_tone_detect", "Start background tone detection with cadence", start_tone_detect_api, "[name]");
-		SWITCH_ADD_API(api_interface, "stop_tone_detect", "Stop background tone detection with cadence", stop_tone_detect_api, "");
+		SWITCH_ADD_API(api_interface, "start_tone_detect", "Start background tone detection with cadence", start_tone_detect_api, "<uuid> <name>");
+		SWITCH_ADD_API(api_interface, "stop_tone_detect", "Stop background tone detection with cadence", stop_tone_detect_api, "<uuid>");
 	}