From dfcdc9012fd0b7a75ec35de614e9011416612f0e Mon Sep 17 00:00:00 2001 From: Jack Lloyd Date: Fri, 23 Dec 2016 01:12:17 -0500 Subject: Add DL_Group tests Fix a bug in how the 6144 and 8192 IETF MODP groups were encoded; they have g and q values switched. Fixed by just switching the PEM header to match the actual encoded format. Rename DL_Group::X942_DH_PARAMETERS to ANSI_X9_42_DH_PARAMETERS to avoid a macro conflict with Windows cryptography headers (GH #482) --- src/lib/pubkey/dl_group/dl_group.h | 4 ++-- src/lib/pubkey/dl_group/dl_named.cpp | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) (limited to 'src/lib/pubkey/dl_group') diff --git a/src/lib/pubkey/dl_group/dl_group.h b/src/lib/pubkey/dl_group/dl_group.h index 39ad9b954..98a49649c 100644 --- a/src/lib/pubkey/dl_group/dl_group.h +++ b/src/lib/pubkey/dl_group/dl_group.h @@ -49,7 +49,7 @@ class BOTAN_DLL DL_Group DSA_PARAMETERS = ANSI_X9_57, DH_PARAMETERS = ANSI_X9_42, - X942_DH_PARAMETERS = ANSI_X9_42, + ANSI_X9_42_DH_PARAMETERS = ANSI_X9_42, PKCS3_DH_PARAMETERS = PKCS_3 }; @@ -106,7 +106,7 @@ class BOTAN_DLL DL_Group * @param name the name that is configured in the global configuration * for the desired group. If no configuration file is specified, * the default values from the file policy.cpp will be used. For instance, - * use "modp/ietf/768" as name. + * use "modp/ietf/3072". */ DL_Group(const std::string& name); diff --git a/src/lib/pubkey/dl_group/dl_named.cpp b/src/lib/pubkey/dl_group/dl_named.cpp index 9c084c80b..56871657e 100644 --- a/src/lib/pubkey/dl_group/dl_named.cpp +++ b/src/lib/pubkey/dl_group/dl_named.cpp @@ -170,7 +170,7 @@ const char* DL_Group::PEM_for_named_group(const std::string& name) if(name == "modp/ietf/6144") return - "-----BEGIN X942 DH PARAMETERS-----" + "-----BEGIN DSA PARAMETERS-----" "MIIGDAKCAwEA///////////JD9qiIWjCNMTGYouA3BzRKQJOCIpnzHQCC76mOxOb" "IlFKCHmONATd75UZs806QxswKwpt8l8UN0/hNW1tUcJF5IW1dmJefsb0TELppjft" "awv/XLb0Brft7jhr+1qJn6WunyQRfEsf5kkoZlHs5Fs9wgB8uKFjvwWY2kg2HFXT" @@ -204,7 +204,7 @@ const char* DL_Group::PEM_for_named_group(const std::string& name) "2/+sLWKl6gPZFaCqVWZHtr9fpHDsCmYvaQfAG/BTy4r3eU3xlANQ6sXb4u07eqhV" "HsUP3/h1jOZY0Ynqrm0rZPYXeUsZHD/0a7ceAjQCH0ezH6Qwdwlflq2Fujprc0p8" "jzbmIBJ//////////wIBAg==" - "-----END X942 DH PARAMETERS-----"; + "-----END DSA PARAMETERS-----"; if(name == "modp/srp/6144") return @@ -230,7 +230,7 @@ const char* DL_Group::PEM_for_named_group(const std::string& name) if(name == "modp/ietf/8192") return - "-----BEGIN X942 DH PARAMETERS-----" + "-----BEGIN DSA PARAMETERS-----" "MIIIDAKCBAEA///////////JD9qiIWjCNMTGYouA3BzRKQJOCIpnzHQCC76mOxOb" "IlFKCHmONATd75UZs806QxswKwpt8l8UN0/hNW1tUcJF5IW1dmJefsb0TELppjft" "awv/XLb0Brft7jhr+1qJn6WunyQRfEsf5kkoZlHs5Fs9wgB8uKFjvwWY2kg2HFXT" @@ -274,7 +274,7 @@ const char* DL_Group::PEM_for_named_group(const std::string& name) "gfat7p0DFZ562dE8UzaVCfwfonwW75iHcDpVtRsiy/RM0BKu4LJ5jmKEI0KO/NWk" "DK72v1DY6ohev3Omuf15teGPZ9E0GsgjenXDz8kgBKHFpA42a8RNABdq9xwV5IyG" "034BNyPKrHIjqzv01U8YKHE7K0pv5A+rdEBctziwZMBuzHbp7///////////AgEC" - "-----END X942 DH PARAMETERS-----"; + "-----END DSA PARAMETERS-----"; if(name == "modp/srp/8192") return -- cgit v1.2.3