diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-04 10:16:52 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-04 10:16:52 +0000 |
commit | 74324c86663f57a19c1de303ee8c8e5449db9ef2 (patch) | |
tree | 11f5da9de4212875ce5811be2e1050e076378c9a /sysklogd/logger.c | |
parent | 4e5f82c76f08614d0b69f9ec4a8baac303af15f6 (diff) | |
download | busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.tar.gz busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.tar.bz2 busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.zip |
Audit bb_common_bufsiz usage, add script which looks for misuse.
tr: stop using globals needlessly.
code: -103 bytes
Diffstat (limited to 'sysklogd/logger.c')
-rw-r--r-- | sysklogd/logger.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/sysklogd/logger.c b/sysklogd/logger.c index 971a6beae..d9c3e5048 100644 --- a/sysklogd/logger.c +++ b/sysklogd/logger.c | |||
@@ -106,12 +106,13 @@ int logger_main(int argc, char **argv) | |||
106 | argc -= optind; | 106 | argc -= optind; |
107 | argv += optind; | 107 | argv += optind; |
108 | if (!argc) { | 108 | if (!argc) { |
109 | while (fgets(bb_common_bufsiz1, BUFSIZ, stdin)) { | 109 | #define strbuf bb_common_bufsiz1 |
110 | if (bb_common_bufsiz1[0] | 110 | while (fgets(strbuf, BUFSIZ, stdin)) { |
111 | && NOT_LONE_CHAR(bb_common_bufsiz1, '\n') | 111 | if (strbuf[0] |
112 | && NOT_LONE_CHAR(strbuf, '\n') | ||
112 | ) { | 113 | ) { |
113 | /* Neither "" nor "\n" */ | 114 | /* Neither "" nor "\n" */ |
114 | syslog(i, "%s", bb_common_bufsiz1); | 115 | syslog(i, "%s", strbuf); |
115 | } | 116 | } |
116 | } | 117 | } |
117 | } else { | 118 | } else { |