diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-03-26 21:25:33 +0400 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-03-26 21:25:33 +0400 |
commit | afa7023b465d2abc5a85ecba0917cae974257eaf (patch) | |
tree | 8c8cc570f93e83a6a38f2b184d83cd7f2fa9985e /networking | |
parent | 4e1361a4819ad0fcf76e161bfdcdc8bc29c872ff (diff) |
zcip: make it work on NOMMU (+ improve NOMMU support machinery)
fsck: fix bad English in a comment
Diffstat (limited to 'networking')
-rw-r--r-- | networking/zcip.c | 146 |
1 files changed, 73 insertions, 73 deletions
diff --git a/networking/zcip.c b/networking/zcip.c index c0cf665dd..6035b91e1 100644 --- a/networking/zcip.c +++ b/networking/zcip.c @@ -70,10 +70,6 @@ enum { #define VDBG(fmt,args...) \ do { } while (0) -static unsigned opts; -#define FOREGROUND (opts & 1) -#define QUIT (opts & 2) - /** * Pick a random link local IP address on 169.254/16, except that * the first and last 256 addresses are reserved. @@ -128,49 +124,30 @@ static void arp(int fd, struct sockaddr *saddr, int op, } /** - * Run a script. + * Run a script. argv[2] is already NULL. */ -static int run(const char *script, const char *arg, const char *intf, struct in_addr *ip) +static int run(char *argv[3], const char *intf, struct in_addr *ip) { - int pid, status; - const char *why; + int status; - if(1) { //always true: if (script != NULL) - VDBG("%s run %s %s\n", intf, script, arg); - if (ip != NULL) { - char *addr = inet_ntoa(*ip); - setenv("ip", addr, 1); - bb_info_msg("%s %s %s", arg, intf, addr); - } + VDBG("%s run %s %s\n", intf, argv[0], argv[1]); - pid = vfork(); - if (pid < 0) { // error - why = "vfork"; - goto bad; - } else if (pid == 0) { // child - execl(script, script, arg, NULL); - bb_perror_msg("execl"); - _exit(EXIT_FAILURE); - } + if (ip) { + char *addr = inet_ntoa(*ip); + setenv("ip", addr, 1); + bb_info_msg("%s %s %s", argv[1], intf, addr); + } - if (waitpid(pid, &status, 0) <= 0) { - why = "waitpid"; - goto bad; - } - if (WEXITSTATUS(status) != 0) { - bb_error_msg("script %s failed, exit=%d", - script, WEXITSTATUS(status)); - return -errno; - } + status = wait4pid(spawn(argv)); + if (status < 0) { + bb_perror_msg("%s %s", argv[1], intf); + return -errno; } - return 0; -bad: - status = -errno; - bb_perror_msg("%s %s, %s", arg, intf, why); + if (status != 0) + bb_error_msg("script %s %s failed, exitcode=%d", argv[0], argv[1], status); return status; } - /** * Return milliseconds of random delay, up to "secs" seconds. */ @@ -182,43 +159,58 @@ static unsigned ATTRIBUTE_ALWAYS_INLINE ms_rdelay(unsigned secs) /** * main program */ - -/* Used to be auto variables on main() stack, but - * most of them were zero-inited. Moving them to bss - * is more space-efficient. - */ -static const struct in_addr null_ip; // = { 0 }; -static const struct ether_addr null_addr; // = { {0, 0, 0, 0, 0, 0} }; - -static struct sockaddr saddr; // memset(0); -static struct in_addr ip; // = { 0 }; -static struct ifreq ifr; //memset(0); - -static char *intf; // = NULL; -static char *script; // = NULL; -static suseconds_t timeout; // = 0; // milliseconds -static unsigned conflicts; // = 0; -static unsigned nprobes; // = 0; -static unsigned nclaims; // = 0; -static int ready; // = 0; -static int verbose; // = 0; -static int state = PROBE; - int zcip_main(int argc, char *argv[]); int zcip_main(int argc, char *argv[]) { + int state = PROBE; struct ether_addr eth_addr; const char *why; int fd; + char *r_opt; + unsigned opts; + + /* Ugly trick, but I want these zeroed in one go */ + struct { + const struct in_addr null_ip; + const struct ether_addr null_addr; + struct sockaddr saddr; + struct in_addr ip; + struct ifreq ifr; + char *intf; + char *script_av[3]; + suseconds_t timeout; // milliseconds + unsigned conflicts; + unsigned nprobes; + unsigned nclaims; + int ready; + int verbose; + } L; +#define null_ip (L.null_ip ) +#define null_addr (L.null_addr) +#define saddr (L.saddr ) +#define ip (L.ip ) +#define ifr (L.ifr ) +#define intf (L.intf ) +#define script_av (L.script_av) +#define timeout (L.timeout ) +#define conflicts (L.conflicts) +#define nprobes (L.nprobes ) +#define nclaims (L.nclaims ) +#define ready (L.ready ) +#define verbose (L.verbose ) + + memset(&L, 0, sizeof(L)); +#define FOREGROUND (opts & 1) +#define QUIT (opts & 2) // parse commandline: prog [options] ifname script - char *r_opt; - opt_complementary = "vv:vf"; // -v accumulates and implies -f + // exactly 2 args; -v accumulates and implies -f + opt_complementary = "=2:vv:vf"; opts = getopt32(argc, argv, "fqr:v", &r_opt, &verbose); if (!FOREGROUND) { /* Do it early, before all bb_xx_msg calls */ - logmode = LOGMODE_SYSLOG; openlog(applet_name, 0, LOG_DAEMON); + logmode |= LOGMODE_SYSLOG; } if (opts & 4) { // -r n.n.n.n if (inet_aton(r_opt, &ip) == 0 @@ -227,16 +219,21 @@ int zcip_main(int argc, char *argv[]) bb_error_msg_and_die("invalid link address"); } } + // On NOMMU reexec early (or else we will rerun things twice) +#ifdef BB_NOMMU + if (!FOREGROUND) + bb_daemonize_or_rexec(DAEMON_CHDIR_ROOT, argv); +#endif argc -= optind; argv += optind; - if (argc != 2) - bb_show_usage(); + intf = argv[0]; - script = argv[1]; + script_av[0] = argv[1]; setenv("interface", intf, 1); // initialize the interface (modprobe, ifup, etc) - if (run(script, "init", intf, NULL) < 0) + script_av[1] = (char*)"init"; + if (run(script_av, intf, NULL)) return EXIT_FAILURE; // initialize saddr @@ -271,8 +268,9 @@ int zcip_main(int argc, char *argv[]) // daemonize now; don't delay system startup if (!FOREGROUND) { -//NOMMU +#ifndef BB_NOMMU bb_daemonize(DAEMON_CHDIR_ROOT); +#endif bb_info_msg("start, interface %s", intf); } @@ -375,7 +373,8 @@ int zcip_main(int argc, char *argv[]) state = MONITOR; // link is ok to use earlier // FIXME update filters - run(script, "config", intf, &ip); + script_av[1] = (char*)"config"; + run(script_av, intf, &ip); ready = 1; conflicts = 0; timeout = -1; // Never timeout in the monitor state. @@ -429,8 +428,8 @@ int zcip_main(int argc, char *argv[]) // this shouldn't necessarily exit. bb_error_msg("%s: poll error", intf); if (ready) { - run(script, "deconfig", - intf, &ip); + script_av[1] = (char*)"deconfig"; + run(script_av, intf, &ip); } return EXIT_FAILURE; } @@ -516,7 +515,8 @@ int zcip_main(int argc, char *argv[]) state = PROBE; VDBG("defend conflict -- starting over\n"); ready = 0; - run(script, "deconfig", intf, &ip); + script_av[1] = (char*)"deconfig"; + run(script_av, intf, &ip); // restart the whole protocol pick(&ip); @@ -542,7 +542,7 @@ int zcip_main(int argc, char *argv[]) goto bad; } // switch poll } -bad: + bad: bb_perror_msg("%s, %s", intf, why); return EXIT_FAILURE; } |