r19392: Use torture_setting_* rather than lp_parm_* where possible.
[sfrench/samba-autobuild/.git] / source4 / torture / rpc / winreg.c
index 592f2ff86ab2bbad595f23ebe1bff3f1f41846bc..2516c352769d80aa287ff8b9a041cd732ec20c14 100644 (file)
 */
 
 #include "includes.h"
-#include "librpc/gen_ndr/ndr_winreg.h"
+#include "torture/torture.h"
+#include "librpc/gen_ndr/ndr_winreg_c.h"
+#include "librpc/gen_ndr/ndr_security.h"
+#include "libcli/security/security.h"
+#include "torture/rpc/rpc.h"
+
+#define TEST_KEY_BASE "smbtorture test"
+#define TEST_KEY1 TEST_KEY_BASE "\\spottyfoot"
+#define TEST_KEY2 TEST_KEY_BASE "\\with a SD (#1)"
+
+static void init_initshutdown_String(TALLOC_CTX *mem_ctx, struct initshutdown_String *name, const char *s)
+{
+       name->name = talloc(mem_ctx, struct initshutdown_String_sub);
+       name->name->name = s;
+}
 
 static void init_winreg_String(struct winreg_String *name, const char *s)
 {
@@ -40,10 +54,12 @@ static BOOL test_GetVersion(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
 {
        NTSTATUS status;
        struct winreg_GetVersion r;
-
+       uint32_t v;
        printf("\ntesting GetVersion\n");
 
+       ZERO_STRUCT(r);
        r.in.handle = handle;
+       r.out.version = &v;
 
        status = dcerpc_winreg_GetVersion(p, mem_ctx, &r);
 
@@ -60,6 +76,36 @@ static BOOL test_GetVersion(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        return True;
 }
 
+static BOOL test_NotifyChangeKeyValue(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
+                                                                         struct policy_handle *handle)
+{
+       struct winreg_NotifyChangeKeyValue r;
+       NTSTATUS status;
+
+       printf("\ntesting NotifyChangeKeyValue\n");
+
+       r.in.handle = handle;
+       r.in.watch_subtree = 1;
+       r.in.notify_filter = 0;
+       r.in.unknown = r.in.unknown2 = 0;
+       init_winreg_String(&r.in.string1, NULL);
+       init_winreg_String(&r.in.string2, NULL);
+
+       status = dcerpc_winreg_NotifyChangeKeyValue(p, mem_ctx, &r);
+       
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("NotifyChangeKeyValue failed - %s\n", nt_errstr(status));
+               return False;
+       }
+
+       if (!W_ERROR_IS_OK(r.out.result)) {
+               printf("NotifyChangeKeyValue failed - %s - not considering\n", win_errstr(r.out.result));
+               return True;
+       }
+
+       return True;
+}
+
 static BOOL test_CreateKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                          struct policy_handle *handle, const char *name, 
                           const char *class)
@@ -67,18 +113,18 @@ static BOOL test_CreateKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        struct winreg_CreateKey r;
        struct policy_handle newhandle;
        NTSTATUS status;
-       uint32_t sec_info = 0;
+       enum winreg_CreateAction action_taken = 0;
 
        printf("\ntesting CreateKey\n");
 
        r.in.handle = handle;
-       r.out.handle = &newhandle;
-       init_winreg_String(&r.in.key, name);    
-       init_winreg_String(&r.in.class, class);
-       r.in.reserved = 0x0;
-       r.in.access_mask = 0x02000000;
-       r.in.sec_info = &sec_info;
-       r.in.sec_desc = NULL;
+       r.out.new_handle = &newhandle;
+       init_winreg_String(&r.in.name, name);   
+       init_winreg_String(&r.in.keyclass, class);
+       r.in.options = 0x0;
+       r.in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
+       r.in.action_taken = r.out.action_taken = &action_taken;
+       r.in.secdesc = NULL;
 
        status = dcerpc_winreg_CreateKey(p, mem_ctx, &r);
 
@@ -95,6 +141,113 @@ static BOOL test_CreateKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        return True;
 }
 
