← Commit list

4df8135cf

Denis Vlasenko
Denis Vlasenko
2007-01-12
FAILED Duration: 3s

Subject

undo RESERVE_CONFIG_BUFFER change. aldot, small on-stack char[N] objects are perfectly okay, don't convert them to RESERVE_CONFIG_BUFFERs.

Binary

No binary (build failed)

Architecture & Compiler Flags

Architecture: x86_64-linux-gnu Compiler: gcc (Ubuntu 11.4.0-1ubuntu1~22.04.3) 11.4.0

—

Build Log

(log yok)

Commit Diff GitHub →

sysklogd/logger.c+7 -9
@@ -83,26 +83,24 @@ static int pencode(char *s)
int logger_main(int argc, char **argv)
{
- char *opt_p, *opt_t;
+ char *str_p, *str_t;
int i = 0;
- RESERVE_CONFIG_BUFFER(name, 80);
+ char name[80];
/* Fill out the name string early (may be overwritten later) */
bb_getpwuid(name, geteuid(), sizeof(name));
+ str_t = name;
/* Parse any options */
- getopt32(argc, argv, "p:st:", &opt_p, &opt_t);
+ getopt32(argc, argv, "p:st:", &str_p, &str_t);
if (option_mask32 & 0x2) /* -s */
i |= LOG_PERROR;
- if (option_mask32 & 0x4) /* -t */
- safe_strncpy(name, opt_t, sizeof(name));
- openlog(name, i, 0);
- if (ENABLE_FEATURE_CLEAN_UP)
- RELEASE_CONFIG_BUFFER(name);
+ //if (option_mask32 & 0x4) /* -t */
+ openlog(str_t, i, 0);
i = LOG_USER | LOG_NOTICE;
if (option_mask32 & 0x1) /* -p */
- i = pencode(opt_p);
+ i = pencode(str_p);
argc -= optind;
argv += optind;