minor vfprintf and vfwprintf changes to please static code analyzers
authorSzabolcs Nagy <nsz@port70.net>
Mon, 7 Oct 2013 13:22:24 +0000 (13:22 +0000)
committerSzabolcs Nagy <nsz@port70.net>
Mon, 7 Oct 2013 13:22:24 +0000 (13:22 +0000)
add missing va_end and remove some unnecessary code.

src/stdio/vfprintf.c
src/stdio/vfwprintf.c

index a2b287bd17c5dcce66821d2a8af55cbbf7a5623a..b5948bd2649677f1c0acfd4e292d9f8dec9484d0 100644 (file)
@@ -530,7 +530,6 @@ static int printf_core(FILE *f, const char *fmt, va_list *ap, union arg *nl_arg,
                /* Check validity of argument type (nl/normal) */
                if (st==NOARG) {
                        if (argpos>=0) return -1;
-                       else if (!f) continue;
                } else {
                        if (argpos>=0) nl_type[argpos]=st, arg=nl_arg[argpos];
                        else if (f) pop_arg(&arg, st, ap);
@@ -660,8 +659,12 @@ int vfprintf(FILE *restrict f, const char *restrict fmt, va_list ap)
        unsigned char internal_buf[80], *saved_buf = 0;
        int ret;
 
+       /* the copy allows passing va_list* even if va_list is an array */
        va_copy(ap2, ap);
-       if (printf_core(0, fmt, &ap2, nl_arg, nl_type) < 0) return -1;
+       if (printf_core(0, fmt, &ap2, nl_arg, nl_type) < 0) {
+               va_end(ap2);
+               return -1;
+       }
 
        FLOCK(f);
        if (!f->buf_size) {
index 3557678f455d0189a169e5589b52c143e2d0b5b5..984ff7b7fc8b9c2c0157786ed7fd5d00a4230dd8 100644 (file)
@@ -253,7 +253,6 @@ static int wprintf_core(FILE *f, const wchar_t *fmt, va_list *ap, union arg *nl_
                /* Check validity of argument type (nl/normal) */
                if (st==NOARG) {
                        if (argpos>=0) return -1;
-                       else if (!f) continue;
                } else {
                        if (argpos>=0) nl_type[argpos]=st, arg=nl_arg[argpos];
                        else if (f) pop_arg(&arg, st, ap);
@@ -287,8 +286,7 @@ static int wprintf_core(FILE *f, const wchar_t *fmt, va_list *ap, union arg *nl_
                case 'S':
                        a = arg.p;
                        z = wmemchr(a, 0, p);
-                       if (!z) z=a+p;
-                       else p=z-a;
+                       if (z) p=z-a;
                        if (w<p) w=p;
                        if (!(fl&LEFT_ADJ)) fprintf(f, "%.*s", w-p, "");
                        out(f, a, p);
@@ -349,8 +347,12 @@ int vfwprintf(FILE *restrict f, const wchar_t *restrict fmt, va_list ap)
        union arg nl_arg[NL_ARGMAX];
        int ret;
 
+       /* the copy allows passing va_list* even if va_list is an array */
        va_copy(ap2, ap);
-       if (wprintf_core(0, fmt, &ap2, nl_arg, nl_type) < 0) return -1;
+       if (wprintf_core(0, fmt, &ap2, nl_arg, nl_type) < 0) {
+               va_end(ap2);
+               return -1;
+       }
 
        FLOCK(f);
        ret = wprintf_core(f, fmt, &ap2, nl_arg, nl_type);