The following commit has been merged in the master branch:
commit 915c9ec007810f99a5ea8be73426fc8882f615fd
Author: Mark Vitale <mvit...@sinenomine.net>
Date: Fri May 12 21:25:36 2023 -0400
rxkad: Free memory used to check rxkad response
Since its introduction with commit 7e4e06b87a09 "Derive DES/fcrypt
session key from other key types", rxkad_derive_des_key has failed to
free the memory associated with its HMAC context struct.
This results in a leak of at least 352 bytes for each rxkad challenge
response processed by an OpenAFS server when using rxkad-kdf.
Free the memory by calling HMAC_CTX_cleanup after each round of the
loop.
Discovered via Solaris libumem.so.1.
Change-Id: Ic9f130f7229c70e4eaa68ba1f0a213b4b23229cc
Reviewed-on: https://gerrit.openafs.org/15427
Tested-by: BuildBot <build...@rampaginggeek.com>
Reviewed-by: Andrew Deason <adea...@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwi...@sinenomine.net>
Reviewed-by: Benjamin Kaduk <ka...@mit.edu>
src/rxkad/ticket5.c | 1 +
1 file changed, 1 insertion(+)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
OpenAFS-cvs@openafs.org
https://lists.openafs.org/mailman/listinfo/openafs-cvs
