Not all application will set-up an app_id and we might end up
checking against those. So avoid doing that because str* kind of like
functions do not like that. Discovered when playing with cluster
receiver and sub-surfaces.
Bug-AGL: SPEC-3382
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I31b454fd3ccb1770775a00b1c953d9cbef827909
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
- if (wl_list_empty(&ivi->popup_pending_apps))
+ if (wl_list_empty(&ivi->popup_pending_apps) || !_app_id)
return false;
wl_list_for_each_safe(p_popup, next_p_popup,
return false;
wl_list_for_each_safe(p_popup, next_p_popup,
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
- if (wl_list_empty(&ivi->split_pending_apps))
+ if (wl_list_empty(&ivi->split_pending_apps) || !_app_id)
return false;
wl_list_for_each_safe(split_surf, next_split_surf,
return false;
wl_list_for_each_safe(split_surf, next_split_surf,
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
- if (wl_list_empty(&ivi->fullscreen_pending_apps))
+ if (wl_list_empty(&ivi->fullscreen_pending_apps) || !_app_id)
return false;
wl_list_for_each_safe(fs_surf, next_fs_surf,
return false;
wl_list_for_each_safe(fs_surf, next_fs_surf,
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
const char *_app_id =
weston_desktop_surface_get_app_id(surface->dsurface);
- if (wl_list_empty(&ivi->remote_pending_apps))
+ if (wl_list_empty(&ivi->remote_pending_apps) || !_app_id)
return false;
wl_list_for_each_safe(remote_surf, next_remote_surf,
return false;
wl_list_for_each_safe(remote_surf, next_remote_surf,