Date: Tue, 8 Dec 2015 04:45:45 +0000 (UTC) From: Garrett Cooper <ngie@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r291980 - head/lib/libc/tests/stdio Message-ID: <201512080445.tB84jjrU019909@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
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);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201512080445.tB84jjrU019909>