+
+/*
+  createkey testing with a SD
+*/
+static BOOL test_CreateKey_sd(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
+                             struct policy_handle *handle, const char *name, 
+                             const char *class, struct policy_handle *newhandle)
+{
+       struct winreg_CreateKey r;
+       NTSTATUS status;
+       enum winreg_CreateAction action_taken = 0;
+       struct security_descriptor *sd;
+       DATA_BLOB sdblob;
+       struct winreg_SecBuf secbuf;
+
+       sd = security_descriptor_create(mem_ctx,
+                                       NULL, NULL,
+                                       SID_NT_AUTHENTICATED_USERS,
+                                       SEC_ACE_TYPE_ACCESS_ALLOWED,
+                                       SEC_GENERIC_ALL,
+                                       SEC_ACE_FLAG_OBJECT_INHERIT,
+                                       NULL);
+
+       status = ndr_push_struct_blob(&sdblob, mem_ctx, sd, 
+                                     (ndr_push_flags_fn_t)ndr_push_security_descriptor);
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("Failed to push security_descriptor ?!\n");
+               return False;
+       }
+
+       secbuf.sd.data = sdblob.data;
+       secbuf.sd.len = sdblob.length;
+       secbuf.sd.size = sdblob.length;
+       secbuf.length = sdblob.length-10;
+       secbuf.inherit = 0;
+
+       printf("\ntesting CreateKey with sd\n");
+
+       r.in.handle = handle;
+       r.out.new_handle = newhandle;
+       init_winreg_String(&r.in.name, name);   
+       init_winreg_String(&r.in.keyclass, class);
+       r.in.options = 0x0;
+       r.in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
+       r.in.action_taken = r.out.action_taken = &action_taken;
+       r.in.secdesc = &secbuf;
+
+       status = dcerpc_winreg_CreateKey(p, mem_ctx, &r);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("CreateKey with sd failed - %s\n", nt_errstr(status));
+               return False;
+       }
+
+       if (!W_ERROR_IS_OK(r.out.result)) {
+               printf("CreateKey with sd failed - %s\n", win_errstr(r.out.result));
+               return False;
+       }
+
+       return True;
+}
+
+static BOOL test_GetKeySecurity(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
+                         struct policy_handle *handle)
+{
+       NTSTATUS status;
+       struct winreg_GetKeySecurity r;
+       struct security_descriptor sd;
+       DATA_BLOB sdblob;
+
+       printf("\ntesting GetKeySecurity\n");
+
+       ZERO_STRUCT(r);
+
+       r.in.handle = handle;
+       r.in.sd = r.out.sd = talloc_zero(mem_ctx, struct KeySecurityData);
+       r.in.sd->size = 0x1000;
+       r.in.sec_info = SECINFO_OWNER | SECINFO_GROUP | SECINFO_DACL;
+
+       status = dcerpc_winreg_GetKeySecurity(p, mem_ctx, &r);
+
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("GetKeySecurity failed - %s\n", nt_errstr(status));
+               return False;
+       }
+
+       if (!W_ERROR_IS_OK(r.out.result)) {
+               printf("GetKeySecurity failed - %s\n", win_errstr(r.out.result));
+               return False;
+       }
+
+       sdblob.data = r.out.sd->data;
+       sdblob.length = r.out.sd->len;
+
+       status = ndr_pull_struct_blob(&sdblob, mem_ctx, &sd, 
+                                     (ndr_pull_flags_fn_t)ndr_pull_security_descriptor);
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("pull_security_descriptor failed - %s\n", nt_errstr(status));
+               return False;
+       }
+       if (p->conn->flags & DCERPC_DEBUG_PRINT_OUT) {
+               NDR_PRINT_DEBUG(security_descriptor, &sd);
+       }
+
+       return True;
+}
+
 static BOOL test_CloseKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
                          struct policy_handle *handle)
 {
@@ -154,10 +307,10 @@ static BOOL test_OpenKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
 
        printf("\ntesting OpenKey\n");
 
-       r.in.handle = hive_handle;
+       r.in.parent_handle = hive_handle;
        init_winreg_String(&r.in.keyname, keyname);
        r.in.unknown = 0x00000000;
-       r.in.access_mask = 0x02000000;
+       r.in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
        r.out.handle = key_handle;
 
        status = dcerpc_winreg_OpenKey(p, mem_ctx, &r);
@@ -176,6 +329,20 @@ static BOOL test_OpenKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        return True;
 }
 
