Changeset 70558 in vbox for trunk/src

Timestamp:

Jan 12, 2018 2:11:39 PM (7 years ago)

Author:

vboxsync

Message:

gccplugin: Added testcase for missing 's' bug - need to validate format types.

Location:

trunk/src/bldprogs

Files:

• VBoxCompilerPlugInsGcc.cpp (modified) (1 diff)
• test-gccplugin-2.c (modified) (1 diff)
• test-gccplugin-3.c (added)

trunk/src/bldprogs/VBoxCompilerPlugInsGcc.cpp

@@ -696 +696 @@
                     State.hFmtLoc       = gimple_location(hStmt);
                     dprintf("     %s() __iprt_format%s__(iFmt=%ld, iArgs=%ld)\n",
-                            DECL_NAME(hFnDecl) ? IDENTIFIER_POINTER(DECL_NAME(hFnDecl)) : "<unamed>",
+                            hFnDecl && DECL_NAME(hFnDecl) ? IDENTIFIER_POINTER(DECL_NAME(hFnDecl)) : "<unamed>",
                             State.fMaybeNull ? "_maybe_null" : "", State.iFmt, State.iArgs);
 

trunk/src/bldprogs/test-gccplugin-2.c

@@ -18 +18 @@
 /* Only valid stuff in this one. */
 extern void MyIprtPrintf(const char *pszFormat, ...) __attribute__((__iprt_format__(1,2)));
-extern void foo(void);
+extern void foo(const char *pszFormat, ...);
 
 extern unsigned long long g_ull;