diff --git a/0001-vdagentd-work-around-GLib-s-fork-issues.patch b/0001-vdagentd-work-around-GLib-s-fork-issues.patch new file mode 100644 index 0000000..2a59c19 --- /dev/null +++ b/0001-vdagentd-work-around-GLib-s-fork-issues.patch @@ -0,0 +1,103 @@ +From 9b8c0ebb9fb573e6ce3c5416371509f416503d0c Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Jakub=20Jank=C5=AF?= +Date: Fri, 20 Mar 2020 10:36:03 +0100 +Subject: [PATCH 1/2] vdagentd: work around GLib's fork issues +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Creating threads is not compatible with forking as only the thread +that calls fork() is inherited. + +Handlers registered with g_unix_signal_add() create a thread so +move these calls after fork. + +Also call g_socket_service_start() after fork to avoid creation of +new threads before it is necessary. + +Fixes: https://gitlab.freedesktop.org/spice/linux/vd_agent/issues/18 + +Also see: https://gitlab.gnome.org/GNOME/glib/issues/2073 + +Signed-off-by: Jakub Janků +Acked-by: Frediano Ziglio +--- + src/udscs.c | 6 ++++++ + src/udscs.h | 2 ++ + src/vdagentd/vdagentd.c | 9 +++++---- + 3 files changed, 13 insertions(+), 4 deletions(-) + +diff --git a/src/udscs.c b/src/udscs.c +index 4de75f8..7c99eed 100644 +--- a/src/udscs.c ++++ b/src/udscs.c +@@ -186,6 +186,7 @@ struct udscs_server *udscs_server_new( + server->read_callback = read_callback; + server->error_cb = error_cb; + server->service = g_socket_service_new(); ++ g_socket_service_stop(server->service); + + g_signal_connect(server->service, "incoming", + G_CALLBACK(udscs_server_accept_cb), server); +@@ -223,6 +224,11 @@ void udscs_server_listen_to_address(struct udscs_server *server, + g_object_unref(sock_addr); + } + ++void udscs_server_start(struct udscs_server *server) ++{ ++ g_socket_service_start(server->service); ++} ++ + void udscs_server_destroy_connection(struct udscs_server *server, + UdscsConnection *conn) + { +diff --git a/src/udscs.h b/src/udscs.h +index 45ebd3f..4f7ea36 100644 +--- a/src/udscs.h ++++ b/src/udscs.h +@@ -98,6 +98,8 @@ void udscs_server_listen_to_address(struct udscs_server *server, + const gchar *addr, + GError **err); + ++void udscs_server_start(struct udscs_server *server); ++ + void udscs_server_destroy_connection(struct udscs_server *server, + UdscsConnection *conn); + +diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c +index cfd0a51..1b63ec8 100644 +--- a/src/vdagentd/vdagentd.c ++++ b/src/vdagentd/vdagentd.c +@@ -1184,10 +1184,6 @@ int main(int argc, char *argv[]) + uinput_device = g_strdup(DEFAULT_UINPUT_DEVICE); + } + +- g_unix_signal_add(SIGINT, signal_handler, NULL); +- g_unix_signal_add(SIGHUP, signal_handler, NULL); +- g_unix_signal_add(SIGTERM, signal_handler, NULL); +- + openlog("spice-vdagentd", do_daemonize ? 0 : LOG_PERROR, LOG_USER); + + /* Setup communication with vdagent process(es) */ +@@ -1240,6 +1236,10 @@ int main(int argc, char *argv[]) + } + #endif + ++ g_unix_signal_add(SIGINT, signal_handler, NULL); ++ g_unix_signal_add(SIGHUP, signal_handler, NULL); ++ g_unix_signal_add(SIGTERM, signal_handler, NULL); ++ + if (want_session_info) + session_info = session_info_create(debug); + if (session_info) { +@@ -1252,6 +1252,7 @@ int main(int argc, char *argv[]) + + active_xfers = g_hash_table_new(g_direct_hash, g_direct_equal); + ++ udscs_server_start(server); + loop = g_main_loop_new(NULL, FALSE); + g_main_loop_run(loop); + +-- +2.25.1 + diff --git a/0002-vdagentd-init-static-uinput-before-fork.patch b/0002-vdagentd-init-static-uinput-before-fork.patch new file mode 100644 index 0000000..2b5aaf4 --- /dev/null +++ b/0002-vdagentd-init-static-uinput-before-fork.patch @@ -0,0 +1,44 @@ +From 7b0435ef66af088c1a1be20b6bc6b0fcb76e4e1a Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Jakub=20Jank=C5=AF?= +Date: Fri, 20 Mar 2020 17:18:32 +0100 +Subject: [PATCH 2/2] vdagentd: init static uinput before fork +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Otherwise the caller doesn't know that the init failed +because we're returning 0 in the parent and 1 in child. + +Signed-off-by: Jakub Janků +Acked-by: Frediano Ziglio +--- + src/vdagentd/vdagentd.c | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c +index 1b63ec8..753c9bf 100644 +--- a/src/vdagentd/vdagentd.c ++++ b/src/vdagentd/vdagentd.c +@@ -1224,9 +1224,6 @@ int main(int argc, char *argv[]) + } + } + +- if (do_daemonize) +- daemonize(); +- + #ifdef WITH_STATIC_UINPUT + uinput = vdagentd_uinput_create(uinput_device, 1024, 768, NULL, 0, + debug > 1, uinput_fake); +@@ -1236,6 +1233,9 @@ int main(int argc, char *argv[]) + } + #endif + ++ if (do_daemonize) ++ daemonize(); ++ + g_unix_signal_add(SIGINT, signal_handler, NULL); + g_unix_signal_add(SIGHUP, signal_handler, NULL); + g_unix_signal_add(SIGTERM, signal_handler, NULL); +-- +2.25.1 + diff --git a/spice-vdagent-0.18.0.tar.bz2 b/spice-vdagent-0.18.0.tar.bz2 deleted file mode 100644 index 7021e12..0000000 Binary files a/spice-vdagent-0.18.0.tar.bz2 and /dev/null differ diff --git a/spice-vdagent-0.20.0.tar.bz2 b/spice-vdagent-0.20.0.tar.bz2 new file mode 100644 index 0000000..6892800 Binary files /dev/null and b/spice-vdagent-0.20.0.tar.bz2 differ diff --git a/spice-vdagent.spec b/spice-vdagent.spec index bcffd57..503f526 100644 --- a/spice-vdagent.spec +++ b/spice-vdagent.spec @@ -1,14 +1,25 @@ Name: spice-vdagent -Version: 0.18.0 -Release: 4 +Version: 0.20.0 +Release: 1 Summary: Agent for Spice guests License: GPLv3+ URL: https://spice-space.org/ Source0: https://spice-space.org/download/releases/%{name}-%{version}.tar.bz2 -BuildRequires: systemd-devel glib2-devel spice-protocol libpciaccess-devel libXrandr-devel dbus-devel -BuildRequires: libXfixes-devel systemd desktop-file-utils libtool libXinerama-devel alsa-lib-devel + + +BuildRequires: systemd-devel +BuildRequires: glib2-devel >= 2.50 +BuildRequires: spice-protocol >= 0.14.1 +BuildRequires: libpciaccess-devel libXrandr-devel libXinerama-devel +BuildRequires: libXfixes-devel systemd desktop-file-utils libtool +BuildRequires: alsa-lib-devel dbus-devel libdrm-devel +# For autoreconf, needed after clipboard patch series +BuildRequires: automake autoconf %{?systemd_requires} + +Patch0001: 0001-vdagentd-work-around-GLib-s-fork-issues.patch +Patch0002: 0002-vdagentd-init-static-uinput-before-fork.patch %description %{name} is an optional component for enhancing user experience and performing guest-oriented management tasks. @@ -19,7 +30,7 @@ installed on domU o.s. to work. The default is 0. %package_help %prep -%autosetup -n %{name}-%{version} +%autosetup -n %{name}-%{version} -p1 %build %configure --with-session-info=systemd --with-init-script=systemd @@ -53,10 +64,13 @@ make install DESTDIR=$RPM_BUILD_ROOT V=2 %files help %defattr(-,root,root) -%doc ChangeLog README TODO +%doc COPYING CHANGELOG.md README.md %{_mandir}/man1/*.1.gz %changelog +* Fri Jul 17 2020 chengguipeng - 0.20.0-1 +- upgrade to 0.20.0 + * Fri Oct 25 2019 openEuler Buildteam - 0.18.0-4 - Type:bugfix - Id:NA