From e554f9ee023e6711c81f47085dc6cfe707260a7c Mon Sep 17 00:00:00 2001 From: Jakob Bornecrantz <jakob@collabora.com> Date: Fri, 18 Jun 2021 17:23:35 +0100 Subject: [PATCH] ipc: Fix NULL check stopping main app visibility events from working --- doc/changes/ipc/mr.721.md | 1 + src/xrt/ipc/server/ipc_server_process.c | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/doc/changes/ipc/mr.721.md b/doc/changes/ipc/mr.721.md index 6d747090e..f8827c87d 100644 --- a/doc/changes/ipc/mr.721.md +++ b/doc/changes/ipc/mr.721.md @@ -2,6 +2,7 @@ - mr.754 - mr.768 - mr.800 +- mr.846 --- Now that there is a interface that allows the compositor to support multi-client rendering use that instead of doing our own rendering. diff --git a/src/xrt/ipc/server/ipc_server_process.c b/src/xrt/ipc/server/ipc_server_process.c index b95766b9c..0dca3b52e 100644 --- a/src/xrt/ipc/server/ipc_server_process.c +++ b/src/xrt/ipc/server/ipc_server_process.c @@ -525,8 +525,8 @@ handle_overlay_client_events(volatile struct ipc_client_state *ics, int active_i return; } - // Does this client have a compositor yet? - if (ics->xc) { + // Does this client have a compositor yet, if not return? + if (ics->xc == NULL) { return; }