diff options
author | Denis Vlasenko | 2008-07-16 23:04:49 +0000 |
---|---|---|
committer | Denis Vlasenko | 2008-07-16 23:04:49 +0000 |
commit | fb1642f2caa3690cb40f603fca22eeace09a0bfa (patch) | |
tree | 0ae2f20765b93696322e8c6c5e928b0b1be2cbee | |
parent | c01340fe26b76e172805ff641ad9af6bc45cdc91 (diff) | |
download | busybox-fb1642f2caa3690cb40f603fca22eeace09a0bfa.zip busybox-fb1642f2caa3690cb40f603fca22eeace09a0bfa.tar.gz |
fix up callsites of config_read to check for >= 0
-rw-r--r-- | libbb/parse_config.c | 4 | ||||
-rw-r--r-- | miscutils/crond.c | 2 | ||||
-rw-r--r-- | networking/nameif.c | 2 | ||||
-rw-r--r-- | util-linux/mdev.c | 2 |
4 files changed, 5 insertions, 5 deletions
diff --git a/libbb/parse_config.c b/libbb/parse_config.c index e63204b..f070992 100644 --- a/libbb/parse_config.c +++ b/libbb/parse_config.c @@ -19,7 +19,7 @@ Typical usage: // open file if (config_open(filename, &p)) { // parse line-by-line - while (*config_read(&p, t, 3, 0, delimiters, comment_char)) { // 0..3 tokens + while (*config_read(&p, t, 3, 0, delimiters, comment_char) >= 0) { // 0..3 tokens // use tokens bb_error_msg("TOKENS: [%s][%s][%s]", t[0], t[1], t[2]); } @@ -77,7 +77,7 @@ int FAST_FUNC config_read(parser_t *parser, char **tokens, int ntokens, int mint //TODO: speed up xmalloc_fgetline by internally using fgets, not fgetc line = xmalloc_fgetline(parser->fp); if (!line) - return line; + return -1; parser->lineno++; // handle continuations. Tito's code stolen :) diff --git a/miscutils/crond.c b/miscutils/crond.c index af37bb1..41f1517 100644 --- a/miscutils/crond.c +++ b/miscutils/crond.c @@ -468,7 +468,7 @@ static void SynchronizeFile(const char *fileName) file->cf_User = xstrdup(fileName); pline = &file->cf_LineBase; - while (--maxLines && (n=config_read(&parser, tokens, 6, 0, " \t", '#')) > 0) { + while (--maxLines && (n=config_read(&parser, tokens, 6, 0, " \t", '#')) >= 0) { CronLine *line; if (DebugOpt) { diff --git a/networking/nameif.c b/networking/nameif.c index 506f4fa..c5a715e 100644 --- a/networking/nameif.c +++ b/networking/nameif.c @@ -163,7 +163,7 @@ int nameif_main(int argc, char **argv) struct parser_t parser; if (config_open(&parser, fname)) { char *tokens[2]; - while (config_read(&parser, tokens, 2, 2, " \t", '#')) + while (config_read(&parser, tokens, 2, 2, " \t", '#') >= 0) prepend_new_eth_table(&clist, tokens[0], tokens[1]); config_close(&parser); } diff --git a/util-linux/mdev.c b/util-linux/mdev.c index 8968e45..c04410c 100644 --- a/util-linux/mdev.c +++ b/util-linux/mdev.c @@ -101,7 +101,7 @@ static void make_device(char *path, int delete) if (!config_open(&parser, "/etc/mdev.conf")) goto end_parse; - while (config_read(&parser, tokens, 4, 3, " \t", '#')) { + while (config_read(&parser, tokens, 4, 3, " \t", '#') >= 0) { regmatch_t off[1+9*ENABLE_FEATURE_MDEV_RENAME_REGEXP]; char *val; |