isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make a copy of ap in sudo_debug_vprintf2() in case the static buffer

Browse Source 
is not big enough and we need to call vasprintf().
This commit is contained in:
Todd C. Miller
2014-10-07 12:26:32 -06:00
parent dc67442151
commit 1dd96778e5
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
lib/util/sudo_debug.c
View File

@@ -370,8 +370,7 @@ void
sudo_debug_vprintf2(const char *func, const char *file, int lineno, int level,
const char *fmt, va_list ap)
{
int buflen, pri, subsys, saved_errno = errno;
char static_buf[1024], *buf = static_buf;
int pri, subsys;
if (!sudo_debug_initialized)
return;
@@ -382,10 +381,15 @@ sudo_debug_vprintf2(const char *func, const char *file, int lineno, int level,
/* Make sure we want debug info at this level. */
if (subsys < num_subsystems && sudo_debug_settings[subsys] >= pri) {
char static_buf[1024], *buf = static_buf;
int buflen, saved_errno = errno;
va_list ap2;
va_copy(ap2, ap);
buflen = fmt ? vsnprintf(static_buf, sizeof(static_buf), fmt, ap) : 0;
if (buflen >= (int)sizeof(static_buf)) {
/* Not enough room in static buf, allocate dynamically. */
buflen = vasprintf(&buf, fmt, ap);
buflen = vasprintf(&buf, fmt, ap2);
}
if (buflen != -1) {
int errcode = ISSET(level, SUDO_DEBUG_ERRNO) ? saved_errno : 0;
@@ -396,10 +400,10 @@ sudo_debug_vprintf2(const char *func, const char *file, int lineno, int level,
if (buf != static_buf)
free(buf);
}
}
va_end(ap2);
errno = saved_errno;
}
}
#ifdef NO_VARIADIC_MACROS
void