diff options
| author | Richard Levitte <levitte@openssl.org> | 2019-12-10 20:38:09 +0100 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2019-12-13 10:03:57 +0100 |
| commit | 97ba39547d4c79b60131475a1512cc7d7e8952b2 (patch) | |
| tree | b8b755d7dacf7bbb294485a0f0d4275193137f53 /test/namemap_internal_test.c | |
| parent | dd0139f416257ec5632414ed3ad8c61d07ba07ec (diff) | |
test/namemap_internal_test.c: use "cookie" instead of "foo"...
... in test_namemap()
Because tests may sometimes run in random order (subject of the
environment variable OPENSSL_TEST_RAND_ORDER being defined), and we're
dealing with the global namemap, each test must use names that are
globally unique for that test. Unfortunately, we used "foo" in two of
them, which might lead to surprising results.
Fixes #10401
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/10601)
Diffstat (limited to 'test/namemap_internal_test.c')
| -rw-r--r-- | test/namemap_internal_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/namemap_internal_test.c b/test/namemap_internal_test.c index 0b67cb1d3f..263364adbd 100644 --- a/test/namemap_internal_test.c +++ b/test/namemap_internal_test.c @@ -26,7 +26,7 @@ static int test_namemap(OSSL_NAMEMAP *nm) int check2 = ossl_namemap_name2num(nm, NAME2); int check3 = ossl_namemap_name2num(nm, ALIAS1); int check4 = ossl_namemap_name2num(nm, ALIAS1_UC); - int false1 = ossl_namemap_name2num(nm, "foo"); + int false1 = ossl_namemap_name2num(nm, "cookie"); return TEST_int_ne(num1, 0) && TEST_int_ne(num2, 0) |