]> git.ipfire.org Git - thirdparty/openssl.git/commitdiff
kdf_scrypt_reset(): NULLify freed pointers
authorTomas Mraz <tomas@openssl.org>
Mon, 20 Jan 2025 07:53:21 +0000 (08:53 +0100)
committerTomas Mraz <tomas@openssl.org>
Thu, 23 Jan 2025 11:07:33 +0000 (12:07 +0100)
Otherwise doublefree happens with further usage.

Reviewed-by: Paul Dale <ppzgs1@gmail.com>
Reviewed-by: Shane Lontis <shane.lontis@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/26488)

providers/implementations/kdfs/scrypt.c

index 7d9456dd742809ab7ac76b67c86dbf6fe59fe87e..659b20eda14ff93696ea4b41d6a8997c4f6ca807 100644 (file)
@@ -92,7 +92,9 @@ static void kdf_scrypt_reset(void *vctx)
     KDF_SCRYPT *ctx = (KDF_SCRYPT *)vctx;
 
     OPENSSL_free(ctx->salt);
+    ctx->salt = NULL;
     OPENSSL_clear_free(ctx->pass, ctx->pass_len);
+    ctx->pass = NULL;
     kdf_scrypt_init(ctx);
 }