summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Vlasenko2007-05-29 21:41:28 +0000
committerDenis Vlasenko2007-05-29 21:41:28 +0000
commit4ef7d3a2de3df27dde80a64987202f10aebfe925 (patch)
treec34efb08a5db806cdee923dd81fffdd8c46f0cdd
parentfe3e1776ada6bf8cf39e08f4974bd6cc6d29175d (diff)
downloadbusybox-4ef7d3a2de3df27dde80a64987202f10aebfe925.zip
busybox-4ef7d3a2de3df27dde80a64987202f10aebfe925.tar.gz
verror_msg: don't printf(NULL)
-rw-r--r--libbb/verror_msg.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/libbb/verror_msg.c b/libbb/verror_msg.c
index 526b9d3..082e7bf 100644
--- a/libbb/verror_msg.c
+++ b/libbb/verror_msg.c
@@ -20,6 +20,9 @@ void bb_verror_msg(const char *s, va_list p, const char* strerr)
va_list p2;
va_copy(p2, p);
+ if (!s) /* nomsg[_and_die] uses NULL fmt */
+ s = ""; /* some libc don't like printf(NULL) */
+
if (logmode & LOGMODE_STDIO) {
fflush(stdout);
fprintf(stderr, "%s: ", applet_name);
@@ -28,7 +31,7 @@ void bb_verror_msg(const char *s, va_list p, const char* strerr)
fputs(msg_eol, stderr);
else
fprintf(stderr, "%s%s%s",
- s ? ": " : "",
+ s[0] ? ": " : "",
strerr, msg_eol);
}
if (ENABLE_FEATURE_SYSLOG && (logmode & LOGMODE_SYSLOG)) {
@@ -36,8 +39,10 @@ void bb_verror_msg(const char *s, va_list p, const char* strerr)
vsyslog(LOG_ERR, s, p2);
else {
char *msg;
- if (vasprintf(&msg, s, p2) < 0)
- bb_error_msg_and_die(bb_msg_memory_exhausted);
+ if (vasprintf(&msg, s, p2) < 0) {
+ fprintf(stderr, "%s: %s\n", applet_name, bb_msg_memory_exhausted);
+ xfunc_die();
+ }
syslog(LOG_ERR, "%s: %s", msg, strerr);
free(msg);
}