From b766c394569cce356fceb63d83da79581c0997b9 Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Sun, 4 Sep 2005 11:10:37 +0000 Subject: General cleanup of command line parsing to allow "busybox" to work as a prefix. (I.E. any argv[0] that starts with "busybox" winds up in busybox_main().) Added testing/busybox.tests which tests the following permutations: ./busybox ./busybox-suffix ./busybox cat ./busybox-suffix cat ./busybox --help ./busybox-suffix --help ./busybox --help cat ./busybox-suffix --help cat ./busybox --help unknown ./busybox-suffix --help unknown ./unknown Also repair the test suite so ./runtest calls the ".tests" scripts properly. Note: you can now go "busybox busybox busbox ls -l" and it'll take it. The new code is pretty generic. I can block that if anybody can come up with a good reason to... --- applets/applets.c | 52 ++++++++++++++-------------------------------------- 1 file changed, 14 insertions(+), 38 deletions(-) (limited to 'applets/applets.c') diff --git a/applets/applets.c b/applets/applets.c index ce9ecbbc5..21b0c3dbc 100644 --- a/applets/applets.c +++ b/applets/applets.c @@ -90,8 +90,7 @@ static int suid_cfg_readable; -extern void -bb_show_usage (void) +extern void bb_show_usage (void) { const char *format_string; const char *usage_string = usage_messages; @@ -112,8 +111,7 @@ bb_show_usage (void) exit (EXIT_FAILURE); } -static int -applet_name_compare (const void *x, const void *y) +static int applet_name_compare (const void *x, const void *y) { const char *name = x; const struct BB_applet *applet = y; @@ -123,47 +121,25 @@ applet_name_compare (const void *x, const void *y) extern const size_t NUM_APPLETS; -struct BB_applet * -find_applet_by_name (const char *name) +struct BB_applet *find_applet_by_name (const char *name) { return bsearch (name, applets, NUM_APPLETS, sizeof (struct BB_applet), applet_name_compare); } -void -run_applet_by_name (const char *name, int argc, char **argv) +void run_applet_by_name (const char *name, int argc, char **argv) { - static int recurse_level = 0; - extern int been_there_done_that; /* From busybox.c */ - -#ifdef CONFIG_FEATURE_SUID_CONFIG - if (recurse_level == 0) - parse_config_file (); -#endif - - recurse_level++; - /* Do a binary search to find the applet entry given the name. */ - if ((applet_using = find_applet_by_name (name)) != NULL) { - bb_applet_name = applet_using->name; - if (argv[1] && strcmp (argv[1], "--help") == 0) { - if (strcmp (applet_using->name, "busybox") == 0) { - if (argv[2]) - applet_using = find_applet_by_name (argv[2]); - else - applet_using = NULL; - } - if (applet_using) - bb_show_usage (); - been_there_done_that = 1; - busybox_main (0, NULL); + if(ENABLE_FEATURE_SUID_CONFIG) parse_config_file (); + + if(!strncmp(name, "busybox", 7)) busybox_main(argc, argv); + /* Do a binary search to find the applet entry given the name. */ + applet_using = find_applet_by_name(name); + if(applet_using) { + bb_applet_name = applet_using->name; + if(argc==2 && !strcmp(argv[1], "--help")) bb_show_usage (); + if(ENABLE_FEATURE_SUID) check_suid (applet_using); + exit ((*(applet_using->main)) (argc, argv)); } -#ifdef CONFIG_FEATURE_SUID - check_suid (applet_using); -#endif - - exit ((*(applet_using->main)) (argc, argv)); - } - recurse_level--; } -- cgit v1.2.3