[ros-diffs] [reactos] 135/158: [WINESYNC] reg/tests: Modify delete_tree() to support the use of registry views.

[winesync]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=44c2f0d9d613224437610…

commit 44c2f0d9d613224437610f44df1a62212c12b77e
Author:     winesync <ros-dev(a)reactos.org>
AuthorDate: Mon Jan 17 18:47:36 2022 +0100
Commit:     Thomas Csovcsity <thc.fr13nd(a)gmail.com>
CommitDate: Sun Jun 19 13:06:37 2022 +0200

    [WINESYNC] reg/tests: Modify delete_tree() to support the use of registry views.
    
    Signed-off-by: Hugh McMaster <hugh.mcmaster(a)outlook.com>
    Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
    
    wine commit id d790c593490bfb363cf34ab2d00dddbaf04ddd15 by Hugh McMaster
<hugh.mcmaster(a)outlook.com>
---
 modules/rostests/winetests/reg/add.c      | 17 +++++++++------
 modules/rostests/winetests/reg/copy.c     | 36 +++++++++++++++----------------
 modules/rostests/winetests/reg/delete.c   |  4 ++--
 modules/rostests/winetests/reg/export.c   |  8 +++----
 modules/rostests/winetests/reg/import.c   | 12 +++++------
 modules/rostests/winetests/reg/query.c    |  6 +++---
 modules/rostests/winetests/reg/reg_test.h |  2 +-
 sdk/tools/winesync/reg.cfg                |  2 +-
 8 files changed, 45 insertions(+), 42 deletions(-)

diff --git a/modules/rostests/winetests/reg/add.c b/modules/rostests/winetests/reg/add.c
index 3b4a895d0c5..bb72c4a3737 100644
--- a/modules/rostests/winetests/reg/add.c
+++ b/modules/rostests/winetests/reg/add.c
@@ -154,7 +154,7 @@ void delete_key_(const char *file, unsigned line, HKEY root, const
char *path, R
     }
 }
 
