select.c: Make the ugly hacker uglier...
authorHolger Hans Peter Freyther <zecke@selfish.org>
Sun, 11 Apr 2010 15:33:19 +0000 (17:33 +0200)
committerHolger Hans Peter Freyther <zecke@selfish.org>
Sun, 11 Apr 2010 15:34:52 +0000 (17:34 +0200)
Right now I'm seeing crashes when removing a link and deleting
it and I need this hack to make it not crash. We will have to
understand if llist_for_each_entry_safe has a bug or if we are
doing something bad with the list (anchors not properly initialized).

src/select.c

index 9517778..2f6afa7 100644 (file)
@@ -121,7 +121,8 @@ restart:
                /* ugly, ugly hack. If more than one filedescriptors were
                 * unregistered, they might have been consecutive and
                 * llist_for_each_entry_safe() is no longer safe */
-               if (unregistered_count > 1)
+               /* this seems to happen with the last element of the list as well */
+               if (unregistered_count >= 1)
                        goto restart;
        }
        return work;