vStringObjPrintf: fix potential va_start leak

va_start() may allocate resources and thus has to be matched by
va_end(). Found with cppcheck 1.67.

1 files changed, 2 insertions, 1 deletions

diff --git a/src/sysync/stringutils.cpp b/src/sysync/stringutils.cpp
index f62a35c..437eb3f 100755
--- a/src/sysync/stringutils.cpp
+++ b/src/sysync/stringutils.cpp
@@ -823,9 +823,10 @@ void vStringObjPrintf(string &aStringObj, const char *aFormat, bool aAppend, va_
   #endif
   {
     // small default buffer was big enough, add it
-    if (actualsize<0) return; // abort, error
+    if (actualsize<0) goto done; // abort, error
     aStringObj += buf;
   }
+ done:
   #ifndef NO_VSNPRINTF
   va_end(args);
   #endif