]> git.ipfire.org Git - thirdparty/bind9.git/commit
wipe hmac keys correctly pre-3.0 libcrypto
authorAydın Mercan <aydin@isc.org>
Thu, 5 Feb 2026 12:01:52 +0000 (15:01 +0300)
committerAydın Mercan <aydin@isc.org>
Fri, 6 Feb 2026 11:14:43 +0000 (14:14 +0300)
commita531f00a755d6f0dd9d1c252a756707b2c3d3bd2
tree7609f5b6578f2be23ba61128f022bba2037cd1be
parent2aba6375484518227c627b540772ba5dd525d519
wipe hmac keys correctly pre-3.0 libcrypto

A lingering `sizeof` from the prototype era of !11094 caused the
key-wipe in `isc_hmac_key_destroy` to use `sizeof(key->len)` instead of
`key->len` for the length argument of `isc_safe_memwipe`.

This results in a buffer overflow of zero bytes in HMAC keys that are
less than 4 bytes. As such, the overflow can only be visibile in keys
that are less than 32-bits, which is beyond broken and creating such
keys are only possible in testing.

Therefore, this change is *not* a security fix since the conditions are
never reachable in any imaginable deployment scenario.

Builds that use OpenSSL >=3.0 are unaffected as the `sizeof` was only
remaining in pre-3.0 builds.
lib/isc/crypto/ossl1_1.c