summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEdward Thomson <ethomson@edwardthomson.com>2021-12-13 15:31:21 +0000
committerEdward Thomson <ethomson@edwardthomson.com>2022-01-22 19:20:11 -0500
commit0aa4a883257ade65e218d24598c81fe7063527ce (patch)
tree00636d645982f3d0c6924c0238c064b94207915d
parent95fa146c61868d06c07cb6d3dbe46222ac429525 (diff)
downloadlibgit2-0aa4a883257ade65e218d24598c81fe7063527ce.tar.gz
sha: ensure we test both cng and cryptoapi on windows
When GIT_SHA1_WIN32 or GIT_SHA256_WIN32 is used, ensure that we test both CryptoNG ("cng") and CryptoAPI.
-rw-r--r--src/hash/win32.c21
-rw-r--r--src/hash/win32.h7
-rw-r--r--tests/core/sha1.c30
-rw-r--r--tests/core/sha256.c32
4 files changed, 90 insertions, 0 deletions
diff --git a/src/hash/win32.c b/src/hash/win32.c
index 3320f7968..bc5833b1d 100644
--- a/src/hash/win32.c
+++ b/src/hash/win32.c
@@ -239,6 +239,27 @@ static int hash_provider_init(void)
return error;
}
+git_hash_win32_provider_t git_hash_win32_provider(void)
+{
+ return hash_provider.type;
+}
+
+int git_hash_win32_set_provider(git_hash_win32_provider_t provider)
+{
+ if (provider == hash_provider.type)
+ return 0;
+
+ hash_provider_shutdown();
+
+ if (provider == GIT_HASH_WIN32_CNG)
+ return cng_provider_init();
+ else if (provider == GIT_HASH_WIN32_CRYPTOAPI)
+ return cryptoapi_provider_init();
+
+ git_error_set(GIT_ERROR_SHA, "unsupported win32 provider");
+ return -1;
+}
+
/* CryptoAPI: available in Windows XP and newer */
GIT_INLINE(int) hash_cryptoapi_init(git_hash_win32_ctx *ctx)
diff --git a/src/hash/win32.h b/src/hash/win32.h
index c05804753..a9fb87aee 100644
--- a/src/hash/win32.h
+++ b/src/hash/win32.h
@@ -38,6 +38,13 @@ typedef struct {
} ctx;
} git_hash_win32_ctx;
+/*
+ * Gets/sets the current hash provider (cng or cryptoapi). This is only
+ * for testing purposes.
+ */
+git_hash_win32_provider_t git_hash_win32_provider(void);
+int git_hash_win32_set_provider(git_hash_win32_provider_t provider);
+
#ifdef GIT_SHA1_WIN32
struct git_hash_sha1_ctx {
git_hash_win32_ctx win32;
diff --git a/tests/core/sha1.c b/tests/core/sha1.c
index 9ccdaab3c..2f9a553d5 100644
--- a/tests/core/sha1.c
+++ b/tests/core/sha1.c
@@ -3,13 +3,25 @@
#define FIXTURE_DIR "sha1"
+#ifdef GIT_SHA1_WIN32
+static git_hash_win32_provider_t orig_provider;
+#endif
+
void test_core_sha1__initialize(void)
{
+#ifdef GIT_SHA1_WIN32
+ orig_provider = git_hash_win32_provider();
+#endif
+
cl_fixture_sandbox(FIXTURE_DIR);
}
void test_core_sha1__cleanup(void)
{
+#ifdef GIT_SHA1_WIN32
+ git_hash_win32_set_provider(orig_provider);
+#endif
+
cl_fixture_cleanup(FIXTURE_DIR);
}
@@ -68,3 +80,21 @@ void test_core_sha1__detect_collision_attack(void)
#endif
}
+void test_core_sha1__win32_providers(void)
+{
+#ifdef GIT_SHA1_WIN32
+ unsigned char expected[GIT_HASH_SHA1_SIZE] = {
+ 0x38, 0x76, 0x2c, 0xf7, 0xf5, 0x59, 0x34, 0xb3, 0x4d, 0x17,
+ 0x9a, 0xe6, 0xa4, 0xc8, 0x0c, 0xad, 0xcc, 0xbb, 0x7f, 0x0a
+ };
+ unsigned char actual[GIT_HASH_SHA1_SIZE];
+
+ git_hash_win32_set_provider(GIT_HASH_WIN32_CRYPTOAPI);
+ cl_git_pass(sha1_file(actual, FIXTURE_DIR "/shattered-1.pdf"));
+ cl_assert_equal_i(0, memcmp(expected, actual, GIT_HASH_SHA1_SIZE));
+
+ git_hash_win32_set_provider(GIT_HASH_WIN32_CNG);
+ cl_git_pass(sha1_file(actual, FIXTURE_DIR "/shattered-1.pdf"));
+ cl_assert_equal_i(0, memcmp(expected, actual, GIT_HASH_SHA1_SIZE));
+#endif
+}
diff --git a/tests/core/sha256.c b/tests/core/sha256.c
index a44de32b7..b476527fd 100644
--- a/tests/core/sha256.c
+++ b/tests/core/sha256.c
@@ -3,13 +3,25 @@
#define FIXTURE_DIR "sha1"
+#ifdef GIT_SHA256_WIN32
+static git_hash_win32_provider_t orig_provider;
+#endif
+
void test_core_sha256__initialize(void)
{
+#ifdef GIT_SHA256_WIN32
+ orig_provider = git_hash_win32_provider();
+#endif
+
cl_fixture_sandbox(FIXTURE_DIR);
}
void test_core_sha256__cleanup(void)
{
+#ifdef GIT_SHA256_WIN32
+ git_hash_win32_set_provider(orig_provider);
+#endif
+
cl_fixture_cleanup(FIXTURE_DIR);
}
@@ -79,3 +91,23 @@ void test_core_sha256__pdf(void)
cl_assert_equal_i(0, memcmp(expected, actual, GIT_HASH_SHA256_SIZE));
}
+void test_core_sha256__win32_providers(void)
+{
+#ifdef GIT_SHA256_WIN32
+ unsigned char expected[GIT_HASH_SHA256_SIZE] = {
+ 0x2b, 0xb7, 0x87, 0xa7, 0x3e, 0x37, 0x35, 0x2f,
+ 0x92, 0x38, 0x3a, 0xbe, 0x7e, 0x29, 0x02, 0x93,
+ 0x6d, 0x10, 0x59, 0xad, 0x9f, 0x1b, 0xa6, 0xda,
+ 0xaa, 0x9c, 0x1e, 0x58, 0xee, 0x69, 0x70, 0xd0
+ };
+ unsigned char actual[GIT_HASH_SHA256_SIZE];
+
+ git_hash_win32_set_provider(GIT_HASH_WIN32_CRYPTOAPI);
+ cl_git_pass(sha256_file(actual, FIXTURE_DIR "/shattered-1.pdf"));
+ cl_assert_equal_i(0, memcmp(expected, actual, GIT_HASH_SHA256_SIZE));
+
+ git_hash_win32_set_provider(GIT_HASH_WIN32_CNG);
+ cl_git_pass(sha256_file(actual, FIXTURE_DIR "/shattered-1.pdf"));
+ cl_assert_equal_i(0, memcmp(expected, actual, GIT_HASH_SHA256_SIZE));
+#endif
+}