svn commit: r291980 - head/lib/libc/tests/stdio

Garrett Cooper ngie at FreeBSD.org
Tue Dec 8 04:45:46 UTC 2015


Author: ngie
Date: Tue Dec  8 04:45:44 2015
New Revision: 291980
URL: https://svnweb.freebsd.org/changeset/base/291980

Log:
  Add missing va_ends for corresponding va_starts to clean up variable arguments
  initialized in _test_fmt(..)
  
  MFC after: 3 days
  Reported by: cppcheck
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/lib/libc/tests/stdio/printbasic_test.c
  head/lib/libc/tests/stdio/printfloat_test.c

Modified: head/lib/libc/tests/stdio/printbasic_test.c
==============================================================================
--- head/lib/libc/tests/stdio/printbasic_test.c	Tue Dec  8 04:40:03 2015	(r291979)
+++ head/lib/libc/tests/stdio/printbasic_test.c	Tue Dec  8 04:45:44 2015	(r291980)
@@ -94,6 +94,8 @@ _testfmt(const char *result, const char 
 		    "wprintf(\"%ls\", %s) ==> [%ls], expected [%ls]\n",
 		    wfmt, argstr, ws, wresult);
 	}
+	va_end(ap);
+	va_end(ap2);
 }
 
 ATF_TC_WITHOUT_HEAD(int_within_limits);

Modified: head/lib/libc/tests/stdio/printfloat_test.c
==============================================================================
--- head/lib/libc/tests/stdio/printfloat_test.c	Tue Dec  8 04:40:03 2015	(r291979)
+++ head/lib/libc/tests/stdio/printfloat_test.c	Tue Dec  8 04:45:44 2015	(r291980)
@@ -86,6 +86,8 @@ _testfmt(const char *result, const char 
 		    "wprintf(\"%ls\", %s) ==> [%ls], expected [%ls]\n",
 		    wfmt, argstr, ws, wresult);
 	}
+	va_end(ap);
+	va_end(ap2);
 }
 
 ATF_TC_WITHOUT_HEAD(float_within_limits);


More information about the svn-src-all mailing list