From 351b6c0543390a3e1289aec7cf14058ffc4eb6af Mon Sep 17 00:00:00 2001 From: =?utf8?q?=C3=98yvind=20Kol=C3=A5s?= Date: Wed, 16 Jun 2010 17:44:06 +0100 Subject: [PATCH] state: avoid walking off empty list When removing the last key in a list, the last part of the for statement could cause dereferencing (NULL)->next and thus segfaulting. --- clutter/clutter-state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/clutter/clutter-state.c b/clutter/clutter-state.c index e3b35c5..9e93f18 100644 --- a/clutter/clutter-state.c +++ b/clutter/clutter-state.c @@ -273,7 +273,7 @@ clutter_state_remove_key_internal (ClutterState *this, { GList *k; - for (k = target_state->keys; k; k = k->next) + for (k = target_state->keys; k != NULL; k = k?k->next:NULL) { ClutterStateKey *key = k->data; -- 2.7.4