]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
machine_account_secrets: do not generate single DES keys
authorIsaac Boukris <iboukris@gmail.com>
Mon, 16 Sep 2019 12:17:08 +0000 (15:17 +0300)
committerIsaac Boukris <iboukris@sn-devel-184>
Tue, 19 Nov 2019 14:48:41 +0000 (14:48 +0000)
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14202

Signed-off-by: Isaac Boukris <iboukris@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source3/passdb/machine_account_secrets.c

index dfc21f295a1f9a96b7069b25653b527a964cfab1..efba80f147457575b5cc7351a9c6540c874bfba9 100644 (file)
@@ -1031,7 +1031,6 @@ static int secrets_domain_info_kerberos_keys(struct secrets_domain_info1_passwor
        krb5_keyblock key;
        DATA_BLOB aes_256_b = data_blob_null;
        DATA_BLOB aes_128_b = data_blob_null;
-       DATA_BLOB des_md5_b = data_blob_null;
        bool ok;
 #endif /* HAVE_ADS */
        DATA_BLOB arc4_b = data_blob_null;
@@ -1177,32 +1176,6 @@ static int secrets_domain_info_kerberos_keys(struct secrets_domain_info1_passwor
                return ENOMEM;
        }
 
-       krb5_ret = smb_krb5_create_key_from_string(krb5_ctx,
-                                                  NULL,
-                                                  &salt,
-                                                  &cleartext_utf8,
-                                                  ENCTYPE_DES_CBC_MD5,
-                                                  &key);
-       if (krb5_ret != 0) {
-               DBG_ERR("generation of a des-cbc-md5 key failed: %s\n",
-                       smb_get_krb5_error_message(krb5_ctx, krb5_ret, keys));
-               krb5_free_context(krb5_ctx);
-               TALLOC_FREE(keys);
-               TALLOC_FREE(salt_data);
-               return krb5_ret;
-       }
-       des_md5_b = data_blob_talloc(keys,
-                                    KRB5_KEY_DATA(&key),
-                                    KRB5_KEY_LENGTH(&key));
-       krb5_free_keyblock_contents(krb5_ctx, &key);
-       if (des_md5_b.data == NULL) {
-               DBG_ERR("data_blob_talloc failed for des-cbc-md5.\n");
-               krb5_free_context(krb5_ctx);
-               TALLOC_FREE(keys);
-               TALLOC_FREE(salt_data);
-               return ENOMEM;
-       }
-
        krb5_free_context(krb5_ctx);
 no_kerberos:
 
@@ -1227,15 +1200,6 @@ no_kerberos:
        keys[idx].value                 = arc4_b;
        idx += 1;
 
-#ifdef HAVE_ADS
-       if (des_md5_b.length != 0) {
-               keys[idx].keytype               = ENCTYPE_DES_CBC_MD5;
-               keys[idx].iteration_count       = 4096;
-               keys[idx].value                 = des_md5_b;
-               idx += 1;
-       }
-#endif /* HAVE_ADS */
-
        p->salt_data = salt_data;
        p->default_iteration_count = 4096;
        p->num_keys = idx;