[rs-commit] r282 - in /mod_csr/trunk: ChangeLog openssl_setter_compat.h

[rs-commit at redwax.eu]

Author: minfrin at redwax.eu
Date: Sat Feb 15 22:16:30 2020
New Revision: 282

Log:
Don't try and free an algorithm if unset.

Modified:
    mod_csr/trunk/ChangeLog
    mod_csr/trunk/openssl_setter_compat.h

Modified: mod_csr/trunk/ChangeLog
==============================================================================
--- mod_csr/trunk/ChangeLog	(original)
+++ mod_csr/trunk/ChangeLog	Sat Feb 15 22:16:30 2020
@@ -1,3 +1,7 @@
+
+Changes with v0.2.4
+
+ *) Don't try and free an algorithm if unset. [Graham Leggett]
 
 Changes with v0.2.3
 

Modified: mod_csr/trunk/openssl_setter_compat.h
==============================================================================
--- mod_csr/trunk/openssl_setter_compat.h	(original)
+++ mod_csr/trunk/openssl_setter_compat.h	Sat Feb 15 22:16:30 2020
@@ -58,10 +58,14 @@
 
 static void X509_REQ_set1_signature(X509_REQ *req, X509_ALGOR *palg)
 {
-    if (req->sig_alg->algorithm)
-        ASN1_OBJECT_free(req->sig_alg->algorithm);
-    if (req->sig_alg->parameter)
-        ASN1_TYPE_free(req->sig_alg->parameter);
+    if (req->sig_alg) {
+        if (req->sig_alg->algorithm) {
+            ASN1_OBJECT_free(req->sig_alg->algorithm);
+        }
+        if (req->sig_alg->parameter) {
+            ASN1_TYPE_free(req->sig_alg->parameter);
+        }
+    }
     req->sig_alg = palg;
 }
 #define HAVE_X509_REQ_SET1_SIGNATURE 1