Daniel Lehman : advapi32/tests: Add tests for REG_NOTIFY_THREAD_AGNOSTIC.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 19e96b567fad85a058824197c39a57c8d61901fa
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=19e96b567fad85a058824197c39a57c8d61901fa

Author: Daniel Lehman <dlehman25 at gmail.com>
Date:   Sun Apr 29 15:42:35 2018 -0700

advapi32/tests: Add tests for REG_NOTIFY_THREAD_AGNOSTIC.

Signed-off-by: Daniel Lehman <dlehman at esri.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/advapi32/tests/registry.c | 81 ++++++++++++++++++++++++++++++++++++++++++
 include/winnt.h                |  1 +
 2 files changed, 82 insertions(+)

diff --git a/dlls/advapi32/tests/registry.c b/dlls/advapi32/tests/registry.c
index 2f7566c..6d8f90a 100644
--- a/dlls/advapi32/tests/registry.c
+++ b/dlls/advapi32/tests/registry.c
@@ -3464,9 +3464,28 @@ static void test_RegOpenCurrentUser(void)
     RegCloseKey(key);
 }
 
+struct notify_data {
+    HKEY key;
+    DWORD flags;
+    HANDLE event;
+};
+
+static DWORD WINAPI notify_change_thread(void *arg)
+{
+    struct notify_data *data = arg;
+    LONG ret;
+
+    ret = RegNotifyChangeKeyValue(data->key, TRUE,
+            REG_NOTIFY_CHANGE_NAME|data->flags, data->event, TRUE);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+    return 0;
+}
+
 static void test_RegNotifyChangeKeyValue(void)
 {
+    struct notify_data data;
     HKEY key, subkey;
+    HANDLE thread;
     HANDLE event;
     DWORD dwret;
     LONG ret;
@@ -3486,6 +3505,68 @@ static void test_RegNotifyChangeKeyValue(void)
     dwret = WaitForSingleObject(event, 0);
     ok(dwret == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", dwret);
 
+    RegDeleteKeyA(key, "SubKey");
+    RegCloseKey(key);
+
+    /* test same thread with REG_NOTIFY_THREAD_AGNOSTIC */
+    ret = RegOpenKeyA(hkey_main, "TestKey", &key);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+    ret = RegNotifyChangeKeyValue(key, TRUE, REG_NOTIFY_CHANGE_NAME|REG_NOTIFY_THREAD_AGNOSTIC,
+            event, TRUE);
+    if (ret == ERROR_INVALID_PARAMETER)
+    {
+        win_skip("REG_NOTIFY_THREAD_AGNOSTIC is not supported\n");
+        RegCloseKey(key);
+        CloseHandle(event);
+        return;
+    }
+
+    ret = RegCreateKeyA(key, "SubKey", &subkey);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+    dwret = WaitForSingleObject(event, 0);
+    ok(dwret == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", dwret);
+
+    RegDeleteKeyA(key, "SubKey");
+    RegCloseKey(key);
+
+    /* test different thread without REG_NOTIFY_THREAD_AGNOSTIC */
+    ret = RegOpenKeyA(hkey_main, "TestKey", &key);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+
+    data.key = key;
+    data.flags = 0;
+    data.event = event;
+    thread = CreateThread(NULL, 0, notify_change_thread, &data, 0, NULL);
+    WaitForSingleObject(thread, INFINITE);
+    CloseHandle(thread);
+
+    /* the thread exiting causes event to signal on Windows
+       this is worked around on Windows using REG_NOTIFY_THREAD_AGNOSTIC
+       Wine already behaves as if the flag is set */
+    dwret = WaitForSingleObject(event, 0);
+    todo_wine
+    ok(dwret == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", dwret);
+    RegCloseKey(key);
+
+    /* test different thread with REG_NOTIFY_THREAD_AGNOSTIC */
+    ret = RegOpenKeyA(hkey_main, "TestKey", &key);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+
+    data.flags = REG_NOTIFY_THREAD_AGNOSTIC;
+    thread = CreateThread(NULL, 0, notify_change_thread, &data, 0, NULL);
+    WaitForSingleObject(thread, INFINITE);
+    CloseHandle(thread);
+
+    dwret = WaitForSingleObject(event, 0);
+    ok(dwret == WAIT_TIMEOUT, "expected WAIT_TIMEOUT, got %u\n", dwret);
+
+    ret = RegCreateKeyA(key, "SubKey", &subkey);
+    ok(ret == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %d\n", ret);
+
+    dwret = WaitForSingleObject(event, 0);
+    ok(dwret == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", dwret);
+
+    RegDeleteKeyA(key, "SubKey");
     RegDeleteKeyA(key, "");
     RegCloseKey(key);
     CloseHandle(event);
diff --git a/include/winnt.h b/include/winnt.h
index 16d96d8..ba10be7 100644
--- a/include/winnt.h
+++ b/include/winnt.h
@@ -5368,6 +5368,7 @@ typedef struct _TAPE_GET_MEDIA_PARAMETERS {
 #define REG_NOTIFY_CHANGE_ATTRIBUTES 0x02
 #define REG_NOTIFY_CHANGE_LAST_SET   0x04
 #define REG_NOTIFY_CHANGE_SECURITY   0x08
+#define REG_NOTIFY_THREAD_AGNOSTIC   0x10000000
 
 #define KEY_QUERY_VALUE		0x00000001
 #define KEY_SET_VALUE		0x00000002