Fix some cppcheck reports
authorJulien Nabet <serval2412@yahoo.fr>
Fri, 22 Jan 2016 19:57:24 +0000 (20:57 +0100)
committerErik de Castro Lopo <erikd@mega-nerd.com>
Fri, 22 Jan 2016 20:45:54 +0000 (07:45 +1100)
* src/libFLAC/metadata_iterators.c:3247: (error) va_list `va` was opened
  but not closed with va_end().
* src/share/grabbag/snprintf.c:66: (error) va_list `va` was opened but
  not closed with va_end().
* src/utils/flactimer/main.cpp:84: (warning) Either the condition `!fout`
  is redundant or there is possible null pointer dereference.

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
Closes: https://github.com/xiph/flac/pull/8

src/libFLAC/metadata_iterators.c
src/share/grabbag/snprintf.c
src/utils/flactimer/main.cpp

index 4fd58c8..9356416 100644 (file)
@@ -3240,11 +3240,14 @@ local_snprintf(char *str, size_t size, const char *fmt, ...)
        va_list va;
        int rc;
 
-       va_start (va, fmt);
-
 #if defined _MSC_VER
        if (size == 0)
                return 1024;
+#endif
+
+       va_start (va, fmt);
+
+#if defined _MSC_VER
        rc = vsnprintf_s (str, size, _TRUNCATE, fmt, va);
        if (rc < 0)
                rc = size - 1;
index f037377..5b7f482 100644 (file)
@@ -59,11 +59,14 @@ flac_snprintf(char *str, size_t size, const char *fmt, ...)
        va_list va;
        int rc;
 
-       va_start (va, fmt);
-
 #if defined _MSC_VER
        if (size == 0)
                return 1024;
+#endif
+
+       va_start (va, fmt);
+
+#if defined _MSC_VER
        rc = vsnprintf_s (str, size, _TRUNCATE, fmt, va);
        if (rc < 0)
                rc = size - 1;
index dfc9ab7..9245c59 100644 (file)
@@ -81,7 +81,7 @@ int main(int argc, char *argv[])
                }
                fout = fopen(argv[1], "w");
                if(!fout) {
-                       fprintf(fout, "ERROR opening file %s for writing\n", argv[1]);
+                       fprintf(stderr, "ERROR opening file %s for writing\n", argv[1]);
                        return 1;
                }
                argv += 2;