From 8bc983a758aec6f41af61a7b026f21998e798a9f Mon Sep 17 00:00:00 2001
From: Michael Jerris <mike@jerris.com>
Date: Sat, 15 Dec 2007 20:50:15 +0000
Subject: [PATCH] cleanups and a check to bail on woomera messages with too
 many params instead of segfaulting.

git-svn-id: http://svn.freeswitch.org/svn/freeswitch/trunk@6817 d0543943-73ff-0310-b7d9-9358b9ac24b2
---
 src/mod/endpoints/mod_woomera/mod_woomera.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/mod/endpoints/mod_woomera/mod_woomera.c b/src/mod/endpoints/mod_woomera/mod_woomera.c
index 345ec8c192..3a07a0c416 100644
--- a/src/mod/endpoints/mod_woomera/mod_woomera.c
+++ b/src/mod/endpoints/mod_woomera/mod_woomera.c
@@ -557,9 +557,11 @@ static void woomera_printf(woomera_profile * profile, switch_socket_t * socket,
 	va_start(ap, fmt);
 #ifndef vasprintf
 	stuff = (char *) malloc(10240);
+	switch_assert(stuff);
 	vsnprintf(stuff, 10240, fmt, ap);
 #else
 	res = vasprintf(&stuff, fmt, ap);
+	switch_assert(stuff);
 #endif
 	va_end(ap);
 	if (res == -1) {
@@ -694,6 +696,9 @@ static int woomera_message_parse(switch_socket_t * fd, woomera_message * wmsg, i
 				break;
 			}
 		}
+		if (wmsg->last > WOOMERA_ARRAY_LEN) {
+			break;
+		}
 
 		if (!cur || !cur[0]) {
 			break;