aboutsummaryrefslogtreecommitdiffstats
path: root/my-evolution
diff options
context:
space:
mode:
authorJon Trowbridge <trow@ximian.com>2001-10-26 03:45:07 +0800
committerJon Trowbridge <trow@src.gnome.org>2001-10-26 03:45:07 +0800
commit580b80f5b3e1a018fc4aeaa9b1b29c24e556a763 (patch)
tree0ef5beb5e419baa08e806b5227d227ffe8894db9 /my-evolution
parent33fc2cca986200618aed96743fe3aac8cbeb9948 (diff)
downloadgsoc2013-evolution-580b80f5b3e1a018fc4aeaa9b1b29c24e556a763.tar.gz
gsoc2013-evolution-580b80f5b3e1a018fc4aeaa9b1b29c24e556a763.tar.zst
gsoc2013-evolution-580b80f5b3e1a018fc4aeaa9b1b29c24e556a763.zip
It is possible for start_foo.value to be NULL after a cal to
2001-10-25 Jon Trowbridge <trow@ximian.com> * e-summary-tasks.c (sort_uids): It is possible for start_foo.value to be NULL after a cal to cal_component_get_dtstart, so we need to check for this before dereferencing it. (Bug #13259) svn path=/trunk/; revision=14097
Diffstat (limited to 'my-evolution')
-rw-r--r--my-evolution/ChangeLog7
-rw-r--r--my-evolution/e-summary-tasks.c7
2 files changed, 13 insertions, 1 deletions
diff --git a/my-evolution/ChangeLog b/my-evolution/ChangeLog
index cceecbda73..9204d054cd 100644
--- a/my-evolution/ChangeLog
+++ b/my-evolution/ChangeLog
@@ -1,3 +1,10 @@
+2001-10-25 Jon Trowbridge <trow@ximian.com>
+
+ * e-summary-tasks.c (sort_uids): It is possible for
+ start_foo.value to be NULL after a cal to
+ cal_component_get_dtstart, so we need to check for this before
+ dereferencing it. (Bug #13259)
+
2001-10-24 Iain Holmes <iain@ximian.com>
* e-summary-mail.c (e_summary_mail_init): Don't free the mail part.
diff --git a/my-evolution/e-summary-tasks.c b/my-evolution/e-summary-tasks.c
index 1b2548ac57..57350c3bc2 100644
--- a/my-evolution/e-summary-tasks.c
+++ b/my-evolution/e-summary-tasks.c
@@ -151,7 +151,12 @@ sort_uids (gconstpointer a,
cal_component_get_dtstart (comp_a, &start_a);
cal_component_get_dtstart (comp_b, &start_b);
- retval = icaltime_compare (*start_a.value, *start_b.value);
+ if (start_a.value == NULL || start_b.value == NULL) {
+ /* Try to do something reasonable if one or more of our .values is NULL */
+ retval = (start_a.value ? 1 : 0) - (start_b.value ? 1 : 0);
+ } else {
+ retval = icaltime_compare (*start_a.value, *start_b.value);
+ }
cal_component_free_datetime (&start_a);
cal_component_free_datetime (&start_b);