[prev in list] [next in list] [prev in thread] [next in thread] 

List:       asterisk-commits
Subject:    [asterisk-commits] dlee: branch dlee/stasis-http r382285 - in /team/dlee/stasis-http: ./ main/ t...
From:       SVN commits to the Asterisk project <asterisk-commits () lists ! digium ! com>
Date:       2013-02-28 20:20:24
Message-ID: E1UB9xp-0003J8-65 () wibble ! digium ! internal
[Download RAW message or body]

Author: dlee
Date: Thu Feb 28 14:20:21 2013
New Revision: 382285

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=382285
Log:
Addressing opticron's review comments.
........

Merged revisions 382283 from \
                http://svn.asterisk.org/svn/asterisk/team/dlee/stasis-core
........

Merged revisions 382284 from \
http://svn.asterisk.org/svn/asterisk/team/dlee/stasis-app

Modified:
    team/dlee/stasis-http/   (props changed)
    team/dlee/stasis-http/main/channel.c
    team/dlee/stasis-http/main/manager.c
    team/dlee/stasis-http/main/stasis.c
    team/dlee/stasis-http/tests/test_stasis.c

Propchange: team/dlee/stasis-http/
------------------------------------------------------------------------------
--- svnmerge-integrated (original)
+++ svnmerge-integrated Thu Feb 28 14:20:21 2013
@@ -1,1 +1,1 @@
-/team/dlee/stasis-app:1-382278
+/team/dlee/stasis-app:1-382284

Modified: team/dlee/stasis-http/main/channel.c
URL: http://svnview.digium.com/svn/asterisk/team/dlee/stasis-http/main/channel.c?view=diff&rev=382285&r1=382284&r2=382285
 ==============================================================================
--- team/dlee/stasis-http/main/channel.c (original)
+++ team/dlee/stasis-http/main/channel.c Thu Feb 28 14:20:21 2013
@@ -272,6 +272,10 @@
 	}
 	event->variable = ast_strdup(name);
 	event->value = ast_strdup(value);
+	if (event->channel_name == NULL || event->uniquid == NULL || event->variable == \
NULL || event->value == NULL) { +		ast_log(LOG_ERROR, "Allocation failed\n");
+		return;
+	}
 
 	msg = stasis_message_create(ast_channel_varset_event(), event);
 	if (!msg) {

Modified: team/dlee/stasis-http/main/manager.c
URL: http://svnview.digium.com/svn/asterisk/team/dlee/stasis-http/main/manager.c?view=diff&rev=382285&r1=382284&r2=382285
 ==============================================================================
--- team/dlee/stasis-http/main/manager.c (original)
+++ team/dlee/stasis-http/main/manager.c Thu Feb 28 14:20:21 2013
@@ -7482,7 +7482,7 @@
 	char *manager_event = NULL;
 
 	if (!new_snapshot) {
-		/* Ignore cache clearing events; we'll se the hangup first */
+		/* Ignore cache clearing events; we'll see the hangup first */
 		return;
 	}
 

Modified: team/dlee/stasis-http/main/stasis.c
URL: http://svnview.digium.com/svn/asterisk/team/dlee/stasis-http/main/stasis.c?view=diff&rev=382285&r1=382284&r2=382285
 ==============================================================================
--- team/dlee/stasis-http/main/stasis.c (original)
+++ team/dlee/stasis-http/main/stasis.c Thu Feb 28 14:20:21 2013
@@ -309,7 +309,11 @@
 		SCOPED_AO2LOCK(lock, topic);
 		num_subscribers = topic->num_subscribers_current;
 		subscribers = ast_malloc(num_subscribers * sizeof(*subscribers));
-		memcpy(subscribers, topic->subscribers, num_subscribers * sizeof(*subscribers));
+		if (subscribers) {
+			memcpy(subscribers,
+			       topic->subscribers,
+			       num_subscribers * sizeof(*subscribers));
+		}
 	}
 
 	if (!subscribers) {
@@ -327,7 +331,7 @@
 
 			dispatch = dispatch_create(publisher_topic, message, sub);
 			if (!dispatch) {
-				ast_log(LOG_DEBUG, "Allocation failure; dropping message\n");
+				ast_log(LOG_DEBUG, "Allocation failure; dropping dispatch\n");
 				break;
 			}
 

Modified: team/dlee/stasis-http/tests/test_stasis.c
URL: http://svnview.digium.com/svn/asterisk/team/dlee/stasis-http/tests/test_stasis.c?view=diff&rev=382285&r1=382284&r2=382285
 ==============================================================================
--- team/dlee/stasis-http/tests/test_stasis.c (original)
+++ team/dlee/stasis-http/tests/test_stasis.c Thu Feb 28 14:20:21 2013
@@ -545,24 +545,6 @@
 	return AST_TEST_PASS;
 }
 
-/*
-AST_TEST_DEFINE()
-{
-	switch (cmd) {
-	case TEST_INIT:
-		info->name = __func__;
-		info->category = test_category;
-		info->summary = "";
-		info->description = "";
-		return AST_TEST_NOT_RUN;
-	case TEST_EXECUTE:
-		break;
-	}
-
-	return AST_TEST_PASS;
-}
-*/
-
 static int unload_module(void)
 {
 	AST_TEST_UNREGISTER(message_type);


--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-commits mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-commits


[prev in list] [next in list] [prev in thread] [next in thread]