[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 7/8] crypto: use error_abort for unexpected failures
From: |
Daniel P . Berrangé |
Subject: |
[PULL 7/8] crypto: use error_abort for unexpected failures |
Date: |
Tue, 19 Mar 2024 20:21:20 +0000 |
This improves the error diagnosis from the unit test when a cipher
is unexpected not available from
ERROR:../tests/unit/test-crypto-cipher.c:683:test_cipher: assertion failed:
(err == NULL)
Bail out! ERROR:../tests/unit/test-crypto-cipher.c:683:test_cipher: assertion
failed: (err == NULL)
Aborted (core dumped)
to
Unexpected error in qcrypto_cipher_ctx_new() at
../crypto/cipher-gcrypt.c.inc:262:
./build//tests/unit/test-crypto-cipher: Cannot initialize cipher: Invalid
cipher algorithm
Aborted (core dumped)
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
tests/unit/test-crypto-cipher.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tests/unit/test-crypto-cipher.c b/tests/unit/test-crypto-cipher.c
index 11ab1a54fc..d0ea7b4d8e 100644
--- a/tests/unit/test-crypto-cipher.c
+++ b/tests/unit/test-crypto-cipher.c
@@ -676,9 +676,8 @@ static void test_cipher(const void *opaque)
cipher = qcrypto_cipher_new(
data->alg, data->mode,
key, nkey,
- &err);
+ data->plaintext ? &error_abort : &err);
if (data->plaintext) {
- g_assert(err == NULL);
g_assert(cipher != NULL);
} else {
error_free_or_abort(&err);
--
2.43.0
- [PULL 0/8] Misc fixes patches, Daniel P . Berrangé, 2024/03/19
- [PULL 2/8] chardev: lower priority of the HUP GSource in socket chardev, Daniel P . Berrangé, 2024/03/19
- [PULL 1/8] seccomp: report EPERM instead of killing process for spawn set, Daniel P . Berrangé, 2024/03/19
- [PULL 3/8] Revert "chardev/char-socket: Fix TLS io channels sending too much data to the backend", Daniel P . Berrangé, 2024/03/19
- [PULL 4/8] Revert "chardev: use a child source for qio input source", Daniel P . Berrangé, 2024/03/19
- [PULL 6/8] crypto: query gcrypt for cipher availability, Daniel P . Berrangé, 2024/03/19
- [PULL 5/8] crypto: factor out conversion of QAPI to gcrypt constants, Daniel P . Berrangé, 2024/03/19
- [PULL 7/8] crypto: use error_abort for unexpected failures,
Daniel P . Berrangé <=
- [PULL 8/8] crypto: report which ciphers are being skipped during tests, Daniel P . Berrangé, 2024/03/19
- Re: [PULL 0/8] Misc fixes patches, Peter Maydell, 2024/03/20