luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2024-11-11 11:56:16 +01:00
Code Issues Packages Projects Releases Wiki Activity

Add test coverage for G_SPAWN_SEARCH_PATH

Browse Source 
For manual test coverage that would reproduce the bug fixed in !1902,
copy /bin/true (or any other harmless executable) to
/usr/bin/spawn-test-helper.

Signed-off-by: Simon McVittie <smcv@collabora.com>
This commit is contained in:
Simon McVittie 2021-01-28 18:16:36 +00:00
parent 7ff0fb3af5
commit 82adfd7e3a
6 changed files with 447 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
glib/tests/meson.build
View File

@ -89,6 +89,7 @@ glib_tests = {
'slist' : {}, 'slist' : {},
'sort' : {}, 'sort' : {},
'spawn-multithreaded' : {}, 'spawn-multithreaded' : {},
'spawn-path-search' : {},
'spawn-singlethread' : {}, 'spawn-singlethread' : {},
'strfuncs' : {}, 'strfuncs' : {},
'string' : {}, 'string' : {},
@ -236,6 +237,20 @@ foreach test_name, extra_args : glib_tests
test(test_name, exe, env : test_env, timeout : timeout, suite : suite) test(test_name, exe, env : test_env, timeout : timeout, suite : suite)
endforeach endforeach
executable('spawn-path-search-helper', 'spawn-path-search-helper.c',
c_args : test_cargs,
dependencies : test_deps,
install_dir: installed_tests_execdir,
install: installed_tests_enabled,
)
executable('spawn-test-helper', 'spawn-test-helper.c',
c_args : test_cargs,
dependencies : test_deps,
install_dir: installed_tests_execdir,
install: installed_tests_enabled,
)
# test-spawn-echo helper binary required by the spawn tests above # test-spawn-echo helper binary required by the spawn tests above
executable('test-spawn-echo', 'test-spawn-echo.c', executable('test-spawn-echo', 'test-spawn-echo.c',
c_args : test_cargs, c_args : test_cargs,
@ -267,3 +282,5 @@ if not meson.is_cross_build() and host_system != 'windows'
) )
endif endif
endif endif
subdir('path-test-subdir')

6
glib/tests/path-test-subdir/meson.build Normal file
View File

@ -0,0 +1,6 @@
executable('spawn-test-helper', 'spawn-test-helper.c',
c_args : test_cargs,
dependencies : test_deps,
install_dir: join_paths(installed_tests_execdir, 'path-test-subdir'),
install: installed_tests_enabled,
)

7
glib/tests/path-test-subdir/spawn-test-helper.c Normal file
View File

@ -0,0 +1,7 @@
#include <stdio.h>
int main (void)
{
fprintf (stderr, "this is spawn-test-helper from path-test-subdir\n");
return 5;
}

156
glib/tests/spawn-path-search-helper.c Normal file
View File

@ -0,0 +1,156 @@
/*
* Copyright 2021 Collabora Ltd.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 2.1 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see
* <http://www.gnu.org/licenses/>.
*/
#include <errno.h>
#include <stdio.h>
#include <glib.h>
#ifdef G_OS_UNIX
#include <sys/types.h>
#include <sys/wait.h>
#endif
static void
child_setup (gpointer user_data)
{
}
typedef struct
{
int wait_status;
gboolean done;
} ChildStatus;
static ChildStatus child_status = { -1, FALSE };
static void
child_watch_cb (GPid pid,
gint status,
gpointer user_data)
{
child_status.wait_status = status;
child_status.done = TRUE;
}
int
main (int argc,
char **argv)
{
gboolean search_path = FALSE;
gboolean search_path_from_envp = FALSE;
gboolean slow_path = FALSE;
gchar *chdir_child = NULL;
gchar *set_path_in_envp = NULL;
gchar **envp = NULL;
GOptionEntry entries[] =
{
{ "chdir-child", '\0',
G_OPTION_FLAG_NONE, G_OPTION_ARG_FILENAME, &chdir_child,
"Run PROGRAM in this working directory", NULL },
{ "search-path", '\0',
G_OPTION_FLAG_NONE, G_OPTION_ARG_NONE, &search_path,
"Search PATH for PROGRAM", NULL },
{ "search-path-from-envp", '\0',
G_OPTION_FLAG_NONE, G_OPTION_ARG_NONE, &search_path_from_envp,
"Search PATH from specified environment", NULL },
{ "set-path-in-envp", '\0',
G_OPTION_FLAG_NONE, G_OPTION_ARG_FILENAME, &set_path_in_envp,
"Set PATH in specified environment to this value", "PATH", },
{ "slow-path", '\0',
G_OPTION_FLAG_NONE, G_OPTION_ARG_NONE, &slow_path,
"Use a child-setup function to avoid the posix_spawn fast path", NULL },
{ NULL }
};
GError *error = NULL;
int ret = 1;
GSpawnFlags spawn_flags = G_SPAWN_DO_NOT_REAP_CHILD;
GPid pid;
GOptionContext *context = NULL;
context = g_option_context_new ("PROGRAM [ARGS...]");
g_option_context_add_main_entries (context, entries, NULL);
if (!g_option_context_parse (context, &argc, &argv, &error))
{
ret = 2;
goto out;
}
if (argc < 2)
{
g_set_error (&error, G_OPTION_ERROR, G_OPTION_ERROR_FAILED,
"Usage: %s [OPTIONS] PROGRAM [ARGS...]", argv[0]);
ret = 2;
goto out;
}
envp = g_get_environ ();
if (set_path_in_envp != NULL)
envp = g_environ_setenv (envp, "PATH", set_path_in_envp, TRUE);
if (search_path)
spawn_flags |= G_SPAWN_SEARCH_PATH;
if (search_path_from_envp)
spawn_flags |= G_SPAWN_SEARCH_PATH_FROM_ENVP;
if (!g_spawn_async_with_pipes (chdir_child,
&argv[1],
envp,
spawn_flags,
slow_path ? child_setup : NULL,
NULL, /* user_data */
&pid,
NULL, /* stdin */
NULL, /* stdout */
NULL, /* stderr */
&error))
{
ret = 1;
goto out;
}
g_child_watch_add (pid, child_watch_cb, NULL);
while (!child_status.done)
g_main_context_iteration (NULL, TRUE);
g_spawn_close_pid (pid);
#ifdef G_OS_UNIX
if (WIFEXITED (child_status.wait_status))
ret = WEXITSTATUS (child_status.wait_status);
else
ret = 1;
#else
ret = child_status.wait_status;
#endif
out:
if (error != NULL)
fprintf (stderr, "%s\n", error->message);
g_free (set_path_in_envp);
g_free (chdir_child);
g_clear_error (&error);
g_strfreev (envp);
g_option_context_free (context);
return ret;
}

254
glib/tests/spawn-path-search.c Normal file
View File

@ -0,0 +1,254 @@
/*
* Copyright 2021 Collabora Ltd.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 2.1 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, see
* <http://www.gnu.org/licenses/>.
*/
#include <glib.h>
#ifdef G_OS_UNIX
#include <sys/types.h>
#include <sys/wait.h>
#endif
static void
test_do_not_search (void)
{
GPtrArray *argv = g_ptr_array_new_with_free_func (g_free);
gchar *here = g_test_build_filename (G_TEST_BUILT, ".", NULL);
gchar *subdir = g_test_build_filename (G_TEST_BUILT, "path-test-subdir", NULL);
gchar **envp = g_get_environ ();
gchar *out = NULL;
gchar *err = NULL;
GError *error = NULL;
int wait_status = -1;
g_test_summary ("Without G_SPAWN_SEARCH_PATH, spawn-test-helper "
"means ./spawn-test-helper.");
envp = g_environ_setenv (envp, "PATH", subdir, TRUE);
g_ptr_array_add (argv,
g_test_build_filename (G_TEST_BUILT, "spawn-path-search-helper", NULL));
g_ptr_array_add (argv, g_strdup ("--"));
g_ptr_array_add (argv, g_strdup ("spawn-test-helper"));
g_ptr_array_add (argv, NULL);
g_spawn_sync (here,
(char **) argv->pdata,
envp,
G_SPAWN_DEFAULT,
NULL, /* child setup */
NULL, /* user data */
&out,
&err,
&wait_status,
&error);
g_assert_no_error (error);
g_test_message ("%s", out);
g_test_message ("%s", err);
g_assert_nonnull (strstr (err, "this is spawn-test-helper from glib/tests"));
#ifdef G_OS_UNIX
g_assert_true (WIFEXITED (wait_status));
g_assert_cmpint (WEXITSTATUS (wait_status), ==, 0);
#endif
g_strfreev (envp);
g_free (here);
g_free (subdir);
g_free (out);
g_free (err);
g_ptr_array_unref (argv);
}
static void
test_search_path (void)
{
GPtrArray *argv = g_ptr_array_new_with_free_func (g_free);
gchar *here = g_test_build_filename (G_TEST_BUILT, ".", NULL);
gchar *subdir = g_test_build_filename (G_TEST_BUILT, "path-test-subdir", NULL);
gchar **envp = g_get_environ ();
gchar *out = NULL;
gchar *err = NULL;
GError *error = NULL;
int wait_status = -1;
g_test_summary ("With G_SPAWN_SEARCH_PATH, spawn-test-helper "
"means $PATH/spawn-test-helper.");
envp = g_environ_setenv (envp, "PATH", subdir, TRUE);
g_ptr_array_add (argv,
g_test_build_filename (G_TEST_BUILT, "spawn-path-search-helper", NULL));
g_ptr_array_add (argv, g_strdup ("--search-path"));
g_ptr_array_add (argv, g_strdup ("--"));
g_ptr_array_add (argv, g_strdup ("spawn-test-helper"));
g_ptr_array_add (argv, NULL);
g_spawn_sync (here,
(char **) argv->pdata,
envp,
G_SPAWN_DEFAULT,
NULL, /* child setup */
NULL, /* user data */
&out,
&err,
&wait_status,
&error);
g_assert_no_error (error);
g_test_message ("%s", out);
g_test_message ("%s", err);
g_assert_nonnull (strstr (err, "this is spawn-test-helper from path-test-subdir"));
#ifdef G_OS_UNIX
g_assert_true (WIFEXITED (wait_status));
g_assert_cmpint (WEXITSTATUS (wait_status), ==, 5);
#endif
g_strfreev (envp);
g_free (here);
g_free (subdir);
g_free (out);
g_free (err);
g_ptr_array_unref (argv);
}
static void
test_search_path_from_envp (void)
{
GPtrArray *argv = g_ptr_array_new_with_free_func (g_free);
gchar *here = g_test_build_filename (G_TEST_BUILT, ".", NULL);
gchar *subdir = g_test_build_filename (G_TEST_BUILT, "path-test-subdir", NULL);
gchar **envp = g_get_environ ();
gchar *out = NULL;
gchar *err = NULL;
GError *error = NULL;
int wait_status = -1;
g_test_summary ("With G_SPAWN_SEARCH_PATH_FROM_ENVP, spawn-test-helper "
"means $PATH/spawn-test-helper with $PATH from envp.");
envp = g_environ_setenv (envp, "PATH", here, TRUE);
g_ptr_array_add (argv,
g_test_build_filename (G_TEST_BUILT, "spawn-path-search-helper", NULL));
g_ptr_array_add (argv, g_strdup ("--search-path-from-envp"));
g_ptr_array_add (argv, g_strdup ("--set-path-in-envp"));
g_ptr_array_add (argv, g_strdup (subdir));
g_ptr_array_add (argv, g_strdup ("--"));
g_ptr_array_add (argv, g_strdup ("spawn-test-helper"));
g_ptr_array_add (argv, NULL);
g_spawn_sync (here,
(char **) argv->pdata,
envp,
G_SPAWN_DEFAULT,
NULL, /* child setup */
NULL, /* user data */
&out,
&err,
&wait_status,
&error);
g_assert_no_error (error);
g_test_message ("%s", out);
g_test_message ("%s", err);
g_assert_nonnull (strstr (err, "this is spawn-test-helper from path-test-subdir"));
#ifdef G_OS_UNIX
g_assert_true (WIFEXITED (wait_status));
g_assert_cmpint (WEXITSTATUS (wait_status), ==, 5);
#endif
g_strfreev (envp);
g_free (here);
g_free (subdir);
g_free (out);
g_free (err);
g_ptr_array_unref (argv);
}
static void
test_search_path_ambiguous (void)
{
GPtrArray *argv = g_ptr_array_new_with_free_func (g_free);
gchar *here = g_test_build_filename (G_TEST_BUILT, ".", NULL);
gchar *subdir = g_test_build_filename (G_TEST_BUILT, "path-test-subdir", NULL);
gchar **envp = g_get_environ ();
gchar *out = NULL;
gchar *err = NULL;
GError *error = NULL;
int wait_status = -1;
g_test_summary ("With G_SPAWN_SEARCH_PATH and G_SPAWN_SEARCH_PATH_FROM_ENVP, "
"the latter wins.");
envp = g_environ_setenv (envp, "PATH", here, TRUE);
g_ptr_array_add (argv,
g_test_build_filename (G_TEST_BUILT, "spawn-path-search-helper", NULL));
g_ptr_array_add (argv, g_strdup ("--search-path"));
g_ptr_array_add (argv, g_strdup ("--search-path-from-envp"));
g_ptr_array_add (argv, g_strdup ("--set-path-in-envp"));
g_ptr_array_add (argv, g_strdup (subdir));
g_ptr_array_add (argv, g_strdup ("--"));
g_ptr_array_add (argv, g_strdup ("spawn-test-helper"));
g_ptr_array_add (argv, NULL);
g_spawn_sync (here,
(char **) argv->pdata,
envp,
G_SPAWN_DEFAULT,
NULL, /* child setup */
NULL, /* user data */
&out,
&err,
&wait_status,
&error);
g_assert_no_error (error);
g_test_message ("%s", out);
g_test_message ("%s", err);
g_assert_nonnull (strstr (err, "this is spawn-test-helper from path-test-subdir"));
#ifdef G_OS_UNIX
g_assert_true (WIFEXITED (wait_status));
g_assert_cmpint (WEXITSTATUS (wait_status), ==, 5);
#endif
g_strfreev (envp);
g_free (here);
g_free (subdir);
g_free (out);
g_free (err);
g_ptr_array_unref (argv);
}
int
main (int argc,
char **argv)
{
g_test_init (&argc, &argv, NULL);
g_test_add_func ("/spawn/do-not-search", test_do_not_search);
g_test_add_func ("/spawn/search-path", test_search_path);
g_test_add_func ("/spawn/search-path-from-envp", test_search_path_from_envp);
g_test_add_func ("/spawn/search-path-ambiguous", test_search_path_ambiguous);
return g_test_run ();
}

7
glib/tests/spawn-test-helper.c Normal file
View File

@ -0,0 +1,7 @@
#include <stdio.h>
int main (void)
{
fprintf (stderr, "this is spawn-test-helper from glib/tests\n");
return 0;
}