aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDominique Martinet <asmadeus@codewreck.org>2018-06-28 13:37:59 +0900
committerDominique Martinet <asmadeus@codewreck.org>2018-06-28 13:54:35 +0900
commit970687a01c953773d911032c9ab13fd080212296 (patch)
treea99937233941b8c990d7cf9d26f792c330b24cc5
parent9f1d6c58ed16bf2a7c1e4f39b38848adfe426095 (diff)
xdg_shell popup: fix potential segv in handle_destroy
surface could be NULL there if the popup had been made inert before
-rw-r--r--types/xdg_shell/wlr_xdg_popup.c2
-rw-r--r--types/xdg_shell_v6/wlr_xdg_popup_v6.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/types/xdg_shell/wlr_xdg_popup.c b/types/xdg_shell/wlr_xdg_popup.c
index 284f36ad..2435b9c4 100644
--- a/types/xdg_shell/wlr_xdg_popup.c
+++ b/types/xdg_shell/wlr_xdg_popup.c
@@ -186,7 +186,7 @@ static void xdg_popup_handle_destroy(struct wl_client *client,
struct wlr_xdg_surface *surface =
wlr_xdg_surface_from_popup_resource(resource);
- if (!wl_list_empty(&surface->popups)) {
+ if (surface && !wl_list_empty(&surface->popups)) {
wl_resource_post_error(surface->client->resource,
XDG_WM_BASE_ERROR_NOT_THE_TOPMOST_POPUP,
"xdg_popup was destroyed while it was not the topmost popup");
diff --git a/types/xdg_shell_v6/wlr_xdg_popup_v6.c b/types/xdg_shell_v6/wlr_xdg_popup_v6.c
index 334519a5..cc6fa58a 100644
--- a/types/xdg_shell_v6/wlr_xdg_popup_v6.c
+++ b/types/xdg_shell_v6/wlr_xdg_popup_v6.c
@@ -214,7 +214,7 @@ static void xdg_popup_handle_destroy(struct wl_client *client,
struct wlr_xdg_surface_v6 *surface =
xdg_surface_from_xdg_popup_resource(resource);
- if (!wl_list_empty(&surface->popups)) {
+ if (surface && !wl_list_empty(&surface->popups)) {
wl_resource_post_error(surface->client->resource,
ZXDG_SHELL_V6_ERROR_NOT_THE_TOPMOST_POPUP,
"xdg_popup was destroyed while it was not the topmost popup");