From f825bf9e71ac5adfcbdc6ef7f6c2fef8555ee4a3 Mon Sep 17 00:00:00 2001 From: Marius Vlad Date: Thu, 6 Apr 2023 10:59:37 +0300 Subject: [PATCH] compositor: Don't reuse previous return values In case we couldn't enable the output do not return early without setting the proper return value. This causes an issue later one, because the signal list isn't initalized, taking down the compositor with a crash. Bug-AGL: SPEC-4734 Signed-off-by: Marius Vlad Change-Id: Ia5ebc226f6b8f702e710a5976c471d04302bcb00 --- src/compositor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compositor.c b/src/compositor.c index 5ef3b34..250c90c 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -838,7 +838,7 @@ remote_output_init(struct ivi_output *ivi_output, goto err; } - if (weston_output_enable(ivi_output->output) < 0) { + if ((ret = weston_output_enable(ivi_output->output)) < 0) { weston_log("Enabling remoted output \"%s\" failed.\n", output_name); goto err; -- 2.16.6