diff options
author | Denis Vlasenko | 2006-10-23 22:43:02 +0000 |
---|---|---|
committer | Denis Vlasenko | 2006-10-23 22:43:02 +0000 |
commit | 703aa13ff5e2b795319a3e900c1f8df6e3da47b5 (patch) | |
tree | 861026de3497f90795ef52dfd196097a37b754c7 | |
parent | 4e70bf4359676b1ecaf658c7550d60a15ec1a02a (diff) | |
download | busybox-703aa13ff5e2b795319a3e900c1f8df6e3da47b5.zip busybox-703aa13ff5e2b795319a3e900c1f8df6e3da47b5.tar.gz |
watch: fix warning
getty: fix breakage; fix excessive stack usage
-rw-r--r-- | coreutils/watch.c | 4 | ||||
-rw-r--r-- | loginutils/getty.c | 44 |
2 files changed, 25 insertions, 23 deletions
diff --git a/coreutils/watch.c b/coreutils/watch.c index e3e9e06..b188b41 100644 --- a/coreutils/watch.c +++ b/coreutils/watch.c @@ -58,8 +58,9 @@ int watch_main(int argc, char **argv) time_t t; get_terminal_width_height(STDOUT_FILENO, &width, 0); + if (width < 1) width = 1; // paranoia header = xrealloc(header, width--); - // We pad with spaces entire length + // '%-*s' pads header with spaces to the full width snprintf(header, width, "Every %ds: %-*s", period, width, cmd); time(&t); thyme = ctime(&t); @@ -75,4 +76,5 @@ int watch_main(int argc, char **argv) system(cmd); sleep(period); } + return 0; // gcc thinks we can reach this :) } diff --git a/loginutils/getty.c b/loginutils/getty.c index 9e7cb7a..e9b4f07 100644 --- a/loginutils/getty.c +++ b/loginutils/getty.c @@ -409,12 +409,11 @@ static void termio_init(struct termio *tp, int speed, struct options *op) } /* auto_baud - extract baud rate from modem status message */ -static void auto_baud(struct termio *tp) +static void auto_baud(char *buf, unsigned size_buf, struct termio *tp) { int speed; int vmin; unsigned iflag; - char buf[BUFSIZ]; char *bp; int nread; @@ -450,7 +449,7 @@ static void auto_baud(struct termio *tp) */ sleep(1); - nread = read(0, buf, sizeof(buf) - 1); + nread = read(0, buf, size_buf - 1); if (nread > 0) { buf[nread] = '\0'; for (bp = buf; bp < buf + nread; bp++) { @@ -504,10 +503,10 @@ static int caps_lock(const char *s) } #endif -#define logname bb_common_bufsiz1 /* get_logname - get user name, establish parity, speed, erase, kill, eol */ /* return NULL on failure, logname on success */ -static char *get_logname(struct options *op, struct chardata *cp, struct termio *tp) +static char *get_logname(char *logname, unsigned size_logname, + struct options *op, struct chardata *cp, struct termio *tp) { char *bp; char c; /* input character, full eight bits */ @@ -532,8 +531,8 @@ static char *get_logname(struct options *op, struct chardata *cp, struct termio /* Prompt for and read a login name. */ - *logname = 0; - while (*logname) { + logname[0] = '\0'; + while (!logname[0]) { /* Write issue file and prompt, with "parity" bit == 0. */ @@ -542,37 +541,39 @@ static char *get_logname(struct options *op, struct chardata *cp, struct termio /* Read name, watch for break, parity, erase, kill, end-of-line. */ bp = logname; - cp->eol = 0; - while (cp->eol == 0) { + cp->eol = '\0'; + while (cp->eol == '\0') { /* Do not report trivial EINTR/EIO errors. */ - if (read(0, &c, 1) < 1) { if (errno == EINTR || errno == EIO) exit(0); bb_perror_msg_and_die("%s: read", op->tty); } - /* Do BREAK handling elsewhere. */ - if (c == 0 && op->numspeed > 1) + /* Do BREAK handling elsewhere. */ + if (c == '\0' && op->numspeed > 1) return NULL; /* Do parity bit handling. */ - ascval = c & 0177; if (c != ascval) { /* "parity" bit on ? */ - for (bits = 1, mask = 1; mask & 0177; mask <<= 1) + bits = 1; + mask = 1; + while (mask & 0177) { if (mask & ascval) bits++; /* count "1" bits */ + mask <<= 1; + } /* ... |= 2 - even, 1 - odd */ cp->parity |= 2 - (bits & 1); } - /* Do erase, kill and end-of-line processing. */ + /* Do erase, kill and end-of-line processing. */ switch (ascval) { case CR: case NL: - *bp = 0; /* terminate logname */ + *bp = '\0'; /* terminate logname */ cp->eol = ascval; /* set end-of-line char */ break; case BS: @@ -596,8 +597,8 @@ static char *get_logname(struct options *op, struct chardata *cp, struct termio exit(0); default: if (!isascii(ascval) || !isprint(ascval)) { - /* ignore garbage characters */ ; - } else if (bp - logname >= sizeof(logname) - 1) { + /* ignore garbage characters */ + } else if (bp - logname >= size_logname - 1) { bb_error_msg_and_die("%s: input overrun", op->tty); } else { write(1, &c, 1); /* echo the character */ @@ -747,7 +748,6 @@ static void update_utmp(char *line) #endif /* SYSV_STYLE */ -#undef logname int getty_main(int argc, char **argv) { int nullfd; @@ -847,7 +847,7 @@ int getty_main(int argc, char **argv) /* Optionally detect the baud rate from the modem status message. */ debug("before autobaud\n"); if (options.flags & F_PARSE) - auto_baud(&termio); + auto_baud(bb_common_bufsiz1, sizeof(bb_common_bufsiz1), &termio); /* Set the optional timer. */ if (options.timeout) @@ -872,8 +872,8 @@ int getty_main(int argc, char **argv) if (!(options.flags & F_NOPROMPT)) { /* Read the login name. */ debug("reading login name\n"); - /* while ((logname = get_logname(&options, &chardata, &termio)) == 0) */ - logname = get_logname(&options, &chardata, &termio); + logname = get_logname(bb_common_bufsiz1, sizeof(bb_common_bufsiz1), + &options, &chardata, &termio); while (logname == NULL) next_speed(&termio, &options); } |