[commons-crypto] branch master updated: Validate return value from OpenSslNativeJna.ENGINE_free()

[ggregory]

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-crypto.git


The following commit(s) were added to refs/heads/master by this push:
     new 500889b  Validate return value from OpenSslNativeJna.ENGINE_free()
500889b is described below

commit 500889bd1a35cde97b165a37145ceb794c5039e6
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Mon Dec 12 21:40:22 2022 -0500

    Validate return value from OpenSslNativeJna.ENGINE_free()
---
 src/changes/changes.xml                                                 | 1 +
 src/main/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandom.java | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 311c3a3..0dd1ec4 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -76,6 +76,7 @@
       <action issue="CRYPTO-160" type="fix" dev="ggregory" due-to="Gary 
Gregory">Package-private class JavaCryptoRandom extends Random but should 
not.</action>
       <action                    type="fix" dev="ggregory" due-to="Gary 
Gregory">Validate return value from 
OpenSslNativeJna.EVP_CIPHER_CTX_set_padding().</action>
       <action                    type="fix" dev="ggregory" due-to="Gary 
Gregory">Validate return value from OpenSslNativeJna.ENGINE_finish().</action>
+      <action                    type="fix" dev="ggregory" due-to="Gary 
Gregory">Validate return value from OpenSslNativeJna.ENGINE_free().</action>
       <!-- ADD -->
       <action                    type="fix" dev="ggregory" due-to="Gary 
Gregory, Dependabot">Add github/codeql-action 2 #159.</action>
       <action                    type="fix" dev="ggregory" due-to="Gary 
Gregory, Dependabot">Add AES utility class.</action>
diff --git 
a/src/main/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandom.java 
b/src/main/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandom.java
index 4604aab..a8b4be9 100644
--- a/src/main/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandom.java
+++ b/src/main/java/org/apache/commons/crypto/jna/OpenSslJnaCryptoRandom.java
@@ -134,7 +134,7 @@ final class OpenSslJnaCryptoRandom implements CryptoRandom {
 
         if (rdrandEngine != null) {
             throwOnError(OpenSslNativeJna.ENGINE_finish(rdrandEngine));
-            OpenSslNativeJna.ENGINE_free(rdrandEngine);
+            throwOnError(OpenSslNativeJna.ENGINE_free(rdrandEngine));
         }
     }