From 038fe447138aa04d1295cde2e597ec3497ce9214 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 29 Mar 2009 02:23:16 +0000 Subject: sysctl: move code around to get rid of forward references. (incidentally it helps gcc to make code smaller) --- procps/sysctl.c | 209 +++++++++++++++++++++++++++----------------------------- 1 file changed, 101 insertions(+), 108 deletions(-) (limited to 'procps') diff --git a/procps/sysctl.c b/procps/sysctl.c index dce264dac..d0ff20ec7 100644 --- a/procps/sysctl.c +++ b/procps/sysctl.c @@ -7,20 +7,12 @@ * Licensed under GPLv2 or later, see file LICENSE in this tarball for details. * * Changelog: - * v1.01: - * - added -p to preload values from a file - * v1.01.1 - * - busybox applet aware by - * + * v1.01 - added -p to preload values from a file + * v1.01.1 - busybox applet aware by */ #include "libbb.h" -static int sysctl_act_on_setting(char *setting); -static int sysctl_display_all(const char *path); -static int sysctl_handle_preload_file(const char *filename); -static void sysctl_dots_to_slashes(char *name); - enum { FLAG_SHOW_KEYS = 1 << 0, FLAG_SHOW_KEY_ERRORS = 1 << 1, @@ -29,67 +21,52 @@ enum { FLAG_PRELOAD_FILE = 1 << 4, FLAG_WRITE = 1 << 5, }; +#define OPTION_STR "neAapw" -int sysctl_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; -int sysctl_main(int argc UNUSED_PARAM, char **argv) -{ - int retval; - int opt; - - opt = getopt32(argv, "+neAapw"); /* '+' - stop on first non-option */ - argv += optind; - opt ^= (FLAG_SHOW_KEYS | FLAG_SHOW_KEY_ERRORS); - option_mask32 = opt; - - if (opt & FLAG_PRELOAD_FILE) { - option_mask32 |= FLAG_WRITE; - /* xchdir("/proc/sys") is inside */ - return sysctl_handle_preload_file(*argv ? *argv : "/etc/sysctl.conf"); - } - xchdir("/proc/sys"); - /* xchroot(".") - if you are paranoid */ - if (opt & (FLAG_TABLE_FORMAT | FLAG_SHOW_ALL)) { - return sysctl_display_all("."); - } - - retval = 0; - while (*argv) { - sysctl_dots_to_slashes(*argv); - retval |= sysctl_display_all(*argv); - argv++; - } - - return retval; -} - -/* Set sysctl's from a conf file. Format example: - * # Controls IP packet forwarding - * net.ipv4.ip_forward = 0 - */ -static int sysctl_handle_preload_file(const char *filename) +static void sysctl_dots_to_slashes(char *name) { - char *token[2]; - parser_t *parser; + char *cptr, *last_good, *end; - parser = config_open(filename); - /* Must do it _after_ config_open(): */ - xchdir("/proc/sys"); - /* xchroot(".") - if you are paranoid */ + /* Convert minimum number of '.' to '/' so that + * we end up with existing file's name. + * + * Example from bug 3894: + * net.ipv4.conf.eth0.100.mc_forwarding -> + * net/ipv4/conf/eth0.100/mc_forwarding + * NB: net/ipv4/conf/eth0/mc_forwarding *also exists*, + * therefore we must start from the end, and if + * we replaced even one . -> /, start over again, + * but never replace dots before the position + * where last replacement occurred. + * + * Another bug we later had is that + * net.ipv4.conf.eth0.100 + * (without .mc_forwarding) was mishandled. + * + * To set up testing: modprobe 8021q; vconfig add eth0 100 + */ + end = name + strlen(name); + last_good = name - 1; + *end = '.'; /* trick the loop into trying full name too */ -//TODO: ';' is comment char too -//TODO: comment may be only at line start. "var=1 #abc" - "1 #abc" is the value -// (but _whitespace_ from ends should be trimmed first (and we do it right)) -//TODO: "var==1" is mishandled (must use "=1" as a value, but uses "1") - while (config_read(parser, token, 2, 2, "# \t=", PARSE_NORMAL)) { - sysctl_dots_to_slashes(token[0]); - /* Save ~4 bytes by using parser internals */ - /* parser->line is big enough for sprintf */ - sprintf(parser->line, "%s=%s", token[0], token[1]); - sysctl_display_all(parser->line); + again: + cptr = end; + while (cptr > last_good) { + if (*cptr == '.') { + *cptr = '\0'; + //bb_error_msg("trying:'%s'", name); + if (access(name, F_OK) == 0) { + if (cptr != end) /* prevent trailing '/' */ + *cptr = '/'; + //bb_error_msg("replaced:'%s'", name); + last_good = cptr; + goto again; + } + *cptr = '.'; + } + cptr--; } - if (ENABLE_FEATURE_CLEAN_UP) - config_close(parser); - return 0; + *end = '\0'; } static int sysctl_act_on_setting(char *setting) @@ -186,7 +163,7 @@ static int sysctl_act_on_setting(char *setting) return retval; } -static int sysctl_display_all(const char *path) +static int sysctl_act_recursive(const char *path) { DIR *dirp; struct stat buf; @@ -204,7 +181,7 @@ static int sysctl_display_all(const char *path) if (next == NULL) continue; /* d_name is "." or ".." */ /* if path was ".", drop "./" prefix: */ - retval |= sysctl_display_all((next[0] == '.' && next[1] == '/') ? + retval |= sysctl_act_recursive((next[0] == '.' && next[1] == '/') ? next + 2 : next); free(next); } @@ -218,48 +195,64 @@ static int sysctl_display_all(const char *path) return retval; } -static void sysctl_dots_to_slashes(char *name) +/* Set sysctl's from a conf file. Format example: + * # Controls IP packet forwarding + * net.ipv4.ip_forward = 0 + */ +static int sysctl_handle_preload_file(const char *filename) { - char *cptr, *last_good, *end; + char *token[2]; + parser_t *parser; - /* Convert minimum number of '.' to '/' so that - * we end up with existing file's name. - * - * Example from bug 3894: - * net.ipv4.conf.eth0.100.mc_forwarding -> - * net/ipv4/conf/eth0.100/mc_forwarding - * NB: net/ipv4/conf/eth0/mc_forwarding *also exists*, - * therefore we must start from the end, and if - * we replaced even one . -> /, start over again, - * but never replace dots before the position - * where last replacement occurred. - * - * Another bug we later had is that - * net.ipv4.conf.eth0.100 - * (without .mc_forwarding) was mishandled. - * - * To set up testing: modprobe 8021q; vconfig add eth0 100 - */ - end = name + strlen(name); - last_good = name - 1; - *end = '.'; /* trick the loop into trying full name too */ + parser = config_open(filename); + /* Must do it _after_ config_open(): */ + xchdir("/proc/sys"); + /* xchroot(".") - if you are paranoid */ - again: - cptr = end; - while (cptr > last_good) { - if (*cptr == '.') { - *cptr = '\0'; - //bb_error_msg("trying:'%s'", name); - if (access(name, F_OK) == 0) { - if (cptr != end) /* prevent trailing '/' */ - *cptr = '/'; - //bb_error_msg("replaced:'%s'", name); - last_good = cptr; - goto again; - } - *cptr = '.'; - } - cptr--; +//TODO: ';' is comment char too +//TODO: comment may be only at line start. "var=1 #abc" - "1 #abc" is the value +// (but _whitespace_ from ends should be trimmed first (and we do it right)) +//TODO: "var==1" is mishandled (must use "=1" as a value, but uses "1") + while (config_read(parser, token, 2, 2, "# \t=", PARSE_NORMAL)) { + sysctl_dots_to_slashes(token[0]); + /* Save ~4 bytes by using parser internals */ + /* parser->line is big enough for sprintf */ + sprintf(parser->line, "%s=%s", token[0], token[1]); + sysctl_act_recursive(parser->line); } - *end = '\0'; + if (ENABLE_FEATURE_CLEAN_UP) + config_close(parser); + return 0; +} + +int sysctl_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; +int sysctl_main(int argc UNUSED_PARAM, char **argv) +{ + int retval; + int opt; + + opt = getopt32(argv, "+" OPTION_STR); /* '+' - stop on first non-option */ + argv += optind; + opt ^= (FLAG_SHOW_KEYS | FLAG_SHOW_KEY_ERRORS); + option_mask32 = opt; + + if (opt & FLAG_PRELOAD_FILE) { + option_mask32 |= FLAG_WRITE; + /* xchdir("/proc/sys") is inside */ + return sysctl_handle_preload_file(*argv ? *argv : "/etc/sysctl.conf"); + } + xchdir("/proc/sys"); + /* xchroot(".") - if you are paranoid */ + if (opt & (FLAG_TABLE_FORMAT | FLAG_SHOW_ALL)) { + return sysctl_act_recursive("."); + } + + retval = 0; + while (*argv) { + sysctl_dots_to_slashes(*argv); + retval |= sysctl_act_recursive(*argv); + argv++; + } + + return retval; } -- cgit v1.2.3