brl/gnome-shell
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

run-test: Check argc before using argv

Browse Source 
We skip the filename argument when defining ARGV before actually
checking that it was provided.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3599>
This commit is contained in:
Florian Müllner 2025-01-16 15:25:20 +01:00 committed by Marge Bot
parent e7691bd3df
commit 699c7c1b2c
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/run-js-test.c
View File

@ -64,6 +64,12 @@ main (int argc, char **argv)
g_irepository_prepend_search_path (MUTTER_TYPELIB_DIR);
g_irepository_prepend_search_path (SHELL_TYPELIB_DIR);
if (argc < 2)
{
g_printerr ("Missing filename\n");
exit (1);
}
/* prepare command line arguments */
if (!gjs_context_define_string_array (js_context, "ARGV",
argc - 2, (const char**)argv + 2,
@ -73,12 +79,6 @@ main (int argc, char **argv)
exit (1);
}
if (argc < 2)
{
g_printerr ("Missing filename\n");
exit (1);
}
filename = argv[1];
title = g_filename_display_basename (filename);
g_set_prgname (title);