From: Marius Vlad <marius.vlad@collabora.com>
Date: Mon, 20 Jun 2022 13:16:13 +0000 (+0300)
Subject: shell: Fix a typo when adding surface to pending list
X-Git-Tag: 13.93.0~2
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=baf8a28e6329fdb6cdfb943925849d2cb6347730;p=src%2Fagl-compositor.git

shell: Fix a typo when adding surface to pending list

As both names are similar this was probably a typo, and instead of using
the iterator for the list we should use the item itself. This was pretty
hard to track as initially everything was set-up alright.

We were still getting empty lists of pending remote surfaces, when I
observed that we were using the iterator rather than the new item we
just create in that function.

Bug-AGL: SPEC-4445
Reported-by: Scott Murray <scott.murray@konsulko.com>
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I77c4c6721eb4a4e3e8e66faa4fa0584b136d322e
---

diff --git a/src/shell.c b/src/shell.c
index 33a7a72..dc07c26 100644
--- a/src/shell.c
+++ b/src/shell.c
@@ -467,7 +467,7 @@ ivi_set_pending_desktop_surface_remote(struct ivi_output *ioutput,
 	if (!p_remote)
 		return;
 
-	wl_list_insert(&ivi->remote_pending_apps, &remote->link);
+	wl_list_insert(&ivi->remote_pending_apps, &p_remote->link);
 }