-LONG delete_tree(const HKEY key, const char *subkey)
+LONG delete_tree(HKEY root, const char *path, REGSAM sam)
 {
     HKEY hkey;
     LONG ret;
@@ -162,7 +162,7 @@ LONG delete_tree(const HKEY key, const char *subkey)
     DWORD max_subkey_len, subkey_len;
     static const char empty[1];
 
-    ret = RegOpenKeyExA(key, subkey, 0, KEY_READ, &hkey);
+    ret = RegOpenKeyExA(root, path, 0, KEY_READ|sam, &hkey);
     if (ret) return ret;
 
     ret = RegQueryInfoKeyA(hkey, NULL, NULL, NULL, NULL, &max_subkey_len,
@@ -184,11 +184,14 @@ LONG delete_tree(const HKEY key, const char *subkey)
         ret = RegEnumKeyExA(hkey, 0, subkey_name, &subkey_len, NULL, NULL, NULL,
NULL);
         if (ret == ERROR_NO_MORE_ITEMS) break;
         if (ret) goto cleanup;
-        ret = delete_tree(hkey, subkey_name);
+        ret = delete_tree(hkey, subkey_name, sam);
         if (ret) goto cleanup;
     }
 
-    ret = RegDeleteKeyA(hkey, empty);
+    if (!sam)
+        ret = RegDeleteKeyA(hkey, empty);
+    else
+        ret = RegDeleteKeyExA(hkey, empty, sam, 0);
 
 cleanup:
     HeapFree(GetProcessHeap(), 0, subkey_name);
@@ -219,7 +222,7 @@ static void test_command_syntax(void)
 {
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     run_reg_exe("reg add", &r);
@@ -322,7 +325,7 @@ static void test_key_formats(void)
     verify_key(hkey, "https://winehq.org", 0);
 
     close_key(hkey);
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Test validity of trailing backslash after system key */
     run_reg_exe("reg add HKCU\\ /v Value1 /t REG_SZ /d foo /f", &r);
@@ -409,7 +412,7 @@ static void test_add(void)
     verify_reg(hkey, NULL, REG_NONE, "T\0e\0s\0t\0\0", 10, 0);
 
     close_key(hkey);
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 }
 
 static void test_reg_none(void)
diff --git a/modules/rostests/winetests/reg/copy.c
b/modules/rostests/winetests/reg/copy.c
index 2b461c17f5e..c6a83e57276 100644
--- a/modules/rostests/winetests/reg/copy.c
+++ b/modules/rostests/winetests/reg/copy.c
@@ -126,10 +126,10 @@ static void test_copy_empty_key(void)
 {
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, NULL);
@@ -191,10 +191,10 @@ static void test_copy_simple_data(void)
     HKEY hkey;
     DWORD r, dword;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
@@ -261,10 +261,10 @@ static void test_copy_complex_data(void)
     HKEY hkey, subkey;
     DWORD r, dword;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
@@ -330,10 +330,10 @@ static void test_copy_key_order(void)
     HKEY hkey;
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
@@ -355,10 +355,10 @@ static void test_copy_value_order(void)
     HKEY hkey;
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
@@ -380,10 +380,10 @@ static void test_copy_hex_data(void)
     HKEY hkey;
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Try copying empty hex values */
@@ -449,10 +449,10 @@ static void test_copy_embedded_null_values(void)
 {
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     test_import_wstr("\xef\xbb\xbfWindows Registry Editor Version 5.00\n\n"
@@ -482,10 +482,10 @@ static void test_copy_slashes(void)
     HKEY hkey;
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
@@ -508,10 +508,10 @@ static void test_copy_escaped_null_values(void)
     HKEY hkey;
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, COPY_SRC);
+    delete_tree(HKEY_CURRENT_USER, COPY_SRC, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, COPY_SRC, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     add_key(HKEY_CURRENT_USER, COPY_SRC, 0, &hkey);
diff --git a/modules/rostests/winetests/reg/delete.c
b/modules/rostests/winetests/reg/delete.c
index 251a15f3371..e40c0256925 100644
--- a/modules/rostests/winetests/reg/delete.c
+++ b/modules/rostests/winetests/reg/delete.c
@@ -23,7 +23,7 @@ static void test_command_syntax(void)
 {
     DWORD r;
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     run_reg_exe("reg delete", &r);
@@ -74,7 +74,7 @@ static void test_delete(void)
     DWORD r;
     const DWORD deadbeef = 0xdeadbeef;
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Create a test key */
diff --git a/modules/rostests/winetests/reg/export.c
b/modules/rostests/winetests/reg/export.c
index df44baddb30..0f48443384e 100644
--- a/modules/rostests/winetests/reg/export.c
+++ b/modules/rostests/winetests/reg/export.c
@@ -176,7 +176,7 @@ static void test_export(void)
     HKEY hkey, subkey;
     BYTE hex[4], buffer[8];
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     run_reg_exe("reg export", &r);
@@ -287,7 +287,7 @@ static void test_export(void)
     run_reg_exe("reg export HKEY_CURRENT_USER\\" KEY_BASE " file.reg
/y", &r);
     ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r);
     ok(compare_export("file.reg", complex_data_test, 0), "compare_export()
failed\n");
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Test the export order of registry keys */
     add_key(HKEY_CURRENT_USER, KEY_BASE, 0, &hkey);
@@ -413,7 +413,7 @@ static void test_export(void)
     run_reg_exe("reg export HKEY_CURRENT_USER\\" KEY_BASE " file.reg
/y", &r);
     ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r);
     ok(compare_export("file.reg", slashes_test, TODO_REG_COMPARE),
"compare_export() failed\n");
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Test escaped null characters */
     add_key(HKEY_CURRENT_USER, KEY_BASE, 0, &hkey);
@@ -428,7 +428,7 @@ static void test_export(void)
     run_reg_exe("reg export HKEY_CURRENT_USER\\" KEY_BASE " file.reg
/y", &r);
     ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r);
     ok(compare_export("file.reg", escaped_null_test, 0), "compare_export()
failed\n");
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 }
 
 START_TEST(export)
diff --git a/modules/rostests/winetests/reg/import.c
b/modules/rostests/winetests/reg/import.c
index 2d8ece8990b..2531ba336e8 100644
--- a/modules/rostests/winetests/reg/import.c
+++ b/modules/rostests/winetests/reg/import.c
@@ -101,7 +101,7 @@ static void test_import(void)
     LONG err;
     BYTE hex[8];
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     run_reg_exe("reg import", &r);
@@ -1643,7 +1643,7 @@ static void test_import(void)
 
     close_key(hkey);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 }
 
 static void test_unicode_import(void)
@@ -1654,7 +1654,7 @@ static void test_unicode_import(void)
     char buffer[24];
     BYTE hex[8];
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     test_import_wstr("REGEDIT\n", &r);
@@ -3192,7 +3192,7 @@ static void test_unicode_import(void)
 
     close_key(hkey);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 }
 
 static void test_import_with_whitespace(void)
@@ -3200,7 +3200,7 @@ static void test_import_with_whitespace(void)
     HKEY hkey;
     DWORD r, dword;
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     test_import_str("  REGEDIT4\n\n"
@@ -3346,7 +3346,7 @@ static void test_unicode_import_with_whitespace(void)
     HKEY hkey;
     DWORD r, dword;
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     test_import_wstr("\xef\xbb\xbf  Windows Registry Editor Version 5.00\n\n"
diff --git a/modules/rostests/winetests/reg/query.c
b/modules/rostests/winetests/reg/query.c
index 3e8b134363b..270f0aad75d 100644
--- a/modules/rostests/winetests/reg/query.c
+++ b/modules/rostests/winetests/reg/query.c
@@ -164,7 +164,7 @@ static void test_query(void)
     HKEY hkey, subkey;
     BYTE buf[512];
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
     verify_key_nonexist(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     run_reg_exe("reg query", &r);
@@ -270,7 +270,7 @@ static void test_query(void)
     ok(r == REG_EXIT_SUCCESS || r == REG_EXIT_FAILURE /* WinXP */,
        "got exit code %d, expected 0\n", r);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 
     /* Subkeys only */
     add_key(HKEY_CURRENT_USER, KEY_BASE, 0, &hkey);
@@ -288,7 +288,7 @@ static void test_query(void)
     ok(r == REG_EXIT_SUCCESS, "got exit code %d, expected 0\n", r);
     compare_query(buf, test8b, FALSE, 0);
 
-    delete_tree(HKEY_CURRENT_USER, KEY_BASE);
+    delete_tree(HKEY_CURRENT_USER, KEY_BASE, 0);
 }
 
 START_TEST(query)
diff --git a/modules/rostests/winetests/reg/reg_test.h
b/modules/rostests/winetests/reg/reg_test.h
index fdc747af1e9..a5d7169beb4 100644
--- a/modules/rostests/winetests/reg/reg_test.h
+++ b/modules/rostests/winetests/reg/reg_test.h
@@ -64,7 +64,7 @@ void add_key_(const char *file, unsigned line, const HKEY root, const
char *path
 #define delete_key(r,p,s) delete_key_(__FILE__,__LINE__,r,p,s)
 void delete_key_(const char *file, unsigned line, HKEY root, const char *path, REGSAM
sam);
 
-LONG delete_tree(const HKEY key, const char *subkey);
+LONG delete_tree(HKEY root, const char *path, REGSAM sam);
 
 #define add_value(k,n,t,d,s) add_value_(__FILE__,__LINE__,k,n,t,d,s)
 void add_value_(const char *file, unsigned line, HKEY hkey, const char *name,
diff --git a/sdk/tools/winesync/reg.cfg b/sdk/tools/winesync/reg.cfg
index 9ab5d25993d..079df8ba158 100644
--- a/sdk/tools/winesync/reg.cfg
+++ b/sdk/tools/winesync/reg.cfg
@@ -4,4 +4,4 @@ directories:
 files:
   programs/reg/resource.h: base/applications/cmdutils/reg/resource.h
 tags:
-  wine: 7c9f372480511cba8abd90ed04480911d96036de
+  wine: d790c593490bfb363cf34ab2d00dddbaf04ddd15