Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Leap:15.6:Update
spice-vdagent.17026
vdagentd-work-around-GLib-s-fork-issues.patch
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File vdagentd-work-around-GLib-s-fork-issues.patch of Package spice-vdagent.17026
From ed40c7b8a610ab8dfe188bd81bf74461dd2e6488 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jakub=20Jank=C5=AF?= <jjanku@redhat.com> Date: Fri, 20 Mar 2020 10:36:03 +0100 Subject: [PATCH 06/20] vdagentd: work around GLib's fork issues MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Git-commit: 9b8c0ebb9fb573e6ce3c5416371509f416503d0c References: bsc#1173749 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ů <jjanku@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com> Signed-off-by: Bruce Rogers <brogers@suse.com> --- 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 bc8fa8d..f5dcc1b 100644 --- a/src/udscs.c +++ b/src/udscs.c @@ -189,6 +189,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); @@ -226,6 +227,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 b3c1e41..1310152 100644 --- a/src/udscs.h +++ b/src/udscs.h @@ -112,6 +112,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 cfdf820..8bf5318 100644 --- a/src/vdagentd/vdagentd.c +++ b/src/vdagentd/vdagentd.c @@ -1145,10 +1145,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) */ @@ -1201,6 +1197,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) { @@ -1213,6 +1213,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.29.0
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor