shared: call va_end in all cases

diff --git a/src/shared/log.c b/src/shared/log.c
index 7b0a9144a2bcae4244bdbcab11ee7768983f8f34..b61845859f3cca5300b7750fa812a564cd2c3913 100644 (file)
--- a/src/shared/log.c
+++ b/src/shared/log.c
@@ -719,7 +719,6 @@ int log_struct_internal(
 
                         format = va_arg(ap, char *);
                 }
-                va_end(ap);
 
                 zero(mh);
                 mh.msg_iov = iovec;
@@ -731,6 +730,7 @@ int log_struct_internal(
                         r = 1;
 
         finish:
+                va_end(ap);
                 for (i = 1; i < n; i += 2)
                         free(iovec[i].iov_base);
 

diff --git a/src/shared/util.c b/src/shared/util.c
index be94515d9d8740e852a8fa56ca461676d2456bd6..97f766c33c5b043559472d3b6817d442626c4bc1 100644 (file)
--- a/src/shared/util.c
+++ b/src/shared/util.c
@@ -5024,8 +5024,10 @@ char *strjoin(const char *x, ...) {
                                 break;
 
                         n = strlen(t);
-                        if (n > ((size_t) -1) - l)
+                        if (n > ((size_t) -1) - l) {
+                                va_end(ap);
                                 return NULL;
+                        }
 
                         l += n;
                 }