From c12f53090bd41dbb87279083bc442769cb0610f0 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Fri, 6 Oct 2006 09:49:47 +0000 Subject: dnsd fix; option_mask32 added. dnsd needs more love. --- include/libbb.h | 3 +- libbb/getopt32.c | 4 ++ modutils/insmod.c | 17 +++---- networking/dnsd.c | 138 ++++++++++++++++++++++++++------------------------ networking/ifupdown.c | 17 +++---- procps/top.c | 11 ++-- runit/chpst.c | 73 +++++++++++++------------- sysklogd/syslogd.c | 39 +++++++------- 8 files changed, 154 insertions(+), 148 deletions(-) diff --git a/include/libbb.h b/include/libbb.h index adfeca590..84c8af4b6 100644 --- a/include/libbb.h +++ b/include/libbb.h @@ -206,9 +206,10 @@ extern void xsetgid(gid_t gid); extern void xsetuid(uid_t uid); extern off_t fdlength(int fd); -#define BB_GETOPT_ERROR 0x80000000UL +enum { BB_GETOPT_ERROR = 0x80000000 }; extern const char *opt_complementary; extern const struct option *applet_long_options; +extern uint32_t option_mask32; extern uint32_t getopt32(int argc, char **argv, const char *applet_opts, ...); extern int bb_vfprintf(FILE * __restrict stream, const char * __restrict format, diff --git a/libbb/getopt32.c b/libbb/getopt32.c index e08496578..2f2f0b9e9 100644 --- a/libbb/getopt32.c +++ b/libbb/getopt32.c @@ -302,6 +302,8 @@ static const struct option bb_default_long_options[] = { const struct option *applet_long_options = bb_default_long_options; #endif +uint32_t option_mask32; + uint32_t getopt32(int argc, char **argv, const char *applet_opts, ...) { @@ -512,5 +514,7 @@ loop_arg_is_opt: argc -= optind; if (argc < min_arg || (max_arg >= 0 && argc > max_arg)) bb_show_usage(); + + option_mask32 = flags; return flags; } diff --git a/modutils/insmod.c b/modutils/insmod.c index 0554def06..348feed8d 100644 --- a/modutils/insmod.c +++ b/modutils/insmod.c @@ -693,7 +693,6 @@ enum { STRVERSIONLEN = 32 }; /*======================================================================*/ -static unsigned option_mask; #define OPTION_STR "sLo:fkvqx" USE_FEATURE_INSMOD_LOAD_MAP("m") enum { OPT_s = 0x1, // -s /* log to syslog */ @@ -713,13 +712,13 @@ enum { OPT_x = 0x80, // -x /* do not export externs */ OPT_m = 0x100, // -m /* print module load map */ }; -#define flag_force_load (option_mask & OPT_f) -#define flag_autoclean (option_mask & OPT_k) -#define flag_verbose (option_mask & OPT_v) -#define flag_quiet (option_mask & OPT_q) -#define flag_noexport (option_mask & OPT_x) +#define flag_force_load (option_mask32 & OPT_f) +#define flag_autoclean (option_mask32 & OPT_k) +#define flag_verbose (option_mask32 & OPT_v) +#define flag_quiet (option_mask32 & OPT_q) +#define flag_noexport (option_mask32 & OPT_x) #ifdef CONFIG_FEATURE_INSMOD_LOAD_MAP -#define flag_print_load_map (option_mask & OPT_m) +#define flag_print_load_map (option_mask32 & OPT_m) #else #define flag_print_load_map 0 #endif @@ -3983,8 +3982,8 @@ int insmod_main( int argc, char **argv) struct utsname myuname; /* Parse any options */ - option_mask = getopt32(argc, argv, OPTION_STR, &opt_o); - if (option_mask & OPT_o) { // -o /* name the output module */ + getopt32(argc, argv, OPTION_STR, &opt_o); + if (option_mask32 & OPT_o) { // -o /* name the output module */ free(m_name); m_name = xstrdup(opt_o); } diff --git a/networking/dnsd.c b/networking/dnsd.c index ab3704707..35f6c1096 100644 --- a/networking/dnsd.c +++ b/networking/dnsd.c @@ -23,6 +23,12 @@ static char *fileconf = "/etc/dnsd.conf"; #define LOCK_FILE "/var/run/dnsd.lock" #define LOG_FILE "/var/log/dnsd.log" +// Must matct getopt32 call +#define OPT_daemon (option_mask32 & 0x10) +#define OPT_verbose (option_mask32 & 0x20) + +//#define DEBUG 1 + enum { MAX_HOST_LEN = 16, // longest host name allowed is 15 IP_STRING_LEN = 18, // .xxx.xxx.xxx.xxx\0 @@ -70,7 +76,8 @@ struct dns_entry { // element of known name, ip address and reversed ip address }; static struct dns_entry *dnsentry = NULL; -static int daemonmode = 0; +// FIXME! unused! :( +static int daemonmode; static uint32_t ttl = DEFAULT_TTL; /* @@ -79,7 +86,7 @@ static uint32_t ttl = DEFAULT_TTL; static void convname(char *a, uint8_t *q) { int i = (q[0] == '.') ? 0 : 1; - for(; i < MAX_HOST_LEN-1 && *q; i++, q++) + for (; i < MAX_HOST_LEN-1 && *q; i++, q++) a[i] = tolower(*q); a[0] = i - 1; a[i] = 0; @@ -91,9 +98,10 @@ static void convname(char *a, uint8_t *q) static void undot(uint8_t * rip) { int i = 0, s = 0; - while(rip[i]) i++; - for(--i; i >= 0; i--) { - if(rip[i] == '.') { + while (rip[i]) + i++; + for (--i; i >= 0; i--) { + if (rip[i] == '.') { rip[i] = s; s = 0; } else s++; @@ -124,41 +132,42 @@ static void log_message(char *filename, char *message) * converting to a length/string substring for that label. */ -static int getfileentry(FILE * fp, struct dns_entry *s, int verb) +static int getfileentry(FILE * fp, struct dns_entry *s) { unsigned int a,b,c,d; char *r, *name; -restart: - if(!(r = bb_get_line_from_file(fp))) + restart: + r = bb_get_line_from_file(fp); + if (!r) return -1; - while(*r == ' ' || *r == '\t') { + while (*r == ' ' || *r == '\t') { r++; - if(!*r || *r == '#' || *r == '\n') + if (!*r || *r == '#' || *r == '\n') goto restart; /* skipping empty/blank and commented lines */ } name = r; - while(*r != ' ' && *r != '\t') + while (*r != ' ' && *r != '\t') r++; *r++ = 0; - if(sscanf(r,"%u.%u.%u.%u",&a,&b,&c,&d) != 4) - goto restart; /* skipping wrong lines */ + if (sscanf(r, "%u.%u.%u.%u", &a, &b, &c, &d) != 4) + goto restart; /* skipping wrong lines */ - sprintf(s->ip,"%u.%u.%u.%u",a,b,c,d); - sprintf(s->rip,".%u.%u.%u.%u",d,c,b,a); + sprintf(s->ip, "%u.%u.%u.%u", a, b, c, d); + sprintf(s->rip, ".%u.%u.%u.%u", d, c, b, a); undot((uint8_t*)s->rip); convname(s->name,(uint8_t*)name); - if(verb) - fprintf(stderr,"\tname:%s, ip:%s\n",&(s->name[1]),s->ip); + if (OPT_verbose) + fprintf(stderr, "\tname:%s, ip:%s\n", &(s->name[1]),s->ip); - return 0; /* warningkiller */ + return 0; } /* * Read hostname/IP records from file */ -static void dnsentryinit(int verb) +static void dnsentryinit(void) { FILE *fp; struct dns_entry *m, *prev; @@ -170,7 +179,7 @@ static void dnsentryinit(int verb) m = xmalloc(sizeof(struct dns_entry)); m->next = NULL; - if (getfileentry(fp, m, verb)) + if (getfileentry(fp, m)) break; if (prev == NULL) @@ -224,28 +233,32 @@ static int table_lookup(uint16_t type, uint8_t * as, uint8_t * qs) char *p,*q; q = (char *)&(qs[1]); p = &(d->name[1]); - fprintf(stderr, "\ntest: %d <%s> <%s> %d", strlen(p), p, q, strlen(q)); + fprintf(stderr, "\n%s: %d/%d p:%s q:%s %d", + __FUNCTION__, strlen(p), (int)(d->name[0]), p, q, strlen(q)); #endif if (type == REQ_A) { /* search by host name */ - for(i = 1; i <= (int)(d->name[0]); i++) - if(tolower(qs[i]) != d->name[i]) - continue; + for (i = 1; i <= (int)(d->name[0]); i++) + if (tolower(qs[i]) != d->name[i]) + break; + if (i > (int)(d->name[0])) { #ifdef DEBUG - fprintf(stderr, " OK"); + fprintf(stderr, " OK"); #endif - strcpy((char *)as, d->ip); + strcpy((char *)as, d->ip); #ifdef DEBUG - fprintf(stderr, " %s ", as); + fprintf(stderr, " as:%s\n", as); #endif return 0; - } - else if (type == REQ_PTR) { /* search by IP-address */ + } + } else + if (type == REQ_PTR) { /* search by IP-address */ if (!strncmp((char*)&d->rip[1], (char*)&qs[1], strlen(d->rip)-1)) { strcpy((char *)as, d->name); return 0; } } - } while ((d = d->next) != NULL); + d = d->next; + } while (d); return -1; } @@ -253,7 +266,7 @@ static int table_lookup(uint16_t type, uint8_t * as, uint8_t * qs) /* * Decode message and generate answer */ -#define eret(s) do { fprintf (stderr, "%s\n", s); return -1; } while (0) +#define eret(s) do { fputs(s, stderr); return -1; } while (0) static int process_packet(uint8_t * buf) { struct dns_head *head; @@ -269,13 +282,13 @@ static int process_packet(uint8_t * buf) head = (struct dns_head *)buf; if (head->nquer == 0) - eret("no queries"); + eret("no queries\n"); - if ((head->flags & 0x8000)) - eret("ignoring response packet"); + if (head->flags & 0x8000) + eret("ignoring response packet\n"); from = (void *)&head[1]; // start of query string - next = answb = from + strlen((char *)&head[1]) + 1 + sizeof(struct dns_prop); // where to append answer block + next = answb = from + strlen((char *)from) + 1 + sizeof(struct dns_prop); // where to append answer block outr.rlen = 0; // may change later outr.r = NULL; @@ -299,9 +312,8 @@ static int process_packet(uint8_t * buf) goto empty_packet; // We have a standard query - - log_message(LOG_FILE, (char *)head); - lookup_result = table_lookup(type, answstr, (uint8_t*)(&head[1])); + log_message(LOG_FILE, (char *)from); + lookup_result = table_lookup(type, answstr, (uint8_t*)from); if (lookup_result != 0) { outr.flags = 3 | 0x0400; //name do not exist and auth goto empty_packet; @@ -335,7 +347,7 @@ static int process_packet(uint8_t * buf) memcpy(next, (void *)answstr, outr.rlen); next += outr.rlen; - empty_packet: + empty_packet: flags = ntohs(head->flags); // clear rcode and RA, set responsebit and our new flags @@ -358,35 +370,31 @@ static void interrupt(int x) exit(2); } -#define is_daemon() (flags&16) -#define is_verbose() (flags&32) -//#define DEBUG 1 - int dnsd_main(int argc, char **argv) { int udps; uint16_t port = 53; uint8_t buf[MAX_PACK_LEN]; - unsigned long flags = 0; char *listen_interface = "0.0.0.0"; - char *sttl=NULL, *sport=NULL; + char *sttl, *sport; - if(argc > 1) - flags = getopt32(argc, argv, "i:c:t:p:dv", &listen_interface, &fileconf, &sttl, &sport); - if(sttl) - if(!(ttl = atol(sttl))) + getopt32(argc, argv, "i:c:t:p:dv", &listen_interface, &fileconf, &sttl, &sport); + //if (option_mask32 & 0x1) // -i + //if (option_mask32 & 0x2) // -c + if (option_mask32 & 0x4) // -t + if (!(ttl = atol(sttl))) bb_show_usage(); - if(sport) - if(!(port = atol(sport))) + if (option_mask32 & 0x8) // -p + if (!(port = atol(sport))) bb_show_usage(); - if(is_verbose()) { - fprintf(stderr,"listen_interface: %s\n", listen_interface); - fprintf(stderr,"ttl: %d, port: %d\n", ttl, port); - fprintf(stderr,"fileconf: %s\n", fileconf); + if (OPT_verbose) { + bb_info_msg("listen_interface: %s", listen_interface); + bb_info_msg("ttl: %d, port: %d", ttl, port); + bb_info_msg("fileconf: %s", fileconf); } - if(is_daemon()) + if (OPT_daemon) #ifdef BB_NOMMU /* reexec for vfork() do continue parent */ vfork_daemon_rexec(1, 0, argc, argv, "-d"); @@ -394,7 +402,7 @@ int dnsd_main(int argc, char **argv) xdaemon(1, 0); #endif - dnsentryinit(is_verbose()); + dnsentryinit(); signal(SIGINT, interrupt); signal(SIGPIPE, SIG_IGN); @@ -417,20 +425,20 @@ int dnsd_main(int argc, char **argv) FD_ZERO(&fdset); FD_SET(udps, &fdset); // Block until a message arrives - if((r = select(udps + 1, &fdset, NULL, NULL, NULL)) < 0) + r = select(udps + 1, &fdset, NULL, NULL, NULL); + if (r < 0) bb_perror_msg_and_die("select error"); - else - if(r == 0) + if (r == 0) bb_perror_msg_and_die("select spurious return"); - /* Can this test ever be false? */ + /* Can this test ever be false? - yes */ if (FD_ISSET(udps, &fdset)) { struct sockaddr_in from; int fromlen = sizeof(from); r = recvfrom(udps, buf, sizeof(buf), 0, (struct sockaddr *)&from, (void *)&fromlen); - if(is_verbose()) + if (OPT_verbose) fprintf(stderr, "\n--- Got UDP "); log_message(LOG_FILE, "\n--- Got UDP "); @@ -445,9 +453,7 @@ int dnsd_main(int argc, char **argv) r, 0, (struct sockaddr *)&from, fromlen); } - } // end if - } // end while + } // end if + } // end while return 0; } - - diff --git a/networking/ifupdown.c b/networking/ifupdown.c index 6334cbc43..f572b487d 100644 --- a/networking/ifupdown.c +++ b/networking/ifupdown.c @@ -89,7 +89,6 @@ struct interfaces_file_t struct mapping_defn_t *mappings; }; -static unsigned option_mask; #define OPTION_STR "anvf" USE_FEATURE_IFUPDOWN_MAPPING("m") "i:" enum { OPT_do_all = 0x1, @@ -98,11 +97,11 @@ enum { OPT_force = 0x8, OPT_no_mappings = 0x10, }; -#define DO_ALL (option_mask & OPT_do_all) -#define NO_ACT (option_mask & OPT_no_act) -#define VERBOSE (option_mask & OPT_verbose) -#define FORCE (option_mask & OPT_force) -#define NO_MAPPINGS (option_mask & OPT_no_mappings) +#define DO_ALL (option_mask32 & OPT_do_all) +#define NO_ACT (option_mask32 & OPT_no_act) +#define VERBOSE (option_mask32 & OPT_verbose) +#define FORCE (option_mask32 & OPT_force) +#define NO_MAPPINGS (option_mask32 & OPT_no_mappings) static char **__myenviron; @@ -881,10 +880,10 @@ static void set_environ(struct interface_defn_t *iface, const char *mode) static int doit(char *str) { - if (option_mask & (OPT_no_act|OPT_verbose)) { + if (option_mask32 & (OPT_no_act|OPT_verbose)) { puts(str); } - if (!(option_mask & OPT_no_act)) { + if (!(option_mask32 & OPT_no_act)) { pid_t child; int status; @@ -1088,7 +1087,7 @@ int ifupdown_main(int argc, char **argv) cmds = iface_down; } - option_mask = getopt32(argc, argv, OPTION_STR, &interfaces); + getopt32(argc, argv, OPTION_STR, &interfaces); if (argc - optind > 0) { if (DO_ALL) bb_show_usage(); } else diff --git a/procps/top.c b/procps/top.c index ebfbcb80c..be1c45faa 100644 --- a/procps/top.c +++ b/procps/top.c @@ -34,8 +34,7 @@ typedef int (*cmp_t)(procps_status_t *P, procps_status_t *Q); static procps_status_t *top; /* Hehe */ static int ntop; -static unsigned option_mask; -#define OPT_BATCH_MODE (option_mask & 0x4) +#define OPT_BATCH_MODE (option_mask32 & 0x4) #ifdef CONFIG_FEATURE_USE_TERMIOS static int pid_sort(procps_status_t *P, procps_status_t *Q) @@ -398,11 +397,11 @@ int top_main(int argc, char **argv) /* do normal option parsing */ interval = 5; opt_complementary = "-"; - option_mask = getopt32(argc, argv, "d:n:b", + getopt32(argc, argv, "d:n:b", &sinterval, &siterations); - if (option_mask & 0x1) interval = atoi(sinterval); // -d - if (option_mask & 0x2) iterations = atoi(siterations); // -n - //if (option_mask & 0x4) // -b + if (option_mask32 & 0x1) interval = atoi(sinterval); // -d + if (option_mask32 & 0x2) iterations = atoi(siterations); // -n + //if (option_mask32 & 0x4) // -b /* change to /proc */ xchdir("/proc"); diff --git a/runit/chpst.c b/runit/chpst.c index b26d62866..4c44e3c0c 100644 --- a/runit/chpst.c +++ b/runit/chpst.c @@ -2,13 +2,12 @@ #include -static unsigned option_mask; // Must match constants in chpst_main! -#define OPT_verbose (option_mask & 0x2000) -#define OPT_pgrp (option_mask & 0x4000) -#define OPT_nostdin (option_mask & 0x8000) -#define OPT_nostdout (option_mask & 0x10000) -#define OPT_nostderr (option_mask & 0x20000) +#define OPT_verbose (option_mask32 & 0x2000) +#define OPT_pgrp (option_mask32 & 0x4000) +#define OPT_nostdin (option_mask32 & 0x8000) +#define OPT_nostdout (option_mask32 & 0x10000) +#define OPT_nostderr (option_mask32 & 0x20000) static char *set_user; static char *env_user; @@ -224,28 +223,28 @@ int chpst_main(int argc, char **argv) { char *m,*d,*o,*p,*f,*c,*r,*t,*n; - option_mask = getopt32(argc, argv, "u:U:e:m:d:o:p:f:c:r:t:/:n:vP012", + getopt32(argc, argv, "u:U:e:m:d:o:p:f:c:r:t:/:n:vP012", &set_user,&env_user,&env_dir, &m,&d,&o,&p,&f,&c,&r,&t,&root,&n); - // if (option_mask & 0x1) // -u - // if (option_mask & 0x2) // -U - // if (option_mask & 0x4) // -e - if (option_mask & 0x8) limits = limitl = limita = limitd = bb_xgetularg10(m); // -m - if (option_mask & 0x10) limitd = bb_xgetularg10(d); // -d - if (option_mask & 0x20) limito = bb_xgetularg10(o); // -o - if (option_mask & 0x40) limitp = bb_xgetularg10(p); // -p - if (option_mask & 0x80) limitf = bb_xgetularg10(f); // -f - if (option_mask & 0x100) limitc = bb_xgetularg10(c); // -c - if (option_mask & 0x200) limitr = bb_xgetularg10(r); // -r - if (option_mask & 0x400) limitt = bb_xgetularg10(t); // -t - // if (option_mask & 0x800) // -/ - if (option_mask & 0x1000) nicelvl = bb_xgetlarg_bnd_sfx(n, 10, -20, 20, NULL); // -n + // if (option_mask32 & 0x1) // -u + // if (option_mask32 & 0x2) // -U + // if (option_mask32 & 0x4) // -e + if (option_mask32 & 0x8) limits = limitl = limita = limitd = bb_xgetularg10(m); // -m + if (option_mask32 & 0x10) limitd = bb_xgetularg10(d); // -d + if (option_mask32 & 0x20) limito = bb_xgetularg10(o); // -o + if (option_mask32 & 0x40) limitp = bb_xgetularg10(p); // -p + if (option_mask32 & 0x80) limitf = bb_xgetularg10(f); // -f + if (option_mask32 & 0x100) limitc = bb_xgetularg10(c); // -c + if (option_mask32 & 0x200) limitr = bb_xgetularg10(r); // -r + if (option_mask32 & 0x400) limitt = bb_xgetularg10(t); // -t + // if (option_mask32 & 0x800) // -/ + if (option_mask32 & 0x1000) nicelvl = bb_xgetlarg_bnd_sfx(n, 10, -20, 20, NULL); // -n // The below consts should match #defines at top! - //if (option_mask & 0x2000) OPT_verbose = 1; // -v - //if (option_mask & 0x4000) OPT_pgrp = 1; // -P - //if (option_mask & 0x8000) OPT_nostdin = 1; // -0 - //if (option_mask & 0x10000) OPT_nostdout = 1; // -1 - //if (option_mask & 0x20000) OPT_nostderr = 1; // -2 + //if (option_mask32 & 0x2000) OPT_verbose = 1; // -v + //if (option_mask32 & 0x4000) OPT_pgrp = 1; // -P + //if (option_mask32 & 0x8000) OPT_nostdin = 1; // -0 + //if (option_mask32 & 0x10000) OPT_nostdout = 1; // -1 + //if (option_mask32 & 0x20000) OPT_nostderr = 1; // -2 } argv += optind; if (!argv || !*argv) bb_show_usage(); @@ -311,19 +310,19 @@ static void envdir(int argc, char **argv) static void softlimit(int argc, char **argv) { char *a,*c,*d,*f,*l,*m,*o,*p,*r,*s,*t; - option_mask = getopt32(argc, argv, "a:c:d:f:l:m:o:p:r:s:t:", + getopt32(argc, argv, "a:c:d:f:l:m:o:p:r:s:t:", &a,&c,&d,&f,&l,&m,&o,&p,&r,&s,&t); - if (option_mask & 0x001) limita = bb_xgetularg10(a); // -a - if (option_mask & 0x002) limitc = bb_xgetularg10(c); // -c - if (option_mask & 0x004) limitd = bb_xgetularg10(d); // -d - if (option_mask & 0x008) limitf = bb_xgetularg10(f); // -f - if (option_mask & 0x010) limitl = bb_xgetularg10(l); // -l - if (option_mask & 0x020) limits = limitl = limita = limitd = bb_xgetularg10(m); // -m - if (option_mask & 0x040) limito = bb_xgetularg10(o); // -o - if (option_mask & 0x080) limitp = bb_xgetularg10(p); // -p - if (option_mask & 0x100) limitr = bb_xgetularg10(r); // -r - if (option_mask & 0x200) limits = bb_xgetularg10(s); // -s - if (option_mask & 0x400) limitt = bb_xgetularg10(t); // -t + if (option_mask32 & 0x001) limita = bb_xgetularg10(a); // -a + if (option_mask32 & 0x002) limitc = bb_xgetularg10(c); // -c + if (option_mask32 & 0x004) limitd = bb_xgetularg10(d); // -d + if (option_mask32 & 0x008) limitf = bb_xgetularg10(f); // -f + if (option_mask32 & 0x010) limitl = bb_xgetularg10(l); // -l + if (option_mask32 & 0x020) limits = limitl = limita = limitd = bb_xgetularg10(m); // -m + if (option_mask32 & 0x040) limito = bb_xgetularg10(o); // -o + if (option_mask32 & 0x080) limitp = bb_xgetularg10(p); // -p + if (option_mask32 & 0x100) limitr = bb_xgetularg10(r); // -r + if (option_mask32 & 0x200) limits = bb_xgetularg10(s); // -s + if (option_mask32 & 0x400) limitt = bb_xgetularg10(t); // -t argv += optind; if (!argv[0]) bb_show_usage(); slimit(); diff --git a/sysklogd/syslogd.c b/sysklogd/syslogd.c index 40b98c1cb..254d7e73c 100644 --- a/sysklogd/syslogd.c +++ b/sysklogd/syslogd.c @@ -55,7 +55,6 @@ static struct sockaddr_in remoteaddr; #endif /* options */ -static unsigned option_mask; /* Correct regardless of combination of CONFIG_xxx */ enum { OPTBIT_mark = 0, // -m @@ -298,7 +297,7 @@ static void message(char *fmt, ...) fl.l_len = 1; #ifdef CONFIG_FEATURE_IPC_SYSLOG - if ((option_mask & OPT_circularlog) && shbuf) { + if ((option_mask32 & OPT_circularlog) && shbuf) { char b[1024]; va_start(arguments, fmt); @@ -404,7 +403,7 @@ static void logMessage(int pri, char *msg) /* todo: supress duplicates */ #ifdef CONFIG_FEATURE_REMOTE_LOG - if (option_mask & OPT_remotelog) { + if (option_mask32 & OPT_remotelog) { char line[MAXLINE + 1]; /* trying connect the socket */ if (-1 == remotefd) { @@ -419,12 +418,12 @@ static void logMessage(int pri, char *msg) } } - if (option_mask & OPT_locallog) + if (option_mask32 & OPT_locallog) #endif { /* now spew out the message to wherever it is supposed to go */ if (pri == 0 || LOG_PRI(pri) < logLevel) { - if (option_mask & OPT_small) + if (option_mask32 & OPT_small) message("%s %s\n", timestamp, msg); else message("%s %s %s %s\n", timestamp, LocalHostName, res, msg); @@ -531,7 +530,7 @@ static void doSyslogd(void) if (chmod(lfile, 0666) < 0) { bb_perror_msg_and_die("cannot set permission on %s", lfile); } - if (ENABLE_FEATURE_IPC_SYSLOG && (option_mask & OPT_circularlog)) { + if (ENABLE_FEATURE_IPC_SYSLOG && (option_mask32 & OPT_circularlog)) { ipcsyslog_init(); } @@ -575,26 +574,26 @@ int syslogd_main(int argc, char **argv) char *p; /* do normal option parsing */ - option_mask = getopt32(argc, argv, OPTION_STR, OPTION_PARAM); - if (option_mask & OPT_mark) MarkInterval = atoi(opt_m) * 60; // -m - //if (option_mask & OPT_nofork) // -n - //if (option_mask & OPT_outfile) // -O - if (option_mask & OPT_loglevel) { // -l + getopt32(argc, argv, OPTION_STR, OPTION_PARAM); + if (option_mask32 & OPT_mark) MarkInterval = atoi(opt_m) * 60; // -m + //if (option_mask32 & OPT_nofork) // -n + //if (option_mask32 & OPT_outfile) // -O + if (option_mask32 & OPT_loglevel) { // -l logLevel = atoi(opt_l); /* Valid levels are between 1 and 8 */ if (logLevel < 1 || logLevel > 8) bb_show_usage(); } - //if (option_mask & OPT_small) // -S + //if (option_mask32 & OPT_small) // -S #if ENABLE_FEATURE_ROTATE_LOGFILE - if (option_mask & OPT_filesize) logFileSize = atoi(opt_s) * 1024; // -s - if (option_mask & OPT_rotatecnt) { // -b + if (option_mask32 & OPT_filesize) logFileSize = atoi(opt_s) * 1024; // -s + if (option_mask32 & OPT_rotatecnt) { // -b logFileRotate = atoi(opt_b); if (logFileRotate > 99) logFileRotate = 99; } #endif #if ENABLE_FEATURE_REMOTE_LOG - if (option_mask & OPT_remotelog) { // -R + if (option_mask32 & OPT_remotelog) { // -R int port = 514; char *host = xstrdup(opt_R); p = strchr(host, ':'); @@ -608,10 +607,10 @@ int syslogd_main(int argc, char **argv) remoteaddr.sin_port = htons(port); free(host); } - //if (option_mask & OPT_locallog) // -L + //if (option_mask32 & OPT_locallog) // -L #endif #if ENABLE_FEATURE_IPC_SYSLOG - if (option_mask & OPT_circularlog) { // -C + if (option_mask32 & OPT_circularlog) { // -C if (opt_C) { int buf_size = atoi(opt_C); if (buf_size >= 4) @@ -621,8 +620,8 @@ int syslogd_main(int argc, char **argv) #endif /* If they have not specified remote logging, then log locally */ - if (ENABLE_FEATURE_REMOTE_LOG && !(option_mask & OPT_remotelog)) - option_mask |= OPT_locallog; + if (ENABLE_FEATURE_REMOTE_LOG && !(option_mask32 & OPT_remotelog)) + option_mask32 |= OPT_locallog; /* Store away localhost's name before the fork */ gethostname(LocalHostName, sizeof(LocalHostName)); @@ -633,7 +632,7 @@ int syslogd_main(int argc, char **argv) umask(0); - if (!(option_mask & OPT_nofork)) { + if (!(option_mask32 & OPT_nofork)) { #ifdef BB_NOMMU vfork_daemon_rexec(0, 1, argc, argv, "-n"); #else -- cgit v1.2.3