diff options
| author | Viktor Dukhovni <openssl-users@dukhovni.org> | 2018-12-09 18:37:56 -0500 |
|---|---|---|
| committer | Viktor Dukhovni <openssl-users@dukhovni.org> | 2018-12-09 22:02:48 -0500 |
| commit | 1057c2c39f5df45c36c0fc4d78dc9d3b91f78bc6 (patch) | |
| tree | 0e20cb671cd05d2574c9134721faf47efbb9bf3a /test/hmactest.c | |
| parent | 9b340281873643d2b8a33047dc8bfa607f7e0c3c (diff) | |
Cleaner disposal of ephemeral engine ids and names
Engine names and ids are typically static strings. If an application
actually dynamically allocated these, the application owns the
storage, and should dispose of it via the original handle, rather
than the "const char *" returned by the engine.
In any case, this resolves the test code issue without resort to
"unconst" macros/casts.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'test/hmactest.c')
| -rw-r--r-- | test/hmactest.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/hmactest.c b/test/hmactest.c index 529670c2c9..893e6169b4 100644 --- a/test/hmactest.c +++ b/test/hmactest.c @@ -38,7 +38,7 @@ static struct test_st { "e9139d1e6ee064ef8cf514fc7dc83e86", }, { - "\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b", + "\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b\x0b", 16, "Hi There", 8, "9294727a3638bb1c13f48ef8158bfc9d", }, @@ -47,7 +47,7 @@ static struct test_st { "750c783e6ab0b503eaa86e310a5db738", }, { - "\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa", + "\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa\xaa", 16, { 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, 0xdd, |