From 91ddb848264326ad2cec68ca750424b1649e7f64 Mon Sep 17 00:00:00 2001 From: Stef Walter Date: Tue, 22 Mar 2011 17:10:32 +0100 Subject: Fix calls to empathy_tls_verifier_new() in tests. --- tests/empathy-tls-test.c | 87 ++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 76 insertions(+), 11 deletions(-) diff --git a/tests/empathy-tls-test.c b/tests/empathy-tls-test.c index 31cadfbf6..f2b667d0b 100644 --- a/tests/empathy-tls-test.c +++ b/tests/empathy-tls-test.c @@ -430,6 +430,10 @@ test_certificate_verify_success_with_pkcs11_lookup (Test *test, EmpTLSCertificateRejectReason reason = 0; GError *error = NULL; EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "www.collabora.co.uk", + NULL + }; /* * In this test the mock TLS connection only has one certificate @@ -440,13 +444,14 @@ test_certificate_verify_success_with_pkcs11_lookup (Test *test, test->mock = mock_tls_certificate_new_and_register (test->dbus, "dhansak-collabora.cer", NULL); - /* We add teh collabora directory with the collabora root */ + /* We add the collabora directory with the collabora root */ add_pkcs11_module_for_testing (test, "gkm-roots-store-standalone.so", "collabora-ca"); ensure_certificate_proxy (test); - verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk"); + verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk", + reference_identities); empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); g_main_loop_run (test->loop); if (!empathy_tls_verifier_verify_finish (verifier, test->result, &reason, @@ -466,6 +471,10 @@ test_certificate_verify_success_with_full_chain (Test *test, EmpTLSCertificateRejectReason reason = 0; GError *error = NULL; EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "www.collabora.co.uk", + NULL + }; /* * In this test the mock TLS connection has a full certificate @@ -475,13 +484,14 @@ test_certificate_verify_success_with_full_chain (Test *test, test->mock = mock_tls_certificate_new_and_register (test->dbus, "dhansak-collabora.cer", "collabora-ca/collabora-ca.cer", NULL); - /* We add teh collabora directory with the collabora root */ + /* We add the collabora directory with the collabora root */ add_pkcs11_module_for_testing (test, "gkm-roots-store-standalone.so", "collabora-ca"); ensure_certificate_proxy (test); - verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk"); + verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk", + reference_identities); empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); g_main_loop_run (test->loop); if (!empathy_tls_verifier_verify_finish (verifier, test->result, &reason, @@ -501,6 +511,10 @@ test_certificate_verify_root_not_found (Test *test, EmpTLSCertificateRejectReason reason = 0; GError *error = NULL; EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "www.collabora.co.uk", + NULL + }; test->mock = mock_tls_certificate_new_and_register (test->dbus, "dhansak-collabora.cer", NULL); @@ -509,7 +523,8 @@ test_certificate_verify_root_not_found (Test *test, ensure_certificate_proxy (test); - verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk"); + verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk", + reference_identities); empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); g_main_loop_run (test->loop); @@ -531,6 +546,10 @@ test_certificate_verify_root_not_anchored (Test *test, EmpTLSCertificateRejectReason reason = 0; GError *error = NULL; EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "www.collabora.co.uk", + NULL + }; test->mock = mock_tls_certificate_new_and_register (test->dbus, "dhansak-collabora.cer", "collabora-ca/collabora-ca.cer", NULL); @@ -539,7 +558,8 @@ test_certificate_verify_root_not_anchored (Test *test, ensure_certificate_proxy (test); - verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk"); + verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk", + reference_identities); empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); g_main_loop_run (test->loop); @@ -555,23 +575,66 @@ test_certificate_verify_root_not_anchored (Test *test, } static void -test_certificate_verify_hostname_invalid (Test *test, +test_certificate_verify_identities_invalid (Test *test, gconstpointer data G_GNUC_UNUSED) { EmpTLSCertificateRejectReason reason = 0; GError *error = NULL; EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "invalid.host.name", + NULL + }; + + test->mock = mock_tls_certificate_new_and_register (test->dbus, + "dhansak-collabora.cer", "collabora-ca/collabora-ca.cer", NULL); + + /* We add the collabora directory with the collabora root */ + add_pkcs11_module_for_testing (test, "gkm-roots-store-standalone.so", + "collabora-ca"); + + ensure_certificate_proxy (test); + + verifier = empathy_tls_verifier_new (test->cert, "invalid.host.name", + reference_identities); + empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); + g_main_loop_run (test->loop); + + if (empathy_tls_verifier_verify_finish (verifier, test->result, &reason, + NULL, &error)) + g_assert_not_reached (); + + /* And it should say we're self-signed (oddly enough) */ + g_assert_cmpuint (reason, ==, EMP_TLS_CERTIFICATE_REJECT_REASON_HOSTNAME_MISMATCH); + + g_clear_error (&error); + g_object_unref (verifier); +} + +static void +test_certificate_verify_uses_reference_identities (Test *test, + gconstpointer data G_GNUC_UNUSED) +{ + EmpTLSCertificateRejectReason reason = 0; + GError *error = NULL; + EmpathyTLSVerifier *verifier; + const gchar *reference_identities[] = { + "invalid.host.name", + NULL + }; test->mock = mock_tls_certificate_new_and_register (test->dbus, "dhansak-collabora.cer", "collabora-ca/collabora-ca.cer", NULL); - /* We add teh collabora directory with the collabora root */ + /* We add the collabora directory with the collabora root */ add_pkcs11_module_for_testing (test, "gkm-roots-store-standalone.so", "collabora-ca"); ensure_certificate_proxy (test); - verifier = empathy_tls_verifier_new (test->cert, "invalid.host.name"); + /* Should be using the reference_identities and not host name for checks */ + verifier = empathy_tls_verifier_new (test->cert, "www.collabora.co.uk", + reference_identities); empathy_tls_verifier_verify_async (verifier, fetch_callback_result, test); g_main_loop_run (test->loop); @@ -605,8 +668,10 @@ main (int argc, setup, test_certificate_verify_root_not_found, teardown); g_test_add ("/tls/certificate_verify_root_not_anchored", Test, NULL, setup, test_certificate_verify_root_not_anchored, teardown); - g_test_add ("/tls/certificate_verify_hostname_invalid", Test, NULL, - setup, test_certificate_verify_hostname_invalid, teardown); + g_test_add ("/tls/certificate_verify_identities_invalid", Test, NULL, + setup, test_certificate_verify_identities_invalid, teardown); + g_test_add ("/tls/certificate_verify_uses_reference_identities", Test, NULL, + setup, test_certificate_verify_uses_reference_identities, teardown); result = g_test_run (); test_deinit (); -- cgit v1.2.3