[PATCH 26/26] kernel32: Do not use a function pointer as string (Coverity)

Marcus Meissner marcus at jet.franken.de
Sun May 29 16:31:03 CDT 2011


Hi,

CID 1148

We might be trying to print this string. E.g. +relay does print it.
So just pass in a string and not a function pointer.

Ciao, Marcus
---
 dlls/kernel32/tests/locale.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/kernel32/tests/locale.c b/dlls/kernel32/tests/locale.c
index 1268c1d..e13247e 100644
--- a/dlls/kernel32/tests/locale.c
+++ b/dlls/kernel32/tests/locale.c
@@ -2440,13 +2440,13 @@ static void test_SetLocaleInfoA(void)
 
   /* IDATE */
   SetLastError(0);
-  bRet = SetLocaleInfoA(lcid, LOCALE_IDATE, (LPSTR)test_SetLocaleInfoA);
+  bRet = SetLocaleInfoA(lcid, LOCALE_IDATE, "test_SetLocaleInfoA");
   ok(!bRet && GetLastError() == ERROR_INVALID_FLAGS,
      "Expected ERROR_INVALID_FLAGS, got %d\n", GetLastError());
 
   /* ILDATE */
   SetLastError(0);
-  bRet = SetLocaleInfoA(lcid, LOCALE_ILDATE, (LPSTR)test_SetLocaleInfoA);
+  bRet = SetLocaleInfoA(lcid, LOCALE_ILDATE, "test_SetLocaleInfoA");
   ok(!bRet && GetLastError() == ERROR_INVALID_FLAGS,
      "Expected ERROR_INVALID_FLAGS, got %d\n", GetLastError());
 }
-- 
1.7.3.4




More information about the wine-patches mailing list