@dbungert: While the patch that simply copies the list is a possible way of doing things, a very experienced user on the Arch Linux forums had concerns about doing things that way, see https://bbs.archlinux.org/viewtopic.php?pid=2090570#p2090570 "But creating a copy of the list is inefficient and only glosses over the actual problem (and probably will cause X11 errors and/or behavioral differences - maybe maintain them b/c of the status quo ante)"
In the two-item case, as long as the rest of Openbox code keeps the list consistent after removing *and adding* an item (because keep in mind the code doesn't just remove stuff, it also adds stuff), everything should be fine. itPrev->next should point to a valid item unless the underlying list has become corrupted. In the one item case, the first iteration through the loop works, and with only one item, there will only be one iteration through the loop, so presumably the crash won't happen unless the one item case triggers a crash in some other way. I'll test both and report back what happens. -- You received this bug notification because you are a member of Ubuntu Desktop Bugs, which is subscribed to glib2.0 in Ubuntu. https://bugs.launchpad.net/bugs/2011751 Title: openbox crashed with SIGABRT To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/glib2.0/+bug/2011751/+subscriptions -- desktop-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/desktop-bugs
