Francois Gouget : user32: GetClipboardData() should set last error when the format is not found.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 25d6c1a3b497c1530b4ed2c769ec5728b7c006bc
URL:    https://gitlab.winehq.org/wine/wine/-/commit/25d6c1a3b497c1530b4ed2c769ec572...

Author: Francois Gouget 
Date:   Tue Dec 20 15:33:56 2022 +0100

user32: GetClipboardData() should set last error when the format is not found.

This has the benefit of indicating why GetClipboardData() failed and of
matching the Windows 11 behavior.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54190

---

dlls/user32/tests/clipboard.c | 8 ++++++--
 dlls/win32u/clipboard.c       | 6 +++++-
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/dlls/user32/tests/clipboard.c b/dlls/user32/tests/clipboard.c
index 3131c2e0e8c..7414af996fb 100644
--- a/dlls/user32/tests/clipboard.c
+++ b/dlls/user32/tests/clipboard.c
@@ -986,7 +986,9 @@ static void test_synthesized(void)

SetLastError(0xdeadbeef);
     data = GetClipboardData( CF_TEXT );
-    ok(GetLastError() == 0xdeadbeef, "bad last error %ld\n", GetLastError());
+    ok(GetLastError() == ERROR_NOT_FOUND /* win11 */ ||
+       broken(GetLastError() == 0xdeadbeef),
+       "bad last error %ld\n", GetLastError());
     ok(!data, "GetClipboardData() should have returned NULL\n");

r = CloseClipboard();
@@ -1587,7 +1589,9 @@ static void test_handles( HWND hwnd )

SetLastError( 0xdeadbeef );
     data = GetClipboardData( CF_RIFF );
-    ok( GetLastError() == 0xdeadbeef, "unexpected last error %ld\n", GetLastError() );
+    ok( GetLastError() == ERROR_NOT_FOUND /* win11 */ ||
+        broken(GetLastError() == 0xdeadbeef),
+        "unexpected last error %ld\n", GetLastError() );
     ok( !data, "wrong data %p\n", data );

h = SetClipboardData( CF_PRIVATEFIRST + 7, htext4 );
diff --git a/dlls/win32u/clipboard.c b/dlls/win32u/clipboard.c
index d53cd966d36..6cf484a56ca 100644
--- a/dlls/win32u/clipboard.c
+++ b/dlls/win32u/clipboard.c
@@ -720,7 +720,11 @@ HANDLE WINAPI NtUserGetClipboardData( UINT format, struct get_clipboard_params *
             params->data_size = size;
             return 0;
         }
-        if (status == STATUS_OBJECT_NAME_NOT_FOUND) return 0; /* no such format */
+        if (status == STATUS_OBJECT_NAME_NOT_FOUND)
+        {
+            RtlSetLastWin32Error( ERROR_NOT_FOUND ); /* no such format */
+            return 0;
+        }
         if (status)
         {
             RtlSetLastWin32Error( RtlNtStatusToDosError( status ));