diff options
| author | tb <> | 2023-12-28 21:56:12 +0000 |
|---|---|---|
| committer | tb <> | 2023-12-28 21:56:12 +0000 |
| commit | 4e4593b9566b4947b4b7d631fae8797c12740aca (patch) | |
| tree | 21d0f6dd472a206ac8bdde60864ab3910a2f4ec9 /src/lib/libcrypto/crypto_init.c | |
| parent | 20547dc33c870f1d044c7fed8beb186ccdb88db0 (diff) | |
| download | openbsd-4e4593b9566b4947b4b7d631fae8797c12740aca.tar.gz openbsd-4e4593b9566b4947b4b7d631fae8797c12740aca.tar.bz2 openbsd-4e4593b9566b4947b4b7d631fae8797c12740aca.zip | |
Clean up and fix pkey_cmac_keygen()
A void pointer can be passed without any cast or assigning it to an
intermediate variable. That's one of hte puzzling things in old OpenSSL
code: there are plenty of unnecessary casts and assignments of void
pointers.
Make use of this fact and rework the function to be single exit, error
check consistently, including the EVP_PKEY_assign() call that can't
really fail and free the cmkey on exit.
Why coverity didn't flag this one is another mystery.
ok jsing
Diffstat (limited to 'src/lib/libcrypto/crypto_init.c')
0 files changed, 0 insertions, 0 deletions