diff options
| author | tb <> | 2023-12-28 21:56:12 +0000 |
|---|---|---|
| committer | tb <> | 2023-12-28 21:56:12 +0000 |
| commit | 9c04f3e6f64642074c016ed4b3fdcbce9db722ed (patch) | |
| tree | 21d0f6dd472a206ac8bdde60864ab3910a2f4ec9 /src/lib/libcrypto/dsa/dsa_pmeth.c | |
| parent | d3667a15b63915ec5a5f0ff81ddefd2a8a818762 (diff) | |
| download | openbsd-9c04f3e6f64642074c016ed4b3fdcbce9db722ed.tar.gz openbsd-9c04f3e6f64642074c016ed4b3fdcbce9db722ed.tar.bz2 openbsd-9c04f3e6f64642074c016ed4b3fdcbce9db722ed.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/dsa/dsa_pmeth.c')
0 files changed, 0 insertions, 0 deletions