+static BOOL test_Cleanup(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
+                        struct policy_handle *handle, const char *key)
+{
+       struct winreg_DeleteKey r;
+
+       r.in.handle = handle;
+
+       init_winreg_String(&r.in.key, key);
+       dcerpc_winreg_DeleteKey(p, mem_ctx, &r);
+
+       return True;
+}
+
+
 static BOOL test_DeleteKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
                           struct policy_handle *handle, const char *key)
 {
@@ -207,11 +374,25 @@ static BOOL test_QueryInfoKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
 {
        NTSTATUS status;
        struct winreg_QueryInfoKey r;
+       uint32_t num_subkeys, max_subkeylen, max_subkeysize,
+               num_values, max_valnamelen, max_valbufsize,
+               secdescsize;
+       NTTIME last_changed_time;
 
        printf("\ntesting QueryInfoKey\n");
 
+       ZERO_STRUCT(r);
        r.in.handle = handle;
-       init_winreg_String(&r.in.class, class);
+       r.out.num_subkeys = &num_subkeys;
+       r.out.max_subkeylen = &max_subkeylen;
+       r.out.max_subkeysize = &max_subkeysize;
+       r.out.num_values = &num_values;
+       r.out.max_valnamelen = &max_valnamelen;
+       r.out.max_valbufsize = &max_valbufsize;
+       r.out.secdescsize = &secdescsize;
+       r.out.last_changed_time = &last_changed_time;
+       
+       init_winreg_String(&r.in.class_in, class);
        
        status = dcerpc_winreg_QueryInfoKey(p, mem_ctx, &r);
 
@@ -235,35 +416,35 @@ static BOOL test_EnumKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                         struct policy_handle *handle, int depth)
 {
        struct winreg_EnumKey r;
-       struct winreg_EnumKeyNameRequest keyname;
-       struct winreg_String classname;
-       struct winreg_Time tm;
+       struct winreg_StringBuf class, name;
        NTSTATUS status;
+       NTTIME t = 0;
 
        printf("Testing EnumKey\n\n");
 
+       class.name   = "";
+       class.size   = 1024;
+
        r.in.handle = handle;
        r.in.enum_index = 0;
-       r.in.key_name_len = r.out.key_name_len = 0;
-       r.in.unknown = r.out.unknown = 0x0414;
-       keyname.unknown = 0x0000020a;
-       init_winreg_String(&keyname.key_name, NULL);
-       init_winreg_String(&classname, NULL);
-       r.in.in_name = &keyname;
-       r.in.class = &classname;
-       tm.low = tm.high = 0x7fffffff;
-       r.in.last_changed_time = &tm;
+       r.in.name = &name;
+       r.in.keyclass = &class;
+       r.out.name = &name;
+       r.in.last_changed_time = &t;
 
        do {
+               name.name   = NULL;
+               name.size   = 1024;
+
                status = dcerpc_winreg_EnumKey(p, mem_ctx, &r);
 
                if (NT_STATUS_IS_OK(status) && W_ERROR_IS_OK(r.out.result)) {
                        struct policy_handle key_handle;
 
-                       printf("EnumKey: %d: %s\n", r.in.enum_index, r.out.out_name->name);
+                       printf("EnumKey: %d: %s\n", r.in.enum_index, r.out.name->name);
 
                        if (!test_OpenKey(
-                                   p, mem_ctx, handle, r.out.out_name->name,
+                                   p, mem_ctx, handle, r.out.name->name,
                                    &key_handle)) {
                        } else {
                                test_key(p, mem_ctx, &key_handle, depth + 1);
@@ -289,21 +470,62 @@ static BOOL test_EnumKey(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        return True;
 }
 
+static BOOL test_QueryMultipleValues(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, struct policy_handle *handle, const char *valuename)
+{
+       struct winreg_QueryMultipleValues r;
+       NTSTATUS status;
+       uint32_t bufsize=0;
+
+       printf("Testing QueryMultipleValues\n");
+
+       r.in.key_handle = handle;
+       r.in.values = r.out.values = talloc_array(mem_ctx, struct QueryMultipleValue, 1);
+       r.in.values[0].name = talloc(mem_ctx, struct winreg_String);
+       r.in.values[0].name->name = valuename;
+       r.in.values[0].offset = 0;
+       r.in.values[0].length = 0;
+       r.in.values[0].type = 0;
+
+       r.in.num_values = 1;
+       r.in.buffer_size = r.out.buffer_size = talloc(mem_ctx, uint32_t);
+       *r.in.buffer_size = bufsize;
+       do { 
+               *r.in.buffer_size = bufsize;
+               r.in.buffer = r.out.buffer = talloc_zero_array(mem_ctx, uint8_t, 
+                                                              *r.in.buffer_size);
+
+               status = dcerpc_winreg_QueryMultipleValues(p, mem_ctx, &r);
+       
+               if(NT_STATUS_IS_ERR(status)) {
+                       printf("QueryMultipleValues failed - %s\n", nt_errstr(status));
+                       return False;
+               }
+               talloc_free(r.in.buffer);
+               bufsize += 0x20;
+       } while (W_ERROR_EQUAL(r.out.result, WERR_MORE_DATA));
+
+       if (!W_ERROR_IS_OK(r.out.result)) {
+               printf("QueryMultipleValues failed - %s\n", win_errstr(r.out.result));
+               return False;
+       }
+
+       return True;
+}
+
 static BOOL test_QueryValue(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, struct policy_handle *handle, const char *valuename)
 {
        struct winreg_QueryValue r;
        NTSTATUS status;
-       struct EnumValueNameOut valname;
-       uint32 zero = 0;
-       uint32 offered = 0xfff;
-
-       valname.name = valuename;
+       enum winreg_Type zero_type = 0;
+       uint32_t offered = 0xfff;
+       uint32_t zero = 0;
 
        printf("Testing QueryValue\n");
 
        r.in.handle = handle;
+       r.in.data = NULL;
        r.in.value_name.name = valuename;
-       r.in.type = &zero;
+       r.in.type = &zero_type;
        r.in.size = &offered;
        r.in.length = &zero;
 
@@ -325,29 +547,25 @@ static BOOL test_EnumValue(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                           struct policy_handle *handle, int max_valnamelen, int max_valbufsize)
 {
        struct winreg_EnumValue r;
-       struct EnumValueIn buf_name;
-       struct EnumValueIn buf_val;
-       uint32 type;
-       uint32 len1 = max_valbufsize, len2 = 0;
+       enum winreg_Type type = 0;
+       uint32_t size = max_valbufsize, zero = 0;
        BOOL ret = True;
+       uint8_t buf8;
+       struct winreg_StringBuf name;
 
        printf("testing EnumValue\n");
 
+       name.name   = "";
+       name.size   = 1024;
+
        r.in.handle = handle;
        r.in.enum_index = 0;
-       r.in.name_in.len = 0;
-       r.in.name_in.max_len = max_valnamelen * 2;
-       buf_name.max_len = max_valnamelen;
-       buf_name.offset = 0;
-       buf_name.len = 0;
-       r.in.name_in.buffer = &buf_name;
+       r.in.name = &name;
+       r.out.name = &name;
        r.in.type = &type;
-       buf_val.max_len = max_valbufsize;
-       buf_val.offset = 0;
-       buf_val.len = 0;
-       r.in.value_in = &buf_val;
-       r.in.value_len1 = &len1;
-       r.in.value_len2 = &len2;
+       r.in.value = &buf8;
+       r.in.length = &zero;
+       r.in.size = &size;
        
        do {
                NTSTATUS status = dcerpc_winreg_EnumValue(p, mem_ctx, &r);
@@ -357,7 +575,8 @@ static BOOL test_EnumValue(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                }
 
                if (W_ERROR_IS_OK(r.out.result)) {
-                       ret &= test_QueryValue(p, mem_ctx, handle, r.out.name_out.name);
+                       ret &= test_QueryValue(p, mem_ctx, handle, r.out.name->name);
+                       ret &= test_QueryMultipleValues(p, mem_ctx, handle, r.out.name->name);
                }
 
                r.in.enum_index++;
@@ -371,108 +590,59 @@ static BOOL test_EnumValue(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        return ret;
 }
 
-static BOOL test_OpenHKLM(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
-                         struct policy_handle *handle)
+static BOOL test_InitiateSystemShutdown(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
+                       const char *msg, uint32_t timeout)
 {
+       struct winreg_InitiateSystemShutdown r;
        NTSTATUS status;
-       struct winreg_OpenHKLM r;
-       struct winreg_OpenUnknown unknown;
-       BOOL ret = True;
-
-       printf("\ntesting OpenHKLM\n");
-
-       unknown.unknown0 = 0x84e0;
-       unknown.unknown1 = 0x0000;
-       r.in.unknown = &unknown;
-       r.in.access_required = SEC_FLAG_MAXIMUM_ALLOWED;
-       r.out.handle = handle;
+       uint16_t hostname = 0x0;
+       
+       r.in.hostname = &hostname;
+       r.in.message = talloc(mem_ctx, struct initshutdown_String);
+       init_initshutdown_String(mem_ctx, r.in.message, msg);
+       r.in.force_apps = 1;
+       r.in.timeout = timeout;
+       r.in.reboot = 1;
 
-       status = dcerpc_winreg_OpenHKLM(p, mem_ctx, &r);
+       status = dcerpc_winreg_InitiateSystemShutdown(p, mem_ctx, &r);
 
        if (!NT_STATUS_IS_OK(status)) {
-               printf("OpenHKLM failed - %s\n", nt_errstr(status));
+               printf("InitiateSystemShutdown failed - %s\n", nt_errstr(status));
                return False;
        }
 
        if (!W_ERROR_IS_OK(r.out.result)) {
-               printf("OpenHKLM failed - %s\n", win_errstr(r.out.result));
-               return False;
-       }
-
-       return ret;
-}
-
-static BOOL test_OpenHKU(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
-                        struct policy_handle *handle)
-{
-       NTSTATUS status;
-       struct winreg_OpenHKU r;
-       struct winreg_OpenUnknown unknown;
-       BOOL ret = True;
-
-       printf("\ntesting OpenHKU\n");
-
-       unknown.unknown0 = 0x84e0;
-       unknown.unknown1 = 0x0000;
-       r.in.unknown = &unknown;
-       r.in.access_required = SEC_FLAG_MAXIMUM_ALLOWED;
-       r.out.handle = handle;
-
-       status = dcerpc_winreg_OpenHKU(p, mem_ctx, &r);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               printf("OpenHKU failed - %s\n", nt_errstr(status));
-               return False;
-       }
-
-       return ret;
-}
-
-static BOOL test_OpenHKCR(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
-                         struct policy_handle *handle)
-{
-       NTSTATUS status;
-       struct winreg_OpenHKCR r;
-       struct winreg_OpenUnknown unknown;
-       BOOL ret = True;
-
-       printf("\ntesting OpenHKCR\n");
-
-       unknown.unknown0 = 0x84e0;
-       unknown.unknown1 = 0x0000;
-       r.in.unknown = &unknown;
-       r.in.access_required = SEC_FLAG_MAXIMUM_ALLOWED;
-       r.out.handle = handle;
-
-       status = dcerpc_winreg_OpenHKCR(p, mem_ctx, &r);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               printf("OpenHKCR failed - %s\n", nt_errstr(status));
+               printf("InitiateSystemShutdown failed - %s\n", win_errstr(r.out.result));
                return False;
        }
 
-       return ret;
+       return True;
 }
 
-static BOOL test_InitiateSystemShutdown(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
+static BOOL test_InitiateSystemShutdownEx(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                        const char *msg, uint32_t timeout)
 {
-       struct winreg_InitiateSystemShutdown r;
+       struct winreg_InitiateSystemShutdownEx r;
        NTSTATUS status;
+       uint16_t hostname = 0x0;
        
-       init_winreg_String(&r.in.message, msg);
-       r.in.flags = 0;
+       r.in.hostname = &hostname;
+       r.in.message = talloc(mem_ctx, struct initshutdown_String);
+       init_initshutdown_String(mem_ctx, r.in.message, msg);
+       r.in.force_apps = 1;
        r.in.timeout = timeout;
+       r.in.reboot = 1;
+       r.in.reason = 0;
 
-       status = dcerpc_winreg_InitiateSystemShutdown(p, mem_ctx, &r);
+       status = dcerpc_winreg_InitiateSystemShutdownEx(p, mem_ctx, &r);
 
        if (!NT_STATUS_IS_OK(status)) {
-               printf("InitiateSystemShutdown failed - %s\n", nt_errstr(status));
+               printf("InitiateSystemShutdownEx failed - %s\n", nt_errstr(status));
                return False;
        }
 
        if (!W_ERROR_IS_OK(r.out.result)) {
-               printf("InitiateSystemShutdown failed - %s\n", win_errstr(r.out.result));
+               printf("InitiateSystemShutdownEx failed - %s\n", win_errstr(r.out.result));
                return False;
        }
 
@@ -502,32 +672,6 @@ static BOOL test_AbortSystemShutdown(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx)
        return True;
 }
 
-static BOOL test_OpenHKCU(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
-                         struct policy_handle *handle)
-{
-       NTSTATUS status;
-       struct winreg_OpenHKCU r;
-       struct winreg_OpenUnknown unknown;
-       BOOL ret = True;
-
-       printf("\ntesting OpenHKCU\n");
-
-       unknown.unknown0 = 0x84e0;
-       unknown.unknown1 = 0x0000;
-       r.in.unknown = &unknown;
-       r.in.access_required = SEC_FLAG_MAXIMUM_ALLOWED;
-       r.out.handle = handle;
-
-       status = dcerpc_winreg_OpenHKCU(p, mem_ctx, &r);
-
-       if (!NT_STATUS_IS_OK(status)) {
-               printf("OpenHKCU failed - %s\n", nt_errstr(status));
-               return False;
-       }
-
-       return ret;
-}
-
 #define MAX_DEPTH 2            /* Only go this far down the tree */
 
 static BOOL test_key(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
@@ -539,56 +683,80 @@ static BOOL test_key(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
        if (!test_QueryInfoKey(p, mem_ctx, handle, NULL)) {
        }
 
+
+       if (!test_NotifyChangeKeyValue(p, mem_ctx, handle)) {
+       }
+       
+       if (!test_GetKeySecurity(p, mem_ctx, handle)) {
+       }
+
        if (!test_EnumKey(p, mem_ctx, handle, depth)) {
        }
 
        if (!test_EnumValue(p, mem_ctx, handle, 0xFF, 0xFFFF)) {
        }
 
+
        test_CloseKey(p, mem_ctx, handle);
 
        return True;
 }
 
-typedef BOOL winreg_open_fn(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
-                           struct policy_handle *handle);
+typedef NTSTATUS (*winreg_open_fn)(struct dcerpc_pipe *, TALLOC_CTX *, void *);
 
-static BOOL test_Open(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, void *fn)
+static BOOL test_Open(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, 
+                     const char *name, winreg_open_fn open_fn)
 {
        struct policy_handle handle, newhandle;
-       BOOL ret = True;
-       winreg_open_fn *open_fn = (winreg_open_fn *)fn;
+       BOOL ret = True, created = False, created2 = False, deleted = False;
+       struct winreg_OpenHKLM r;
+       NTSTATUS status;
+
+       printf("Testing %s\n", name);
 
-       if (!open_fn(p, mem_ctx, &handle)) {
+       r.in.system_name = 0;
+       r.in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
+       r.out.handle = &handle;
+       
+       status = open_fn(p, mem_ctx, &r);
+       if (!NT_STATUS_IS_OK(status)) {
                return False;
        }
 
-       if (!test_CreateKey(p, mem_ctx, &handle, "spottyfoot", NULL)) {
-               printf("CreateKey failed\n");
-               ret = False;
+       test_Cleanup(p, mem_ctx, &handle, TEST_KEY1);
+       test_Cleanup(p, mem_ctx, &handle, TEST_KEY2);
+       test_Cleanup(p, mem_ctx, &handle, TEST_KEY_BASE);
+
+       if (!test_CreateKey(p, mem_ctx, &handle, TEST_KEY1, NULL)) {
+               printf("CreateKey failed - not considering a failure\n");
+       } else {
+               created = True;
        }
 
-       if (!test_FlushKey(p, mem_ctx, &handle)) {
+       if (created && !test_FlushKey(p, mem_ctx, &handle)) {
                printf("FlushKey failed\n");
                ret = False;
        }
 
-       if (!test_OpenKey(p, mem_ctx, &handle, "spottyfoot", &newhandle)) {
+       if (created && !test_OpenKey(p, mem_ctx, &handle, TEST_KEY1, &newhandle)) {
                printf("CreateKey failed (OpenKey after Create didn't work)\n");
                ret = False;
        }
 
-       if (!test_DeleteKey(p, mem_ctx, &handle, "spottyfoot")) {
+       if (created && !test_DeleteKey(p, mem_ctx, &handle, TEST_KEY1)) {
                printf("DeleteKey failed\n");
                ret = False;
+       } else {
+               deleted = True;
        }
 
-       if (!test_FlushKey(p, mem_ctx, &handle)) {
+       if (created && !test_FlushKey(p, mem_ctx, &handle)) {
                printf("FlushKey failed\n");
                ret = False;
        }
 
-       if (test_OpenKey(p, mem_ctx, &handle, "spottyfoot", &newhandle)) {
+       if (created && deleted && 
+           test_OpenKey(p, mem_ctx, &handle, TEST_KEY1, &newhandle)) {
                printf("DeleteKey failed (OpenKey after Delete didn't work)\n");
                ret = False;
        }
@@ -598,9 +766,29 @@ static BOOL test_Open(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, void *fn)
                ret = False;
        }
 
+       if (created && test_CreateKey_sd(p, mem_ctx, &handle, TEST_KEY2, 
+                                         NULL, &newhandle)) {
+               created2 = True;
+       }
+
+       if (created2 && !test_GetKeySecurity(p, mem_ctx, &newhandle)) {
+               printf("GetKeySecurity failed\n");
+               ret = False;
+       }
+
+       if (created2 && !test_CloseKey(p, mem_ctx, &newhandle)) {
+               printf("CloseKey failed\n");
+               ret = False;
+       }
+
+       if (created && !test_DeleteKey(p, mem_ctx, &handle, TEST_KEY2)) {
+               printf("DeleteKey failed\n");
+               ret = False;
+       }
+
        /* The HKCR hive has a very large fanout */
 
-       if (open_fn == test_OpenHKCR) {
+       if (open_fn == (void *)dcerpc_winreg_OpenHKCR) {
                if(!test_key(p, mem_ctx, &handle, MAX_DEPTH - 1)) {
                        ret = False;
                }
@@ -610,44 +798,49 @@ static BOOL test_Open(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx, void *fn)
                ret = False;
        }
 
+       test_Cleanup(p, mem_ctx, &handle, TEST_KEY_BASE);
+
        return ret;
 }
 
-BOOL torture_rpc_winreg(void)
+BOOL torture_rpc_winreg(struct torture_context *torture)
 {
         NTSTATUS status;
-       struct dcerpc_pipe *p;
+       struct dcerpc_pipe *p;
        TALLOC_CTX *mem_ctx;
        BOOL ret = True;
-       winreg_open_fn *open_fns[] = { test_OpenHKLM, test_OpenHKU,
-                                      test_OpenHKCR, test_OpenHKCU };
+       struct {
+               const char *name;
+               winreg_open_fn fn;
+       } open_fns[] = {{"OpenHKLM", (winreg_open_fn)dcerpc_winreg_OpenHKLM },
+                       {"OpenHKU",  (winreg_open_fn)dcerpc_winreg_OpenHKU },
+                       {"OpenHKCR", (winreg_open_fn)dcerpc_winreg_OpenHKCR },
+                       {"OpenHKCU", (winreg_open_fn)dcerpc_winreg_OpenHKCU }};
        int i;
-
        mem_ctx = talloc_init("torture_rpc_winreg");
 
-       status = torture_rpc_connection(&p, 
-                                       DCERPC_WINREG_NAME, 
-                                       DCERPC_WINREG_UUID, 
-                                       DCERPC_WINREG_VERSION);
+       status = torture_rpc_connection(mem_ctx, &p, &dcerpc_table_winreg);
 
        if (!NT_STATUS_IS_OK(status)) {
+               talloc_free(mem_ctx);
                return False;
        }
 
-       if(!test_InitiateSystemShutdown(p, mem_ctx, "spottyfood", 30))
-               ret = False;
-
-       if(!test_AbortSystemShutdown(p, mem_ctx))
-               ret = False;
+       if (!torture_setting_bool(torture, "dangerous", False)) {
+               printf("winreg_InitiateShutdown disabled - enable dangerous tests to use\n");
+       } else {
+               ret &= test_InitiateSystemShutdown(p, mem_ctx, "spottyfood", 30);
+               ret &= test_AbortSystemShutdown(p, mem_ctx);
+               ret &= test_InitiateSystemShutdownEx(p, mem_ctx, "spottyfood", 30);
+               ret &= test_AbortSystemShutdown(p, mem_ctx);
+       }
 
        for (i = 0; i < ARRAY_SIZE(open_fns); i++) {
-               if (!test_Open(p, mem_ctx, open_fns[i]))
+               if (!test_Open(p, mem_ctx, open_fns[i].name, open_fns[i].fn))
                        ret = False;
        }
 
-       talloc_destroy(mem_ctx);
-
-        torture_rpc_close(p);
+       talloc_free(mem_ctx);
 
        return ret;
 }