[gnutls-devel] GnuTLS | fips: Zeroize temporary values (!1797)

Read-only notification of GnuTLS library development activities gnutls-devel at lists.gnutls.org
Fri Jan 12 11:26:21 CET 2024



Clemens Lang created a merge request: https://gitlab.com/gnutls/gnutls/-/merge_requests/1797

Project:Branches: cllang/gnutls:cllang-fips-zeroization to gnutls/gnutls:master
Author:   Clemens Lang




The standard says "temporary value(s) generated during the integrity
test of the module's software […] shall be zeroised from the module upon
completion of the integrity test".

That includes the computed HMAC value, which is currently not zeroized
after the test. Add explicit calls to gnutls_memset() to fix that.

Signed-off-by: Clemens Lang <cllang at redhat.com>

## Checklist
 * [x] Commits have `Signed-off-by:` with name/author being identical to the commit author
 * [x] Code modified for feature
 * [ ] ~Test suite updated with functionality tests~
 * [ ] ~Test suite updated with negative tests~
 * [ ] Documentation updated / NEWS entry present (for non-trivial changes)
 * [ ] CI timeout is 2h or higher (see Settings/CICD/General pipelines/Timeout)

## Reviewer's checklist:
 * [ ] Any issues marked for closing are addressed
 * [ ] There is a test suite reasonably covering new functionality or modifications
 * [ ] Function naming, parameters, return values, types, etc., are consistent and according to `CONTRIBUTION.md`
 * [ ] This feature/change has adequate documentation added
 * [ ] No obvious mistakes in the code

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/merge_requests/1797
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20240112/b8cde586/attachment.html>


More information about the Gnutls-devel mailing list