diff options
author | Ryan Dwyer <ryandwyer1@gmail.com> | 2018-07-28 22:13:13 +1000 |
---|---|---|
committer | Ryan Dwyer <ryandwyer1@gmail.com> | 2018-07-28 22:41:24 +1000 |
commit | 84cd22c8cb722daaa9250a792da0f44930accfae (patch) | |
tree | 598a024d300d4bdb700dff328a919b94b06a1e29 /sway/commands/sticky.c | |
parent | a2164c666197e983b2059647e31fd93491893c43 (diff) |
Fix crash when a deferred command destroys a workspace
Example config that produces the crash (with a single output):
workspace 1
workspace 2
Prior to this commit, container_workspace_free would manually mark the
L_FLOATING container as destroying and free it. This assumed the
L_FLOATING container would never be involved in a transaction. This was
a safe assumption when it was implemented, but became an incorrect
assumption once parent/child relationships became transactionised.
This commit removes the L_FLOATING free from container_workspace_free.
When the workspace is destroyed, it starts the normal destroy process on
the L_FLOATING container so it can be freed via transactions.
Diffstat (limited to 'sway/commands/sticky.c')
0 files changed, 0 insertions, 0 deletions