colin/NetworkManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

all: use G_SPAWN_CLOEXEC_PIPES with g_spawn_async_with_pipes()

Browse Source 
G_SPAWN_CLOEXEC_PIPES is supported since glib 2.40, which we already
depend on.
This commit is contained in:
Thomas Haller
2023-04-03 19:38:00 +02:00
parent 17ae6a4413
commit 31fd8f60cf
5 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/core/dns/nm-dns-manager.c
View File

@@ -647,7 +647,7 @@ run_netconfig(NMDnsManager *self, GError **error, int *stdin_fd)
if (!g_spawn_async_with_pipes(NULL,
argv,
NULL,
G_SPAWN_DO_NOT_REAP_CHILD,
G_SPAWN_CLOEXEC_PIPES | G_SPAWN_DO_NOT_REAP_CHILD,
NULL,
NULL,
&pid,

2
src/core/nm-core-utils.c
View File

@@ -5108,7 +5108,7 @@ nm_utils_spawn_helper(const char *const *args,
if (!g_spawn_async_with_pipes("/",
(char **) NM_MAKE_STRV(LIBEXECDIR "/nm-daemon-helper"),
(char **) NM_MAKE_STRV(),
G_SPAWN_DO_NOT_REAP_CHILD,
G_SPAWN_CLOEXEC_PIPES | G_SPAWN_DO_NOT_REAP_CHILD,
NULL,
NULL,
&info->pid,

3
src/libnm-client-test/nm-test-utils-impl.c
View File

@@ -107,7 +107,8 @@ nmtstc_service_init(void)
g_spawn_async_with_pipes(NULL,
(char **) args,
NULL,
G_SPAWN_SEARCH_PATH | G_SPAWN_DO_NOT_REAP_CHILD,
G_SPAWN_CLOEXEC_PIPES | G_SPAWN_SEARCH_PATH
| G_SPAWN_DO_NOT_REAP_CHILD,
NULL,
NULL,
&info->pid,

2
src/libnmc-base/nm-polkit-listener.c
View File

@@ -514,7 +514,7 @@ begin_authentication(AuthRequest *request)
if (!g_spawn_async_with_pipes(NULL,
(char **) helper_argv,
NULL,
G_SPAWN_STDERR_TO_DEV_NULL,
G_SPAWN_CLOEXEC_PIPES | G_SPAWN_STDERR_TO_DEV_NULL,
NULL,
NULL,
NULL,

2
src/libnmc-base/nm-secret-agent-simple.c
View File

@@ -787,7 +787,7 @@ try_spawn_vpn_auth_helper(RequestData *request, GPtrArray *secrets)
if (!g_spawn_async_with_pipes(NULL,
(char **) auth_dialog_argv->pdata,
NULL,
G_SPAWN_DO_NOT_REAP_CHILD,
G_SPAWN_CLOEXEC_PIPES | G_SPAWN_DO_NOT_REAP_CHILD,
NULL,
NULL,
&auth_dialog_pid,