diff options
author | Drew DeVault <sir@cmpwn.com> | 2015-12-29 00:04:12 -0500 |
---|---|---|
committer | Drew DeVault <sir@cmpwn.com> | 2015-12-29 00:04:12 -0500 |
commit | d409620a55c0552d47a75502a35e7c8bde60685d (patch) | |
tree | 6efced3d24de25e83333de9d78e71f4666891f5a | |
parent | Add bar to default config (diff) | |
download | sway-d409620a55c0552d47a75502a35e7c8bde60685d.tar.gz sway-d409620a55c0552d47a75502a35e7c8bde60685d.tar.zst sway-d409620a55c0552d47a75502a35e7c8bde60685d.zip |
Fix use-after-free when closing fullscreen views
-rw-r--r-- | sway/handlers.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sway/handlers.c b/sway/handlers.c index b8bd9eff..6c6d0e60 100644 --- a/sway/handlers.c +++ b/sway/handlers.c | |||
@@ -252,8 +252,12 @@ static void handle_view_destroyed(wlc_handle handle) { | |||
252 | } | 252 | } |
253 | 253 | ||
254 | if (view) { | 254 | if (view) { |
255 | swayc_t *parent = destroy_view(view); | 255 | bool fullscreen = swayc_is_fullscreen(view); |
256 | remove_view_from_scratchpad(view); | 256 | remove_view_from_scratchpad(view); |
257 | swayc_t *parent = destroy_view(view); | ||
258 | if (fullscreen) { | ||
259 | parent->fullscreen = NULL; | ||
260 | } | ||
257 | arrange_windows(parent, -1, -1); | 261 | arrange_windows(parent, -1, -1); |
258 | } else { | 262 | } else { |
259 | // Is it unmanaged? | 263 | // Is it unmanaged? |