Changeset 5c2ef5b for messagelist.c

Timestamp:

Jul 3, 2011, 11:42:26 PM (13 years ago)

Author:

Jason Gross <jgross@mit.edu>

Children:

530c51c

Parents:

785ee77

Message:

Don't segfault when you expunge all messages from the messagelist

When we used owl_list, trying to get an element from an empty list would
return NULL.  When we moved to GPtrArray, we dropped this checking,
without dropping the assumption that, e.g.,
 owl_view_get_element(v, owl_global_get_curmsg(&g));
works everywhere.  This commit adds the logic back in to deal with this
case.

Additionally, don't segfault on things like :unpunt -1.

File:

• messagelist.c (modified) (1 diff)

messagelist.c

@@ -22 +22 @@
 void *owl_messagelist_get_element(const owl_messagelist *ml, int n)
 {
+  if (n >= ml->list->len) return NULL;
   return ml->list->pdata[n];
 }