gtk: cosmetic, make it look safer session_channel_destroy()

Although not necessary, this loop rewrite is easier to read, to see
that the function is safe: the function returns when removing the
item, so it was safe before but as well...

1 files changed, 3 insertions, 2 deletions

diff --git a/gtk/spice-session.c b/gtk/spice-session.c
index c6df211..800185b 100644
--- a/gtk/spice-session.c
+++ b/gtk/spice-session.c
@@ -697,13 +697,14 @@ void spice_session_channel_destroy(SpiceSession *session, SpiceChannel *channel)
 {
     spice_session *s = SPICE_SESSION_GET_PRIVATE(session);
     struct channel *item = NULL;
-    RingItem *ring;
+    RingItem *ring, *next;
 
     g_return_if_fail(s != NULL);
     g_return_if_fail(channel != NULL);
 
     for (ring = ring_get_head(&s->channels); ring != NULL;
-         ring = ring_next(&s->channels, ring)) {
+         ring = next) {
+        next = ring_next(&s->channels, ring);
         item = SPICE_CONTAINEROF(ring, struct channel, link);
         if (item->channel == channel) {
             ring_remove(&item->link);