aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPavel TvrdĂ­k <pawel.tvrdik@gmail.com>2016-03-29 10:37:31 +0200
committerPavel Tvrdik <pawel.tvrdik@gmail.com>2016-06-27 15:07:50 +0200
commit8f01879c5629bd714dfeec968337cfcd4dbe6a87 (patch)
tree0317788989a79a120abbae2443ffd7e2dab68454
parent122deb6d5b14f46f3cfb25bf3f5726005d6a3b3e (diff)
downloadbird-8f01879c5629bd714dfeec968337cfcd4dbe6a87.tar.gz
cppcheck: fix va_end() functions
-rw-r--r--client/util.c2
-rw-r--r--conf/conf.c1
-rw-r--r--sysdep/unix/log.c2
3 files changed, 5 insertions, 0 deletions
diff --git a/client/util.c b/client/util.c
index c35cf8f4..2d6c074d 100644
--- a/client/util.c
+++ b/client/util.c
@@ -40,6 +40,7 @@ bug(const char *msg, ...)
fputs("Internal error: ", stderr);
vlog(msg, args);
vfprintf(stderr, msg, args);
+ va_end(args);
exit(1);
}
@@ -51,5 +52,6 @@ die(const char *msg, ...)
va_start(args, msg);
cleanup();
vlog(msg, args);
+ va_end(args);
exit(1);
}
diff --git a/conf/conf.c b/conf/conf.c
index 825a8e9f..efaeedeb 100644
--- a/conf/conf.c
+++ b/conf/conf.c
@@ -512,6 +512,7 @@ cf_error(char *msg, ...)
va_start(args, msg);
if (bvsnprintf(buf, sizeof(buf), msg, args) < 0)
strcpy(buf, "<bug: error message too long>");
+ va_end(args);
new_config->err_msg = cfg_strdup(buf);
new_config->err_lino = ifs->lino;
new_config->err_file_name = ifs->file_name;
diff --git a/sysdep/unix/log.c b/sysdep/unix/log.c
index 6665d035..631bd691 100644
--- a/sysdep/unix/log.c
+++ b/sysdep/unix/log.c
@@ -209,6 +209,7 @@ bug(const char *msg, ...)
va_start(args, msg);
vlog(L_BUG[0], msg, args);
+ va_end(args);
abort();
}
@@ -226,6 +227,7 @@ die(const char *msg, ...)
va_start(args, msg);
vlog(L_FATAL[0], msg, args);
+ va_end(args);
exit(1);
}