Skip to content

Commit 846ef15

Browse files
authored
Merge pull request #8096 from philljj/fix_coverity_more
Fix more coverity errors.
2 parents f8fc31e + 5fd9e99 commit 846ef15

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

src/x509.c

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2445,6 +2445,7 @@ void* wolfSSL_X509_get_ext_d2i(const WOLFSSL_X509* x509, int nid, int* c,
24452445
sk = NULL;
24462446
}
24472447
}
2448+
24482449
obj = wolfSSL_ASN1_OBJECT_new();
24492450
if (obj == NULL) {
24502451
WOLFSSL_MSG("Issue creating WOLFSSL_ASN1_OBJECT struct");
@@ -2455,6 +2456,15 @@ void* wolfSSL_X509_get_ext_d2i(const WOLFSSL_X509* x509, int nid, int* c,
24552456
obj->grp = oidCertExtType;
24562457
obj->obj = (byte*)(x509->certPolicies[i]);
24572458
obj->objSz = MAX_CERTPOL_SZ;
2459+
2460+
if (wolfSSL_sk_ASN1_OBJECT_push(sk, obj) <= 0) {
2461+
WOLFSSL_MSG("Error pushing ASN1 object onto stack");
2462+
wolfSSL_ASN1_OBJECT_free(obj);
2463+
wolfSSL_sk_ASN1_OBJECT_pop_free(sk, NULL);
2464+
sk = NULL;
2465+
}
2466+
2467+
obj = NULL;
24582468
}
24592469
else {
24602470
WOLFSSL_MSG("No Cert Policy set");

0 commit comments

Comments
 (0)