gnumeric r16681 - in trunk: . plugins/excel

[mortenw svn gnome org]

Author: mortenw
Date: Sun Jun 29 13:08:19 2008
New Revision: 16681
URL: http://svn.gnome.org/viewvc/gnumeric?rev=16681&view=rev

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

	* ms-chart.c (xl_chart_import_error_bar): More checks on parent.
	Fixes #540697.



Modified:
   trunk/NEWS
   trunk/plugins/excel/ChangeLog
   trunk/plugins/excel/ms-chart.c

Modified: trunk/NEWS
==============================================================================
--- trunk/NEWS	(original)
+++ trunk/NEWS	Sun Jun 29 13:08:19 2008
@@ -8,6 +8,7 @@
 	* Fix EOMONTH issue.  [#540069]
 	* Fix phyper hang.  [#540211]
 	* Fix openoffice -pedantic problem.  [#540180]
+	* Fix bogus-xls problem.  [#540697]
 
 --------------------------------------------------------------------------
 Gnumeric 1.9.1

Modified: trunk/plugins/excel/ms-chart.c
==============================================================================
--- trunk/plugins/excel/ms-chart.c	(original)
+++ trunk/plugins/excel/ms-chart.c	Sun Jun 29 13:08:19 2008
@@ -3231,7 +3231,8 @@
 static void
 xl_chart_import_error_bar (XLChartReadState *state, XLChartSeries *series)
 {
-	XLChartSeries *parent = g_ptr_array_index (state->series, series->err_parent);
+	unsigned p = series->err_parent;
+	XLChartSeries *parent;
 	Sheet *sheet;
 	int orig_dim;
 	GogMSDimType msdim;
@@ -3239,6 +3240,9 @@
 		? "x-errors" : "y-errors";
 	GParamSpec *pspec;
 
+	XL_CHECK_CONDITION (p < state->series->len);
+
+	parent = g_ptr_array_index (state->series, p);
 	XL_CHECK_CONDITION (parent != NULL && parent->series != NULL);
 
 	pspec = g_object_class_find_property (