s3: Fix an uninitialized variable read
authorVolker Lendecke <vl@samba.org>
Sat, 26 Mar 2011 12:33:40 +0000 (13:33 +0100)
committerVolker Lendecke <vlendec@samba.org>
Sat, 26 Mar 2011 13:24:28 +0000 (14:24 +0100)
The "is_valid_policy_handle" in the exit path reads uninitialized data

Lets see if this fixes Coverity ID 2172 which complains about h being
dereferenced in the dcerpc_winreg_CloseKey.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Mar 26 14:24:28 CET 2011 on sn-devel-104

source3/services/svc_winreg_glue.c

index dc367c2c41d9b4bd787517fff63284609adfcf04..06f3f9195f94e2b7d5737421a5aeaf90d690e674 100644 (file)
@@ -153,7 +153,8 @@ bool svcctl_set_secdesc(struct messaging_context *msg_ctx,
 {
        struct dcerpc_binding_handle *h = NULL;
        uint32_t access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
-       struct policy_handle hive_hnd, key_hnd;
+       struct policy_handle hive_hnd;
+       struct policy_handle key_hnd = { 0, };
        char *key = NULL;
        bool ok = false;
        TALLOC_CTX *tmp_ctx;