Skip to content

Commit 7fc1495

Browse files
committed
Merge branch 'jk/maint-snprintf-va-copy' into maint
* jk/maint-snprintf-va-copy: compat/snprintf: don't look at va_list twice
2 parents f1c12e1 + a9bfbc5 commit 7fc1495

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

compat/snprintf.c

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,14 @@
1919
#undef vsnprintf
2020
int git_vsnprintf(char *str, size_t maxsize, const char *format, va_list ap)
2121
{
22+
va_list cp;
2223
char *s;
2324
int ret = -1;
2425

2526
if (maxsize > 0) {
26-
ret = vsnprintf(str, maxsize-SNPRINTF_SIZE_CORR, format, ap);
27+
va_copy(cp, ap);
28+
ret = vsnprintf(str, maxsize-SNPRINTF_SIZE_CORR, format, cp);
29+
va_end(cp);
2730
if (ret == maxsize-1)
2831
ret = -1;
2932
/* Windows does not NUL-terminate if result fills buffer */
@@ -42,7 +45,9 @@ int git_vsnprintf(char *str, size_t maxsize, const char *format, va_list ap)
4245
if (! str)
4346
break;
4447
s = str;
45-
ret = vsnprintf(str, maxsize-SNPRINTF_SIZE_CORR, format, ap);
48+
va_copy(cp, ap);
49+
ret = vsnprintf(str, maxsize-SNPRINTF_SIZE_CORR, format, cp);
50+
va_end(cp);
4651
if (ret == maxsize-1)
4752
ret = -1;
4853
}

0 commit comments

Comments
 (0)