root
/
openssl
mirror of https://github.com/openssl/openssl.git
1
0
Fork 0
Code Issues Actions 7 Packages Projects Releases Wiki Activity
openssl/test/ossl_store_test.c

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

401 lines
17 KiB
C
Raw Normal View History

Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
/*
Update copyright year Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14512)
2021-03-11 21:27:36 +08:00
* Copyright 2020-2021 The OpenSSL Project Authors. All Rights Reserved.
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
*
* Licensed under the Apache License 2.0 (the "License"). You may not use
* this file except in compliance with the License. You can obtain a copy
* in the file LICENSE in the source distribution or at
* https://www.openssl.org/source/license.html
*/
test/ossl_store_test.c: Adapt the use of datadir for VMS paths Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15823)
2021-06-22 13:28:26 +08:00
#include <string.h>
TEST: Fix path length in test/ossl_store_test.c The URI length was set to 80 chars, but the URI being built up may need more space, all depending on the paths used to get to the files that are to be loaded. If the result needs more than 80 chars, the test will fail. Fixed by using PATH_MAX. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> Reviewed-by: Paul Dale <paul.dale@oracle.com> (Merged from https://github.com/openssl/openssl/pull/13546)
2020-11-27 15:08:08 +08:00
#include <limits.h>
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
#include <openssl/store.h>
#include <openssl/ui.h>
#include "testutil.h"
TEST: Fix path length in test/ossl_store_test.c The URI length was set to 80 chars, but the URI being built up may need more space, all depending on the paths used to get to the files that are to be loaded. If the result needs more than 80 chars, the test will fail. Fixed by using PATH_MAX. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> Reviewed-by: Paul Dale <paul.dale@oracle.com> (Merged from https://github.com/openssl/openssl/pull/13546)
2020-11-27 15:08:08 +08:00
#ifndef PATH_MAX
# if defined(_WIN32) && defined(_MAX_PATH)
# define PATH_MAX _MAX_PATH
# else
# define PATH_MAX 4096
# endif
#endif
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
typedef enum OPTION_choice {
OPT_ERR = -1,
OPT_EOF = 0,
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
OPT_INPUTDIR,
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
OPT_INFILE,
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
OPT_SM2FILE,
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
OPT_DATADIR,
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
OPT_TEST_ENUM
} OPTION_CHOICE;
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
static const char *inputdir = NULL;
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
static const char *infile = NULL;
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
static const char *sm2file = NULL;
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
static const char *datadir = NULL;
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
static int test_store_open(void)
{
int ret = 0;
OSSL_STORE_CTX *sctx = NULL;
Fix coverity issue: CID 1466485 - Explicit NULL dereference in OSSL_STORE_find() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 15:22:40 +08:00
OSSL_STORE_SEARCH *search = NULL;
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
UI_METHOD *ui_method = NULL;
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
char *input = test_mk_file_path(inputdir, infile);
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
ret = TEST_ptr(input)
&& TEST_ptr(search = OSSL_STORE_SEARCH_by_alias("nothing"))
Fix coverity issue: CID 1466485 - Explicit NULL dereference in OSSL_STORE_find() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 15:22:40 +08:00
&& TEST_ptr(ui_method= UI_create_method("DummyUI"))
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
&& TEST_ptr(sctx = OSSL_STORE_open_ex(input, NULL, NULL, ui_method,
Make the -inform option to be respected if possible Add OSSL_STORE_PARAM_INPUT_TYPE and make it possible to be set when OSSL_STORE_open_ex() or OSSL_STORE_attach() is called. The input type format is enforced only in case the file type file store is used. By default we use FORMAT_UNDEF meaning the input type is not enforced. Fixes #14569 Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15100)
2021-04-30 22:57:53 +08:00
NULL, NULL, NULL, NULL))
Fix coverity issue: CID 1466485 - Explicit NULL dereference in OSSL_STORE_find() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 15:22:40 +08:00
&& TEST_false(OSSL_STORE_find(sctx, NULL))
&& TEST_true(OSSL_STORE_find(sctx, search));
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
UI_destroy_method(ui_method);
Fix coverity issue: CID 1466485 - Explicit NULL dereference in OSSL_STORE_find() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 15:22:40 +08:00
OSSL_STORE_SEARCH_free(search);
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
OSSL_STORE_close(sctx);
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
OPENSSL_free(input);
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
return ret;
}
Adding winstore open test Reviewed-by: Saša Nedvědický <sashan@openssl.org> Reviewed-by: Viktor Dukhovni <viktor@openssl.org> (Merged from https://github.com/openssl/openssl/pull/27393)
2025-04-16 02:46:27 +08:00
#ifndef OPENSSL_NO_WINSTORE
/*
* This is the one of the root certificate authorities from the
* microsoft cert store. We use it to extract the subject name
* so that we can search for it in the store
*/
static const unsigned char mscert[] = {
0x30, 0x82, 0x05, 0xed, 0x30, 0x82, 0x03, 0xd5, 0xa0, 0x03, 0x02, 0x01, 0x02,
0x02, 0x10, 0x3f, 0x8b, 0xc8, 0xb5, 0xfc, 0x9f, 0xb2, 0x96, 0x43, 0xb5, 0x69,
0xd6, 0x6c, 0x42, 0xe1, 0x44, 0x30, 0x0d, 0x06, 0x09, 0x2a, 0x86, 0x48, 0x86,
0xf7, 0x0d, 0x01, 0x01, 0x0b, 0x05, 0x00, 0x30, 0x81, 0x88, 0x31, 0x0b, 0x30,
0x09, 0x06, 0x03, 0x55, 0x04, 0x06, 0x13, 0x02, 0x55, 0x53, 0x31, 0x13, 0x30,
0x11, 0x06, 0x03, 0x55, 0x04, 0x08, 0x13, 0x0a, 0x57, 0x61, 0x73, 0x68, 0x69,
0x6e, 0x67, 0x74, 0x6f, 0x6e, 0x31, 0x10, 0x30, 0x0e, 0x06, 0x03, 0x55, 0x04,
0x07, 0x13, 0x07, 0x52, 0x65, 0x64, 0x6d, 0x6f, 0x6e, 0x64, 0x31, 0x1e, 0x30,
0x1c, 0x06, 0x03, 0x55, 0x04, 0x0a, 0x13, 0x15, 0x4d, 0x69, 0x63, 0x72, 0x6f,
0x73, 0x6f, 0x66, 0x74, 0x20, 0x43, 0x6f, 0x72, 0x70, 0x6f, 0x72, 0x61, 0x74,
0x69, 0x6f, 0x6e, 0x31, 0x32, 0x30, 0x30, 0x06, 0x03, 0x55, 0x04, 0x03, 0x13,
0x29, 0x4d, 0x69, 0x63, 0x72, 0x6f, 0x73, 0x6f, 0x66, 0x74, 0x20, 0x52, 0x6f,
0x6f, 0x74, 0x20, 0x43, 0x65, 0x72, 0x74, 0x69, 0x66, 0x69, 0x63, 0x61, 0x74,
0x65, 0x20, 0x41, 0x75, 0x74, 0x68, 0x6f, 0x72, 0x69, 0x74, 0x79, 0x20, 0x32,
0x30, 0x31, 0x31, 0x30, 0x1e, 0x17, 0x0d, 0x31, 0x31, 0x30, 0x33, 0x32, 0x32,
0x32, 0x32, 0x30, 0x35, 0x32, 0x38, 0x5a, 0x17, 0x0d, 0x33, 0x36, 0x30, 0x33,
0x32, 0x32, 0x32, 0x32, 0x31, 0x33, 0x30, 0x34, 0x5a, 0x30, 0x81, 0x88, 0x31,
0x0b, 0x30, 0x09, 0x06, 0x03, 0x55, 0x04, 0x06, 0x13, 0x02, 0x55, 0x53, 0x31,
0x13, 0x30, 0x11, 0x06, 0x03, 0x55, 0x04, 0x08, 0x13, 0x0a, 0x57, 0x61, 0x73,
0x68, 0x69, 0x6e, 0x67, 0x74, 0x6f, 0x6e, 0x31, 0x10, 0x30, 0x0e, 0x06, 0x03,
0x55, 0x04, 0x07, 0x13, 0x07, 0x52, 0x65, 0x64, 0x6d, 0x6f, 0x6e, 0x64, 0x31,
0x1e, 0x30, 0x1c, 0x06, 0x03, 0x55, 0x04, 0x0a, 0x13, 0x15, 0x4d, 0x69, 0x63,
0x72, 0x6f, 0x73, 0x6f, 0x66, 0x74, 0x20, 0x43, 0x6f, 0x72, 0x70, 0x6f, 0x72,
0x61, 0x74, 0x69, 0x6f, 0x6e, 0x31, 0x32, 0x30, 0x30, 0x06, 0x03, 0x55, 0x04,
0x03, 0x13, 0x29, 0x4d, 0x69, 0x63, 0x72, 0x6f, 0x73, 0x6f, 0x66, 0x74, 0x20,
0x52, 0x6f, 0x6f, 0x74, 0x20, 0x43, 0x65, 0x72, 0x74, 0x69, 0x66, 0x69, 0x63,
0x61, 0x74, 0x65, 0x20, 0x41, 0x75, 0x74, 0x68, 0x6f, 0x72, 0x69, 0x74, 0x79,
0x20, 0x32, 0x30, 0x31, 0x31, 0x30, 0x82, 0x02, 0x22, 0x30, 0x0d, 0x06, 0x09,
0x2a, 0x86, 0x48, 0x86, 0xf7, 0x0d, 0x01, 0x01, 0x01, 0x05, 0x00, 0x03, 0x82,
0x02, 0x0f, 0x00, 0x30, 0x82, 0x02, 0x0a, 0x02, 0x82, 0x02, 0x01, 0x00, 0xb2,
0x80, 0x41, 0xaa, 0x35, 0x38, 0x4d, 0x13, 0x72, 0x32, 0x68, 0x22, 0x4d, 0xb8,
0xb2, 0xf1, 0xff, 0xd5, 0x52, 0xbc, 0x6c, 0xc7, 0xf5, 0xd2, 0x4a, 0x8c, 0x36,
0xee, 0xd1, 0xc2, 0x5c, 0x7e, 0x8c, 0x8a, 0xae, 0xaf, 0x13, 0x28, 0x6f, 0xc0,
0x73, 0xe3, 0x3a, 0xce, 0xd0, 0x25, 0xa8, 0x5a, 0x3a, 0x6d, 0xef, 0xa8, 0xb8,
0x59, 0xab, 0x13, 0x23, 0x68, 0xcd, 0x0c, 0x29, 0x87, 0xd1, 0x6f, 0x80, 0x5c,
0x8f, 0x44, 0x7f, 0x5d, 0x90, 0x01, 0x52, 0x58, 0xac, 0x51, 0xc5, 0x5f, 0x2a,
0x87, 0xdc, 0xdc, 0xd8, 0x0a, 0x1d, 0xc1, 0x03, 0xb9, 0x7b, 0xb0, 0x56, 0xe8,
0xa3, 0xde, 0x64, 0x61, 0xc2, 0x9e, 0xf8, 0xf3, 0x7c, 0xb9, 0xec, 0x0d, 0xb5,
0x54, 0xfe, 0x4c, 0xb6, 0x65, 0x4f, 0x88, 0xf0, 0x9c, 0x48, 0x99, 0x0c, 0x42,
0x0b, 0x09, 0x7c, 0x31, 0x59, 0x17, 0x79, 0x06, 0x78, 0x28, 0x8d, 0x89, 0x3a,
0x4c, 0x03, 0x25, 0xbe, 0x71, 0x6a, 0x5c, 0x0b, 0xe7, 0x84, 0x60, 0xa4, 0x99,
0x22, 0xe3, 0xd2, 0xaf, 0x84, 0xa4, 0xa7, 0xfb, 0xd1, 0x98, 0xed, 0x0c, 0xa9,
0xde, 0x94, 0x89, 0xe1, 0x0e, 0xa0, 0xdc, 0xc0, 0xce, 0x99, 0x3d, 0xea, 0x08,
0x52, 0xbb, 0x56, 0x79, 0xe4, 0x1f, 0x84, 0xba, 0x1e, 0xb8, 0xb4, 0xc4, 0x49,
0x5c, 0x4f, 0x31, 0x4b, 0x87, 0xdd, 0xdd, 0x05, 0x67, 0x26, 0x99, 0x80, 0xe0,
0x71, 0x11, 0xa3, 0xb8, 0xa5, 0x41, 0xe2, 0xa4, 0x53, 0xb9, 0xf7, 0x32, 0x29,
0x83, 0x0c, 0x13, 0xbf, 0x36, 0x5e, 0x04, 0xb3, 0x4b, 0x43, 0x47, 0x2f, 0x6b,
0xe2, 0x91, 0x1e, 0xd3, 0x98, 0x4f, 0xdd, 0x42, 0x07, 0xc8, 0xe8, 0x1d, 0x12,
0xfc, 0x99, 0xa9, 0x6b, 0x3e, 0x92, 0x7e, 0xc8, 0xd6, 0x69, 0x3a, 0xfc, 0x64,
0xbd, 0xb6, 0x09, 0x9d, 0xca, 0xfd, 0x0c, 0x0b, 0xa2, 0x9b, 0x77, 0x60, 0x4b,
0x03, 0x94, 0xa4, 0x30, 0x69, 0x12, 0xd6, 0x42, 0x2d, 0xc1, 0x41, 0x4c, 0xca,
0xdc, 0xaa, 0xfd, 0x8f, 0x5b, 0x83, 0x46, 0x9a, 0xd9, 0xfc, 0xb1, 0xd1, 0xe3,
0xb3, 0xc9, 0x7f, 0x48, 0x7a, 0xcd, 0x24, 0xf0, 0x41, 0x8f, 0x5c, 0x74, 0xd0,
0xac, 0xb0, 0x10, 0x20, 0x06, 0x49, 0xb7, 0xc7, 0x2d, 0x21, 0xc8, 0x57, 0xe3,
0xd0, 0x86, 0xf3, 0x03, 0x68, 0xfb, 0xd0, 0xce, 0x71, 0xc1, 0x89, 0x99, 0x4a,
0x64, 0x01, 0x6c, 0xfd, 0xec, 0x30, 0x91, 0xcf, 0x41, 0x3c, 0x92, 0xc7, 0xe5,
0xba, 0x86, 0x1d, 0x61, 0x84, 0xc7, 0x5f, 0x83, 0x39, 0x62, 0xae, 0xb4, 0x92,
0x2f, 0x47, 0xf3, 0x0b, 0xf8, 0x55, 0xeb, 0xa0, 0x1f, 0x59, 0xd0, 0xbb, 0x74,
0x9b, 0x1e, 0xd0, 0x76, 0xe6, 0xf2, 0xe9, 0x06, 0xd7, 0x10, 0xe8, 0xfa, 0x64,
0xde, 0x69, 0xc6, 0x35, 0x96, 0x88, 0x02, 0xf0, 0x46, 0xb8, 0x3f, 0x27, 0x99,
0x6f, 0xcb, 0x71, 0x89, 0x29, 0x35, 0xf7, 0x48, 0x16, 0x02, 0x35, 0x8f, 0xd5,
0x79, 0x7c, 0x4d, 0x02, 0xcf, 0x5f, 0xeb, 0x8a, 0x83, 0x4f, 0x45, 0x71, 0x88,
0xf9, 0xa9, 0x0d, 0x4e, 0x72, 0xe9, 0xc2, 0x9c, 0x07, 0xcf, 0x49, 0x1b, 0x4e,
0x04, 0x0e, 0x63, 0x51, 0x8c, 0x5e, 0xd8, 0x00, 0xc1, 0x55, 0x2c, 0xb6, 0xc6,
0xe0, 0xc2, 0x65, 0x4e, 0xc9, 0x34, 0x39, 0xf5, 0x9c, 0xb3, 0xc4, 0x7e, 0xe8,
0x61, 0x6e, 0x13, 0x5f, 0x15, 0xc4, 0x5f, 0xd9, 0x7e, 0xed, 0x1d, 0xce, 0xee,
0x44, 0xec, 0xcb, 0x2e, 0x86, 0xb1, 0xec, 0x38, 0xf6, 0x70, 0xed, 0xab, 0x5c,
0x13, 0xc1, 0xd9, 0x0f, 0x0d, 0xc7, 0x80, 0xb2, 0x55, 0xed, 0x34, 0xf7, 0xac,
0x9b, 0xe4, 0xc3, 0xda, 0xe7, 0x47, 0x3c, 0xa6, 0xb5, 0x8f, 0x31, 0xdf, 0xc5,
0x4b, 0xaf, 0xeb, 0xf1, 0x02, 0x03, 0x01, 0x00, 0x01, 0xa3, 0x51, 0x30, 0x4f,
0x30, 0x0b, 0x06, 0x03, 0x55, 0x1d, 0x0f, 0x04, 0x04, 0x03, 0x02, 0x01, 0x86,
0x30, 0x0f, 0x06, 0x03, 0x55, 0x1d, 0x13, 0x01, 0x01, 0xff, 0x04, 0x05, 0x30,
0x03, 0x01, 0x01, 0xff, 0x30, 0x1d, 0x06, 0x03, 0x55, 0x1d, 0x0e, 0x04, 0x16,
0x04, 0x14, 0x72, 0x2d, 0x3a, 0x02, 0x31, 0x90, 0x43, 0xb9, 0x14, 0x05, 0x4e,
0xe1, 0xea, 0xa7, 0xc7, 0x31, 0xd1, 0x23, 0x89, 0x34, 0x30, 0x10, 0x06, 0x09,
0x2b, 0x06, 0x01, 0x04, 0x01, 0x82, 0x37, 0x15, 0x01, 0x04, 0x03, 0x02, 0x01,
0x00, 0x30, 0x0d, 0x06, 0x09, 0x2a, 0x86, 0x48, 0x86, 0xf7, 0x0d, 0x01, 0x01,
0x0b, 0x05, 0x00, 0x03, 0x82, 0x02, 0x01, 0x00, 0x7f, 0x72, 0xcf, 0x0f, 0xb7,
0xc5, 0x15, 0xdb, 0x9b, 0xc0, 0x49, 0xca, 0x26, 0x5b, 0xfe, 0x9e, 0x13, 0xe6,
0xd3, 0xf0, 0xd2, 0xdb, 0x97, 0x5f, 0xf2, 0x4b, 0x3f, 0x4d, 0xb3, 0xae, 0x19,
0xae, 0xed, 0xd7, 0x97, 0xa0, 0xac, 0xef, 0xa9, 0x3a, 0xa3, 0xc2, 0x41, 0xb0,
0xe5, 0xb8, 0x91, 0x9e, 0x13, 0x81, 0x24, 0x03, 0xe6, 0x09, 0xfd, 0x3f, 0x57,
0x40, 0x39, 0x21, 0x24, 0x56, 0xd1, 0x10, 0x2f, 0x4b, 0x40, 0xa9, 0x36, 0x86,
0x4b, 0xb4, 0x53, 0x57, 0x9a, 0xfb, 0xf1, 0x7e, 0x89, 0x8f, 0x11, 0xfe, 0x18,
0x6c, 0x51, 0xaa, 0xe8, 0xed, 0x09, 0x95, 0xb5, 0xe5, 0x71, 0xc9, 0xa1, 0xe9,
0x87, 0x75, 0xa6, 0x15, 0x7f, 0xc9, 0x7e, 0x37, 0x54, 0x5e, 0x74, 0x93, 0xc5,
0xc3, 0x67, 0xcc, 0x0d, 0x4f, 0x6b, 0xa8, 0x17, 0x0c, 0x6d, 0x08, 0x92, 0x7e,
0x8b, 0xdd, 0x81, 0xaa, 0x2d, 0x70, 0x21, 0xc3, 0x3d, 0x06, 0x14, 0xbb, 0xbf,
0x24, 0x5e, 0xa7, 0x84, 0xd7, 0x3f, 0x0f, 0x21, 0x22, 0xbd, 0x4b, 0x00, 0x06,
0xdb, 0x97, 0x1c, 0xd8, 0x5e, 0xd4, 0xc5, 0x0b, 0x5c, 0x87, 0x6e, 0x50, 0xa4,
0xe8, 0xc3, 0x38, 0xa4, 0xfb, 0xcb, 0x2c, 0xc5, 0x92, 0x66, 0x9b, 0x85, 0x5e,
0xcb, 0x7a, 0x6c, 0x93, 0x7c, 0x80, 0x29, 0x58, 0x5b, 0x57, 0xb5, 0x40, 0x69,
0xba, 0x08, 0x79, 0xa6, 0x64, 0x62, 0x15, 0x9d, 0x87, 0x96, 0x45, 0xb5, 0x66,
0x23, 0x20, 0x03, 0x8b, 0x1c, 0x73, 0xa0, 0xd3, 0xa2, 0x79, 0x33, 0xe0, 0x50,
0x59, 0x86, 0xdb, 0x2f, 0xe5, 0x02, 0x25, 0xea, 0x73, 0x2a, 0x9f, 0x00, 0x14,
0xc8, 0x36, 0xc7, 0x92, 0x3b, 0xe9, 0x4e, 0x00, 0xec, 0xd8, 0x56, 0x09, 0xb9,
0x33, 0x49, 0x12, 0xd2, 0x54, 0x0b, 0x01, 0xab, 0xac, 0x47, 0xb6, 0x91, 0x29,
0x7d, 0x4c, 0xb4, 0x75, 0x80, 0x52, 0x01, 0xe8, 0xca, 0x82, 0xf6, 0x9f, 0xcc,
0xac, 0x9c, 0x8f, 0x17, 0xea, 0x2f, 0x26, 0xb0, 0xab, 0x72, 0xac, 0x0b, 0xfe,
0x9e, 0x51, 0x1e, 0xc7, 0x43, 0x55, 0x67, 0x4f, 0x51, 0xb3, 0x57, 0xd6, 0xb6,
0xec, 0xee, 0x52, 0xb7, 0x3a, 0xe9, 0x4e, 0xe1, 0xd7, 0x81, 0x88, 0xbc, 0x4f,
0x8e, 0x75, 0xbb, 0x4b, 0xa8, 0xf0, 0x35, 0xaa, 0x26, 0xd4, 0x67, 0x67, 0x49,
0xb2, 0x70, 0x4c, 0x3b, 0x93, 0xdc, 0x1d, 0xdf, 0x78, 0x90, 0x86, 0x72, 0xb2,
0x38, 0xa4, 0xd1, 0xdc, 0x92, 0x4d, 0xc9, 0x58, 0xeb, 0x2b, 0x12, 0x5c, 0xd4,
0x3b, 0xae, 0x8c, 0x6b, 0xb0, 0x83, 0xe5, 0x01, 0x3f, 0xf8, 0x09, 0x32, 0xf6,
0x93, 0x35, 0x34, 0x22, 0xaf, 0xdd, 0x37, 0x0d, 0x77, 0x09, 0x80, 0x2b, 0xcd,
0x48, 0x00, 0xf1, 0x8c, 0x99, 0x19, 0x47, 0x05, 0x01, 0xe9, 0xd1, 0xbf, 0xd1,
0x4e, 0xd0, 0xe6, 0x28, 0x43, 0x37, 0x99, 0xa4, 0x0a, 0x4a, 0x08, 0xd9, 0x9a,
0x71, 0x73, 0xd2, 0xaa, 0xcd, 0x31, 0x13, 0x63, 0x76, 0xa1, 0x37, 0x6f, 0x92,
0x38, 0x1e, 0x7d, 0x12, 0x3c, 0x66, 0x32, 0xe7, 0xcb, 0x6d, 0xe1, 0xfc, 0x52,
0x89, 0xdd, 0xca, 0xd6, 0x66, 0x05, 0x9a, 0x96, 0x61, 0xbe, 0xa2, 0x28, 0xc7,
0x1c, 0xa3, 0xa7, 0x36, 0x50, 0x3c, 0x3a, 0xa4, 0xdf, 0x4a, 0x6e, 0xe6, 0x87,
0x3b, 0xce, 0xeb, 0xf0, 0xe0, 0x81, 0x37, 0x9d, 0x13, 0x3c, 0x52, 0x8e, 0xbd,
0xb9, 0x1d, 0x34, 0xc6, 0x1d, 0xd5, 0x0a, 0x6a, 0x3d, 0x98, 0x29, 0x70, 0x8c,
0x89, 0x2a, 0xd1, 0xab, 0x82, 0x10, 0x48, 0x1f, 0xdc, 0xf4, 0xef, 0xa5, 0xc5,
0xbb, 0x55, 0x1a, 0x38, 0x63, 0x84, 0x4e, 0xb7, 0x6c, 0xad, 0x95, 0x54, 0xec,
0x65, 0x22, 0x10, 0x49, 0x17, 0xb8, 0xc0, 0x1e, 0xc7, 0x0f, 0xac, 0x54, 0x47
};
#define TEST_CERT_LEN 1521
static int test_store_open_winstore(void)
{
int ret = 0;
OSSL_STORE_CTX *sctx = NULL;
OSSL_STORE_SEARCH *search = NULL;
UI_METHOD *ui_method = NULL;
OSSL_STORE_INFO *info = NULL;
X509* testcert = NULL;
const unsigned char* certptr = mscert;
/*
* Test the winstore, by opening it, searching for the MS root certificate
* and ensure that it was found. Note that we have to search by
* subject name, as winstore only allows searches by that method
*/
ret = TEST_ptr(testcert = d2i_X509(NULL, &certptr, TEST_CERT_LEN))
&& TEST_ptr(search = OSSL_STORE_SEARCH_by_name(X509_get_subject_name(testcert)))
&& TEST_ptr(ui_method = UI_create_method("DummyUI"))
&& TEST_ptr(sctx = OSSL_STORE_open_ex("org.openssl.winstore:", NULL,
NULL, ui_method, NULL, NULL,
NULL, NULL))
&& TEST_true(OSSL_STORE_find(sctx, search))
&& TEST_ptr(info = OSSL_STORE_load(sctx));
UI_destroy_method(ui_method);
OSSL_STORE_INFO_free(info);
OSSL_STORE_SEARCH_free(search);
OSSL_STORE_close(sctx);
X509_free(testcert);
return ret;
}
#endif
Fix coverity issue: CID 1466482 - Resource leak in OSSL_STORE_SEARCH_by_key_fingerprint() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 16:21:46 +08:00
static int test_store_search_by_key_fingerprint_fail(void)
{
int ret;
OSSL_STORE_SEARCH *search = NULL;
ret = TEST_ptr_null(search = OSSL_STORE_SEARCH_by_key_fingerprint(
EVP_sha256(), NULL, 0));
OSSL_STORE_SEARCH_free(search);
return ret;
}
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
static int get_params(const char *uri, const char *type)
{
EVP_PKEY *pkey = NULL;
OSSL_STORE_CTX *ctx = NULL;
OSSL_STORE_INFO *info;
int ret = 0;
Make the -inform option to be respected if possible Add OSSL_STORE_PARAM_INPUT_TYPE and make it possible to be set when OSSL_STORE_open_ex() or OSSL_STORE_attach() is called. The input type format is enforced only in case the file type file store is used. By default we use FORMAT_UNDEF meaning the input type is not enforced. Fixes #14569 Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15100)
2021-04-30 22:57:53 +08:00
ctx = OSSL_STORE_open_ex(uri, NULL, NULL, NULL, NULL, NULL, NULL, NULL);
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
if (!TEST_ptr(ctx))
goto err;
while (!OSSL_STORE_eof(ctx)
&& (info = OSSL_STORE_load(ctx)) != NULL
&& pkey == NULL) {
if (OSSL_STORE_INFO_get_type(info) == OSSL_STORE_INFO_PARAMS) {
pkey = OSSL_STORE_INFO_get1_PARAMS(info);
}
OSSL_STORE_INFO_free(info);
info = NULL;
}
if (pkey != NULL)
ret = EVP_PKEY_is_a(pkey, type);
EVP_PKEY_free(pkey);
err:
OSSL_STORE_close(ctx);
return ret;
}
static int test_store_get_params(int idx)
{
const char *type;
test/ossl_store_test.c: Adapt the use of datadir for VMS paths Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15823)
2021-06-22 13:28:26 +08:00
const char *urifmt;
TEST: Fix path length in test/ossl_store_test.c The URI length was set to 80 chars, but the URI being built up may need more space, all depending on the paths used to get to the files that are to be loaded. If the result needs more than 80 chars, the test will fail. Fixed by using PATH_MAX. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> Reviewed-by: Paul Dale <paul.dale@oracle.com> (Merged from https://github.com/openssl/openssl/pull/13546)
2020-11-27 15:08:08 +08:00
char uri[PATH_MAX];
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
fix some code with obvious wrong coding style Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Richard Levitte <levitte@openssl.org> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/16918)
2021-10-26 15:16:18 +08:00
switch (idx) {
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
#ifndef OPENSSL_NO_DH
case 0:
type = "DH";
break;
case 1:
type = "DHX";
break;
#else
case 0:
case 1:
return 1;
#endif
case 2:
#ifndef OPENSSL_NO_DSA
type = "DSA";
break;
#else
return 1;
#endif
default:
TEST_error("Invalid test index");
return 0;
}
test/ossl_store_test.c: Adapt the use of datadir for VMS paths Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15823)
2021-06-22 13:28:26 +08:00
urifmt = "%s/%s-params.pem";
#ifdef __VMS
{
char datadir_end = datadir[strlen(datadir) - 1];
if (datadir_end == ':' || datadir_end == ']' || datadir_end == '>')
urifmt = "%s%s-params.pem";
}
#endif
if (!TEST_true(BIO_snprintf(uri, sizeof(uri), urifmt, datadir, type)))
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
return 0;
TEST_info("Testing uri: %s", uri);
if (!TEST_true(get_params(uri, type)))
return 0;
return 1;
}
STORE: clear err after ossl_store_get0_loader_int This commit clears the error that might have been set when ossl_store_get0_loader_int has been called as it will try to retrieve a loader for the scheme on an empty store, which will cause the error OSSL_STORE_R_UNREGISTERED_SCHEME to be set. The motivation for this after returning from ossl_store_get0_loader_int, OSSL_STORE_attach will continue and try to fetch a OSSL_STORE_LOADER from the provider. Reviewed-by: Richard Levitte <levitte@openssl.org> Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12901)
2020-09-17 15:48:29 +08:00
/*
* This test verifies that calling OSSL_STORE_ATTACH does not set an
* "unregistered scheme" error when called.
*/
static int test_store_attach_unregistered_scheme(void)
{
int ret;
Add a real type for OSSL_CORE_BIO which is distinct from and not castable to BIO Providers (particularly the FIPS provider) needs access to BIOs from libcrypto. Libcrypto is allowed to change the internal format of the BIO structure and it is still expected to work with providers that were already built. This means that the libcrypto BIO must be distinct from and not castable to the provider side OSSL_CORE_BIO. Unfortunately, this requirement was broken in both directions. This fixes things by forcing the two to be different and any casts break loudly. Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-04 11:53:53 +08:00
OSSL_STORE_CTX *store_ctx = NULL;
OSSL_PROVIDER *provider = NULL;
OSSL_LIB_CTX *libctx = NULL;
BIO *bio = NULL;
char *input = test_mk_file_path(inputdir, sm2file);
ret = TEST_ptr(input)
&& TEST_ptr(libctx = OSSL_LIB_CTX_new())
&& TEST_ptr(provider = OSSL_PROVIDER_load(libctx, "default"))
&& TEST_ptr(bio = BIO_new_file(input, "r"))
&& TEST_ptr(store_ctx = OSSL_STORE_attach(bio, "file", libctx, NULL,
Make the -inform option to be respected if possible Add OSSL_STORE_PARAM_INPUT_TYPE and make it possible to be set when OSSL_STORE_open_ex() or OSSL_STORE_attach() is called. The input type format is enforced only in case the file type file store is used. By default we use FORMAT_UNDEF meaning the input type is not enforced. Fixes #14569 Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/15100)
2021-04-30 22:57:53 +08:00
NULL, NULL, NULL, NULL, NULL))
Add a real type for OSSL_CORE_BIO which is distinct from and not castable to BIO Providers (particularly the FIPS provider) needs access to BIOs from libcrypto. Libcrypto is allowed to change the internal format of the BIO structure and it is still expected to work with providers that were already built. This means that the libcrypto BIO must be distinct from and not castable to the provider side OSSL_CORE_BIO. Unfortunately, this requirement was broken in both directions. This fixes things by forcing the two to be different and any casts break loudly. Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-04 11:53:53 +08:00
&& TEST_int_ne(ERR_GET_LIB(ERR_peek_error()), ERR_LIB_OSSL_STORE)
&& TEST_int_ne(ERR_GET_REASON(ERR_peek_error()),
OSSL_STORE_R_UNREGISTERED_SCHEME);
STORE: clear err after ossl_store_get0_loader_int This commit clears the error that might have been set when ossl_store_get0_loader_int has been called as it will try to retrieve a loader for the scheme on an empty store, which will cause the error OSSL_STORE_R_UNREGISTERED_SCHEME to be set. The motivation for this after returning from ossl_store_get0_loader_int, OSSL_STORE_attach will continue and try to fetch a OSSL_STORE_LOADER from the provider. Reviewed-by: Richard Levitte <levitte@openssl.org> Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12901)
2020-09-17 15:48:29 +08:00
BIO_free(bio);
OSSL_STORE_close(store_ctx);
OSSL_PROVIDER_unload(provider);
OSSL_LIB_CTX_free(libctx);
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
OPENSSL_free(input);
STORE: clear err after ossl_store_get0_loader_int This commit clears the error that might have been set when ossl_store_get0_loader_int has been called as it will try to retrieve a loader for the scheme on an empty store, which will cause the error OSSL_STORE_R_UNREGISTERED_SCHEME to be set. The motivation for this after returning from ossl_store_get0_loader_int, OSSL_STORE_attach will continue and try to fetch a OSSL_STORE_LOADER from the provider. Reviewed-by: Richard Levitte <levitte@openssl.org> Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12901)
2020-09-17 15:48:29 +08:00
return ret;
}
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
const OPTIONS *test_get_options(void)
{
static const OPTIONS test_options[] = {
OPT_TEST_OPTIONS_DEFAULT_USAGE,
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
{ "dir", OPT_INPUTDIR, '/' },
{ "in", OPT_INFILE, '<' },
{ "sm2", OPT_SM2FILE, '<' },
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
{ "data", OPT_DATADIR, 's' },
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
{ NULL }
};
return test_options;
}
int setup_tests(void)
{
OPTION_CHOICE o;
while ((o = opt_next()) != OPT_EOF) {
switch (o) {
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
case OPT_INPUTDIR:
inputdir = opt_arg();
break;
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
case OPT_INFILE:
infile = opt_arg();
break;
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
case OPT_SM2FILE:
sm2file = opt_arg();
break;
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
case OPT_DATADIR:
datadir = opt_arg();
break;
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
case OPT_TEST_CASES:
break;
default:
case OPT_ERR:
return 0;
}
}
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
if (datadir == NULL) {
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
TEST_error("No data directory specified");
return 0;
}
if (inputdir == NULL) {
TEST_error("No input directory specified");
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
return 0;
}
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
if (infile != NULL)
ADD_TEST(test_store_open);
Adding winstore open test Reviewed-by: Saša Nedvědický <sashan@openssl.org> Reviewed-by: Viktor Dukhovni <viktor@openssl.org> (Merged from https://github.com/openssl/openssl/pull/27393)
2025-04-16 02:46:27 +08:00
#ifndef OPENSSL_NO_WINSTORE
ADD_TEST(test_store_open_winstore);
#endif
Fix coverity issue: CID 1466482 - Resource leak in OSSL_STORE_SEARCH_by_key_fingerprint() Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 16:21:46 +08:00
ADD_TEST(test_store_search_by_key_fingerprint_fail);
Test that OSSL_STORE can load various types of params There have been instances where OSSL_STORE got confused between DSA and DH params (e.g. see issue #13046) due the DER encoding of DH and DSA params looking identical. Therefore we test that we get the types that we expect. Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from https://github.com/openssl/openssl/pull/13329)
2020-11-18 20:07:43 +08:00
ADD_ALL_TESTS(test_store_get_params, 3);
test: convert store test to use relative paths Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14419)
2021-03-09 08:57:05 +08:00
if (sm2file != NULL)
ADD_TEST(test_store_attach_unregistered_scheme);
Fix coverity issue: CID 1466486 - Resource leak in OSSL_STORE Note that although this is a false positive currently, it could become possible if any of the methods called change behaviour - so it is safer to add the fix than to ignore it. Added a simple test so that I could prove this was the case. Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org> (Merged from https://github.com/openssl/openssl/pull/12847)
2020-09-10 14:40:24 +08:00
return 1;
}