[PATCH] user32/tests: Use wine_dbgstr_a() instead of %.8s.

Francois Gouget fgouget at codeweavers.com
Sat Aug 7 17:46:38 CDT 2021


This should better handle NULL pointers and bad strings.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
Maybe wine_dbgstr_an(,7) would be even better.
Not very important either way.
---
 dlls/user32/tests/clipboard.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/user32/tests/clipboard.c b/dlls/user32/tests/clipboard.c
index 3e89e679927..bf35a224940 100644
--- a/dlls/user32/tests/clipboard.c
+++ b/dlls/user32/tests/clipboard.c
@@ -1973,7 +1973,7 @@ static void test_data_handles(void)
     ok( is_fixed( h ), "expected fixed mem %p\n", h );
     ok( is_moveable( text ), "expected moveable mem %p\n", text );
     ptr = GlobalLock( h );
-    ok( !strcmp( ptr, "foobar" ), "wrong data '%.8s'\n", ptr );
+    ok( ptr && !strcmp( ptr, "foobar" ), "wrong data %s\n", wine_dbgstr_a(ptr) );
     GlobalUnlock( h );
 
     r = EmptyClipboard();
-- 
2.20.1



More information about the wine-devel mailing list