[Spread-cvs] commit: r449 - in trunk: daemon java/spread

[jonathan at spread.org]

Author: jonathan
Date: 2012-01-06 22:39:20 -0500 (Fri, 06 Jan 2012)
New Revision: 449

Modified:
   trunk/daemon/Changelog
   trunk/java/spread/SpreadMessage.java
Log:
Apply fix from Matt Kowalczyk that allows array of groups to be passed to addGroups without crash

Modified: trunk/daemon/Changelog
===================================================================
--- trunk/daemon/Changelog	2012-01-07 03:35:35 UTC (rev 448)
+++ trunk/daemon/Changelog	2012-01-07 03:39:20 UTC (rev 449)
@@ -1,3 +1,8 @@
+Fri Jan  6 22:37:59 2012  Jonathan Stanton  <jonathan at spreadconcepts.com>
+
+	* SpreadMessage.java: Fix crash when add an array of groups to addGroups.
+	Patch and report by Matt Kowalczyk. 
+
 Fri Jan  6 22:33:23 2012  Jonathan Stanton  <jonathan at spreadconcepts.com>
 
 	* configure.in: Switch from undocumented AC_CONFIG_HEADER() macro

Modified: trunk/java/spread/SpreadMessage.java
===================================================================
--- trunk/java/spread/SpreadMessage.java	2012-01-07 03:35:35 UTC (rev 448)
+++ trunk/java/spread/SpreadMessage.java	2012-01-07 03:39:20 UTC (rev 449)
@@ -721,7 +721,7 @@
 	{
 		int len = groups.length;
 		for(int i = 0 ; i < len ; i++)
-			this.groups.addElement(groups);
+			this.groups.addElement(groups[i]);
 	}
 	
 	// Adds a group to the message's destination group list.