Fix building of test/pbetest.c

The test_pkcs5_pbe() function is required twice:
once `if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5`
and once `if !defined OPENSSL_NO_DES && !defined OPENSSL_NO_SHA1`

Hence there should be `||` between those. Currently the build fails
if the first condition is false, while the second is true.

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/15432)
This commit is contained in:
Petr Gotthard 2021-05-24 11:40:15 +02:00 committed by Pauli
parent b8c09a893c
commit 9080ed2175

View File

@ -16,9 +16,8 @@
#include <openssl/rc4.h>
#include <openssl/md5.h>
#if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5
# if !defined OPENSSL_NO_DES && !defined OPENSSL_NO_SHA1
#if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5 \
|| !defined OPENSSL_NO_DES && !defined OPENSSL_NO_SHA1
static const char pbe_password[] = "MyVoiceIsMyPassport";
static unsigned char pbe_salt[] = {
@ -32,7 +31,6 @@ static unsigned char pbe_plaintext[] = {
0x6c, 0x6c, 0x20, 0x6d, 0x61, 0x64, 0x65, 0x20,
0x6f, 0x66, 0x20, 0x73, 0x74, 0x61, 0x72, 0x73,
};
# endif
#endif
/* Expected output generated using OpenSSL 1.1.1 */
@ -54,8 +52,8 @@ static const unsigned char pbe_ciphertext_des_sha1[] = {
};
#endif
#if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5
# if !defined OPENSSL_NO_DES && !defined OPENSSL_NO_SHA1
#if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5 \
|| !defined OPENSSL_NO_DES && !defined OPENSSL_NO_SHA1
static int test_pkcs5_pbe(const EVP_CIPHER *cipher, const EVP_MD *md,
const unsigned char *exp, const int exp_len)
{
@ -109,7 +107,6 @@ err:
X509_ALGOR_free(algor);
return ret;
}
# endif
#endif
#if !defined OPENSSL_NO_RC4 && !defined OPENSSL_NO_MD5