Fix from review

Author: embhorn

src/x509.c

@@ -11712,18 +11712,21 @@ WOLF_STACK_OF(WOLFSSL_X509_OBJECT)* wolfSSL_sk_X509_OBJECT_deep_copy(
             cert->basicConstSet = req->basicConstSet;
     #ifdef WOLFSSL_CERT_EXT
             if (req->subjKeyIdSz != 0) {
-                if (req->subjKeyIdSz <= CTC_MAX_SKID_SIZE) {
-                    if (req->subjKeyId) {
-                        XMEMCPY(cert->skid, req->subjKeyId,
-                                req->subjKeyIdSz);
-                    }
-                    cert->skidSz = (int)req->subjKeyIdSz;
-                }
-                else {
+                if (req->subjKeyIdSz > CTC_MAX_SKID_SIZE) {
                     WOLFSSL_MSG("Subject Key ID too large");
                     WOLFSSL_ERROR_VERBOSE(BUFFER_E);
                     ret = WOLFSSL_FAILURE;
                 }
+                else if (req->subjKeyId == NULL) {
+                    WOLFSSL_MSG("Subject Key ID missing");
+                    WOLFSSL_ERROR_VERBOSE(BAD_FUNC_ARG);
+                    ret = WOLFSSL_FAILURE;
+                }
+                else {
+                    XMEMCPY(cert->skid, req->subjKeyId,
+                            req->subjKeyIdSz);
+                    cert->skidSz = (int)req->subjKeyIdSz;
+                }
             }
             if (req->keyUsageSet)
                 cert->keyUsage = req->keyUsage;

wolfssl/version.h

@@ -28,8 +28,8 @@
 extern "C" {
 #endif
 
-#define LIBWOLFSSL_VERSION_STRING "5.9.0"
-#define LIBWOLFSSL_VERSION_HEX 0x05009000
+#define LIBWOLFSSL_VERSION_STRING "5.9.1"
+#define LIBWOLFSSL_VERSION_HEX 0x05009001
 
 #ifdef __cplusplus
 }