From d2510f1e79e501a8e6aecb8058746a7456d1367e Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Thu, 18 Apr 2024 15:02:56 +0300 Subject: [PATCH] compositor: Fix wet_process cleanup on compositor shuwdown Bug-AGL: PEC-5104 Signed-off-by: Marius Vlad Change-Id: Ib57d93378847cf2c7fa07782d983a179fa4a7ce8 --- src/compositor.c | 14 ++++++++++++++ src/ivi-compositor.h | 4 ++++ src/xwayland.c | 12 +----------- 3 files changed, 19 insertions(+), 11 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index d82676b..1881702 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -73,6 +73,16 @@ to_ivi_compositor(struct weston_compositor *ec) return weston_compositor_get_user_data(ec); } +void +ivi_process_destroy(struct wet_process *process, int status, bool call_cleanup) +{ + wl_list_remove(&process->link); + if (call_cleanup && process->cleanup) + process->cleanup(process, status, process->cleanup_data); + free(process->path); + free(process); +} + struct ivi_output_config * ivi_init_parsed_options(struct weston_compositor *compositor) { @@ -2034,6 +2044,7 @@ int wet_main(int argc, char *argv[], const struct weston_testsuite_data *test_da bool xwayland = false; struct sigaction action; char *renderer = NULL; + struct wet_process *process, *process_tmp; const struct weston_option core_options[] = { { WESTON_OPTION_STRING, "renderer", 'r', &renderer }, @@ -2252,6 +2263,9 @@ error_compositor: ivi_policy_destroy(ivi.policy); + wl_list_for_each_safe(process, process_tmp, &ivi.child_process_list, link) + ivi_process_destroy(process, 0, false); + error_signals: for (size_t i = 0; i < ARRAY_LENGTH(signals); ++i) if (signals[i]) diff --git a/src/ivi-compositor.h b/src/ivi-compositor.h index 7afb0c9..1303e44 100644 --- a/src/ivi-compositor.h +++ b/src/ivi-compositor.h @@ -29,6 +29,7 @@ #include #include "config.h" +#include #include #include #include @@ -566,4 +567,7 @@ _ivi_set_shell_surface_split(struct ivi_surface *surface, struct ivi_output *out struct ivi_output_config * ivi_init_parsed_options(struct weston_compositor *compositor); +void +ivi_process_destroy(struct wet_process *process, int status, bool call_cleanup); + #endif diff --git a/src/xwayland.c b/src/xwayland.c index 229ce75..3bea411 100644 --- a/src/xwayland.c +++ b/src/xwayland.c @@ -55,16 +55,6 @@ struct wet_xwayland { struct wet_process *process; }; -static void -process_destroy(struct wet_process *process, int status, bool call_cleanup) -{ - wl_list_remove(&process->link); - if (call_cleanup && process->cleanup) - process->cleanup(process, status, process->cleanup_data); - free(process->path); - free(process); -} - static int handle_display_fd(int fd, uint32_t mask, void *data) { @@ -337,7 +327,7 @@ wet_xwayland_destroy(struct weston_compositor *compositor, void *data) /* Calling this will call the process cleanup, in turn cleaning up the * client and the core Xwayland state */ if (wxw->process) - process_destroy(wxw->process, 0, true); + ivi_process_destroy(wxw->process, 0, true); free(wxw); } -- 2.16.6