backend/multi: handle backends depending on each other properly

wl_list_for_each_safe only allows the current list item to be removed.
If a backend destroys itself when another backend is destroyed, this
blows up.
This commit is contained in:
Simon Ser 2020-05-02 16:42:26 +02:00 committed by Drew DeVault
parent e3343cf7d1
commit 7720ce7827

View File

@ -49,8 +49,11 @@ static void multi_backend_destroy(struct wlr_backend *wlr_backend) {
wl_list_remove(&backend->display_destroy.link); wl_list_remove(&backend->display_destroy.link);
struct subbackend_state *sub, *next; // Some backends may depend on other backends, ie. destroying a backend may
wl_list_for_each_safe(sub, next, &backend->backends, link) { // also destroy other backends
while (!wl_list_empty(&backend->backends)) {
struct subbackend_state *sub =
wl_container_of(backend->backends.next, sub, link);
wlr_backend_destroy(sub->backend); wlr_backend_destroy(sub->backend);
} }