From 487413c0584081de05a3fd794f30d03de99c00b0 Mon Sep 17 00:00:00 2001 From: Rich Megginson Date: Thu, 1 Jul 2010 16:31:27 -0600 Subject: Bug 610276 - fix coverity Defect Type: API usage errors issues https://bugzilla.redhat.com/show_bug.cgi?id=610276 Resolves: bug 610276 Bug Description: fix coverity Defect Type: API usage errors issues Reviewed by: nhosoi (Thanks!) Branch: HEAD Fix Description: Must call va_end after calling va_start. Platforms tested: RHEL5 x86_64 Flag Day: no Doc impact: no --- lib/base/util.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'lib/base/util.cpp') diff --git a/lib/base/util.cpp b/lib/base/util.cpp index 7e9bdbdf..d3a7c330 100644 --- a/lib/base/util.cpp +++ b/lib/base/util.cpp @@ -121,9 +121,12 @@ NSAPI_PUBLIC int util_vsnprintf(char *s, int n, register const char *fmt, NSAPI_PUBLIC int util_snprintf(char *s, int n, const char *fmt, ...) { + int rc; va_list args; va_start(args, fmt); - return PR_vsnprintf(s, n, fmt, args); + rc = PR_vsnprintf(s, n, fmt, args); + va_end(args); + return rc; } NSAPI_PUBLIC int util_vsprintf(char *s, register const char *fmt, va_list args) @@ -133,9 +136,12 @@ NSAPI_PUBLIC int util_vsprintf(char *s, register const char *fmt, va_list args) NSAPI_PUBLIC int util_sprintf(char *s, const char *fmt, ...) { + int rc; va_list args; va_start(args, fmt); - return PR_vsnprintf(s, UTIL_PRF_MAXSIZE, fmt, args); + rc = PR_vsnprintf(s, UTIL_PRF_MAXSIZE, fmt, args); + va_end(args); + return rc; } /* --------------------------- util_strcasecmp ---------------------------- */ -- cgit