gnumeric r16659 - in trunk: . plugins/fn-date

[mortenw svn gnome org]

Author: mortenw
Date: Tue Jun 24 00:12:40 2008
New Revision: 16659
URL: http://svn.gnome.org/viewvc/gnumeric?rev=16659&view=rev

Log:
2008-06-23  Morten Welinder  <terra gnome org>

	* functions.c (gnumeric_networkdays): Plug leak.



Modified:
   trunk/NEWS
   trunk/plugins/fn-date/ChangeLog
   trunk/plugins/fn-date/functions.c

Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS	(original)
+++ trunk/NEWS	Tue Jun 24 00:12:40 2008
@@ -49,6 +49,7 @@
 	* Fix PERCENTRANK.  [#539672]
 	* Fix UMR in WORKDAY.  [#539808]
 	* Plug leaks in GROWTH.  [#539827]
+	* Plug leak in NETWORKDAYS.
 
 SeÃn de BÃrca:
 	* Move to tango based icons. [#450444]

Modified: trunk/plugins/fn-date/functions.c
==============================================================================
--- trunk/plugins/fn-date/functions.c	(original)
+++ trunk/plugins/fn-date/functions.c	Tue Jun 24 00:12:40 2008
@@ -1260,9 +1260,15 @@
 	res = end_serial - start_serial;
 	res -= ((res/7)*2);	/* Remove weekends */
 
-	if (argv[2] != NULL)
-		value_area_foreach (argv[2], ei->pos, CELL_ITER_IGNORE_BLANK,
-			(GnmValueIterFunc) &cb_networkdays_holiday, &close);
+	if (argv[2] != NULL) {
+		GnmValue const *e =
+			value_area_foreach (argv[2], ei->pos,
+					    CELL_ITER_IGNORE_BLANK,
+					    (GnmValueIterFunc)&cb_networkdays_holiday,
+					    &close);
+		if (e)
+			return e;
+	}
 
 	res = res - start_offset + end_offset - close.res;