|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
From: "Daniel P. Berrange" <berrange@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
Date: Tue, 29 Aug 2017 17:03:30 +0100
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
Subject: [PATCH] crypto: fix test cert generation to not use SHA1 algorithm
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
GNUTLS 3.6.0 marked SHA1 as untrusted for certificates.
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
Unfortunately the gnutls_x509_crt_sign() method we are
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
using to create certificates in the test suite is fixed
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
to always use SHA1. We must switch to a different method
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
and explicitly ask for SHA256.
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
Reviewed-by: Eric Blake <eblake@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
---
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
tests/crypto-tls-x509-helpers.c | 3 ++-
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
diff --git a/tests/crypto-tls-x509-helpers.c b/tests/crypto-tls-x509-helpers.c
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
index 64073d3bd3..173d4e28fb 100644
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
--- a/tests/crypto-tls-x509-helpers.c
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
+++ b/tests/crypto-tls-x509-helpers.c
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
@@ -406,7 +406,8 @@ test_tls_generate_cert(QCryptoTLSTestCertReq *req,
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
* If no 'ca' is set then we are self signing
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
* the cert. This is done for the root CA certs
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
*/
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
- err = gnutls_x509_crt_sign(crt, ca ? ca : crt, privkey);
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
+ err = gnutls_x509_crt_sign2(crt, ca ? ca : crt, privkey,
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
+ GNUTLS_DIG_SHA256, 0);
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
if (err < 0) {
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
g_critical("Failed to sign certificate %s",
|
|
![](https://seccdn.libravatar.org/avatar/0f8f8434e79f17ac67e6709c15c9c15f6f8894d3b72dfcf5789623403a6a9470?s=16&d=retro) |
a8c6008 |
gnutls_strerror(err));
|