crypto/x509/x509_vfy.c:1207: error: NULL_DEREFERENCE
  pointer `crl_issuer` last assigned on line 1206 could be null and is dereferenced by call to `X509_get_subject_name()` at line 1207, column 27.
Showing all 52 steps of the trace


crypto/x509/x509_vfy.c:1184:1: start of procedure crl_akid_check()
1182.   }
1183.   
1184. > static void crl_akid_check(X509_STORE_CTX *ctx, X509_CRL *crl,
1185.                              X509 **pissuer, int *pcrl_score)
1186.   {

crypto/x509/x509_vfy.c:1187:5: 
1185.                              X509 **pissuer, int *pcrl_score)
1186.   {
1187. >     X509 *crl_issuer = NULL;
1188.       X509_NAME *cnm = X509_CRL_get_issuer(crl);
1189.       int cidx = ctx->error_depth;

crypto/x509/x509_vfy.c:1188:5: 
1186.   {
1187.       X509 *crl_issuer = NULL;
1188. >     X509_NAME *cnm = X509_CRL_get_issuer(crl);
1189.       int cidx = ctx->error_depth;
1190.       int i;

crypto/x509/x509cset.c:106:1: start of procedure X509_CRL_get_issuer()
  104.   #endif
  105.   
  106. > X509_NAME *X509_CRL_get_issuer(const X509_CRL *crl)
  107.   {
  108.       return crl->crl.issuer;

crypto/x509/x509cset.c:108:5: 
  106.   X509_NAME *X509_CRL_get_issuer(const X509_CRL *crl)
  107.   {
  108. >     return crl->crl.issuer;
  109.   }
  110.   

crypto/x509/x509cset.c:109:1: return from a call to X509_CRL_get_issuer
  107.   {
  108.       return crl->crl.issuer;
  109. > }
  110.   
  111.   const STACK_OF(X509_EXTENSION) *X509_CRL_get0_extensions(const X509_CRL *crl)

crypto/x509/x509_vfy.c:1189:5: 
1187.       X509 *crl_issuer = NULL;
1188.       X509_NAME *cnm = X509_CRL_get_issuer(crl);
1189. >     int cidx = ctx->error_depth;
1190.       int i;
1191.   

crypto/x509/x509_vfy.c:1192:9: 
1190.       int i;
1191.   
1192. >     if (cidx != sk_X509_num(ctx->chain) - 1)
1193.           cidx++;
1194.   

include/openssl/x509.h:100:1: start of procedure sk_X509_num()
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/stack/stack.c:277:1: start of procedure OPENSSL_sk_num()
    275.   }
    276.   
    277. > int OPENSSL_sk_num(const OPENSSL_STACK *st)
    278.   {
    279.       if (st == NULL)

crypto/stack/stack.c:279:9: Taking true branch
    277.   int OPENSSL_sk_num(const OPENSSL_STACK *st)
    278.   {
    279.       if (st == NULL)
                   ^
    280.           return -1;
    281.       return st->num;

crypto/stack/stack.c:280:9: 
    278.   {
    279.       if (st == NULL)
    280. >         return -1;
    281.       return st->num;
    282.   }

crypto/stack/stack.c:282:1: return from a call to OPENSSL_sk_num
    280.           return -1;
    281.       return st->num;
    282. > }
    283.   
    284.   void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)

include/openssl/x509.h:100:1: return from a call to sk_X509_num
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:1192:9: Taking true branch
1190.       int i;
1191.   
1192.       if (cidx != sk_X509_num(ctx->chain) - 1)
                ^
1193.           cidx++;
1194.   

crypto/x509/x509_vfy.c:1193:9: 
1191.   
1192.       if (cidx != sk_X509_num(ctx->chain) - 1)
1193. >         cidx++;
1194.   
1195.       crl_issuer = sk_X509_value(ctx->chain, cidx);

crypto/x509/x509_vfy.c:1195:5: 
1193.           cidx++;
1194.   
1195. >     crl_issuer = sk_X509_value(ctx->chain, cidx);
1196.   
1197.       if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {

include/openssl/x509.h:100:1: start of procedure sk_X509_value()
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/stack/stack.c:284:1: start of procedure OPENSSL_sk_value()
    282.   }
    283.   
    284. > void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)

crypto/stack/stack.c:286:9: Taking true branch
    284.   void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)
                   ^
    287.           return NULL;
    288.       return (void *)st->data[i];

crypto/stack/stack.c:287:9: 
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)
    287. >         return NULL;
    288.       return (void *)st->data[i];
    289.   }

crypto/stack/stack.c:289:1: return from a call to OPENSSL_sk_value
    287.           return NULL;
    288.       return (void *)st->data[i];
    289. > }
    290.   
    291.   void *OPENSSL_sk_set(OPENSSL_STACK *st, int i, const void *data)

include/openssl/x509.h:100:1: return from a call to sk_X509_value
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:1197:9: 
1195.       crl_issuer = sk_X509_value(ctx->chain, cidx);
1196.   
1197. >     if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
1198.           if (*pcrl_score & CRL_SCORE_ISSUER_NAME) {
1199.               *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_ISSUER_CERT;

crypto/x509v3/v3_purp.c:780:1: start of procedure X509_check_akid()
  778.   }
  779.   
  780. > int X509_check_akid(X509 *issuer, AUTHORITY_KEYID *akid)
  781.   {
  782.   

crypto/x509v3/v3_purp.c:783:10: Taking false branch
  781.   {
  782.   
  783.       if (!akid)
                  ^
  784.           return X509_V_OK;
  785.   

crypto/x509v3/v3_purp.c:787:9: Taking false branch
  785.   
  786.       /* Check key ids (if present) */
  787.       if (akid->keyid && issuer->skid &&
                 ^
  788.           ASN1_OCTET_STRING_cmp(akid->keyid, issuer->skid))
  789.           return X509_V_ERR_AKID_SKID_MISMATCH;

crypto/x509v3/v3_purp.c:791:9: Taking false branch
  789.           return X509_V_ERR_AKID_SKID_MISMATCH;
  790.       /* Check serial number */
  791.       if (akid->serial &&
                 ^
  792.           ASN1_INTEGER_cmp(X509_get_serialNumber(issuer), akid->serial))
  793.           return X509_V_ERR_AKID_ISSUER_SERIAL_MISMATCH;

crypto/x509v3/v3_purp.c:795:9: Taking false branch
  793.           return X509_V_ERR_AKID_ISSUER_SERIAL_MISMATCH;
  794.       /* Check issuer name */
  795.       if (akid->issuer) {
                 ^
  796.           /*
  797.            * Ugh, for some peculiar reason AKID includes SEQUENCE OF

crypto/x509v3/v3_purp.c:816:5: 
  814.               return X509_V_ERR_AKID_ISSUER_SERIAL_MISMATCH;
  815.       }
  816. >     return X509_V_OK;
  817.   }
  818.   

crypto/x509v3/v3_purp.c:817:1: return from a call to X509_check_akid
  815.       }
  816.       return X509_V_OK;
  817. > }
  818.   
  819.   uint32_t X509_get_extension_flags(X509 *x)

crypto/x509/x509_vfy.c:1197:9: Taking true branch
1195.       crl_issuer = sk_X509_value(ctx->chain, cidx);
1196.   
1197.       if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
                ^
1198.           if (*pcrl_score & CRL_SCORE_ISSUER_NAME) {
1199.               *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_ISSUER_CERT;

crypto/x509/x509_vfy.c:1198:13: Taking false branch
1196.   
1197.       if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
1198.           if (*pcrl_score & CRL_SCORE_ISSUER_NAME) {
                    ^
1199.               *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_ISSUER_CERT;
1200.               *pissuer = crl_issuer;

crypto/x509/x509_vfy.c:1205:10: 
1203.       }
1204.   
1205. >     for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
1206.           crl_issuer = sk_X509_value(ctx->chain, cidx);
1207.           if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))

crypto/x509/x509_vfy.c:1205:18: 
1203.       }
1204.   
1205. >     for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
1206.           crl_issuer = sk_X509_value(ctx->chain, cidx);
1207.           if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))

include/openssl/x509.h:100:1: start of procedure sk_X509_num()
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/stack/stack.c:277:1: start of procedure OPENSSL_sk_num()
    275.   }
    276.   
    277. > int OPENSSL_sk_num(const OPENSSL_STACK *st)
    278.   {
    279.       if (st == NULL)

crypto/stack/stack.c:279:9: Taking true branch
    277.   int OPENSSL_sk_num(const OPENSSL_STACK *st)
    278.   {
    279.       if (st == NULL)
                   ^
    280.           return -1;
    281.       return st->num;

crypto/stack/stack.c:280:9: 
    278.   {
    279.       if (st == NULL)
    280. >         return -1;
    281.       return st->num;
    282.   }

crypto/stack/stack.c:282:1: return from a call to OPENSSL_sk_num
    280.           return -1;
    281.       return st->num;
    282. > }
    283.   
    284.   void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)

include/openssl/x509.h:100:1: return from a call to sk_X509_num
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:1205:18: Loop condition is true. Entering loop body
1203.       }
1204.   
1205.       for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
                         ^
1206.           crl_issuer = sk_X509_value(ctx->chain, cidx);
1207.           if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))

crypto/x509/x509_vfy.c:1206:9: 
1204.   
1205.       for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
1206. >         crl_issuer = sk_X509_value(ctx->chain, cidx);
1207.           if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))
1208.               continue;

include/openssl/x509.h:100:1: start of procedure sk_X509_value()
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/stack/stack.c:284:1: start of procedure OPENSSL_sk_value()
    282.   }
    283.   
    284. > void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)

crypto/stack/stack.c:286:9: Taking true branch
    284.   void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)
                   ^
    287.           return NULL;
    288.       return (void *)st->data[i];

crypto/stack/stack.c:287:9: 
    285.   {
    286.       if (st == NULL || i < 0 || i >= st->num)
    287. >         return NULL;
    288.       return (void *)st->data[i];
    289.   }

crypto/stack/stack.c:289:1: return from a call to OPENSSL_sk_value
    287.           return NULL;
    288.       return (void *)st->data[i];
    289. > }
    290.   
    291.   void *OPENSSL_sk_set(OPENSSL_STACK *st, int i, const void *data)

include/openssl/x509.h:100:1: return from a call to sk_X509_value
  98.   typedef struct x509_cinf_st X509_CINF;
  99.   
  100. > DEFINE_STACK_OF(X509)
  101.   
  102.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:1207:13: 
1205.       for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
1206.           crl_issuer = sk_X509_value(ctx->chain, cidx);
1207. >         if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))
1208.               continue;
1209.           if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {

crypto/x509/x509_cmp.c:99:1: start of procedure X509_get_subject_name()
  97.   #endif
  98.   
  99. > X509_NAME *X509_get_subject_name(const X509 *a)
  100.   {
  101.       return (a->cert_info.subject);

crypto/x509/x509_cmp.c:101:5: 
  99.   X509_NAME *X509_get_subject_name(const X509 *a)
  100.   {
  101. >     return (a->cert_info.subject);
  102.   }
  103.   

static void crl_akid_check(X509_STORE_CTX *ctx, X509_CRL *crl,
                           X509 **pissuer, int *pcrl_score)
{
    X509 *crl_issuer = NULL;
    X509_NAME *cnm = X509_CRL_get_issuer(crl);
    int cidx = ctx->error_depth;
    int i;
    if (cidx != sk_X509_num(ctx->chain) - 1)
        cidx++;
    crl_issuer = sk_X509_value(ctx->chain, cidx);
    if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
        if (*pcrl_score & CRL_SCORE_ISSUER_NAME) {
            *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_ISSUER_CERT;
            *pissuer = crl_issuer;
            return;
        }
    }
    for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {
        crl_issuer = sk_X509_value(ctx->chain, cidx);
        if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))
            continue;
        if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
            *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_SAME_PATH;
            *pissuer = crl_issuer;
            return;
        }
    }
    if (!(ctx->param->flags & X509_V_FLAG_EXTENDED_CRL_SUPPORT))
        return;
    for (i = 0; i < sk_X509_num(ctx->untrusted); i++) {
        crl_issuer = sk_X509_value(ctx->untrusted, i);
        if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))
            continue;
        if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {
            *pissuer = crl_issuer;
            *pcrl_score |= CRL_SCORE_AKID;
            return;
        }
    }
}
['static void crl_akid_check(X509_STORE_CTX *ctx, X509_CRL *crl,\n                           X509 **pissuer, int *pcrl_score)\n{\n    X509 *crl_issuer = NULL;\n    X509_NAME *cnm = X509_CRL_get_issuer(crl);\n    int cidx = ctx->error_depth;\n    int i;\n    if (cidx != sk_X509_num(ctx->chain) - 1)\n        cidx++;\n    crl_issuer = sk_X509_value(ctx->chain, cidx);\n    if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {\n        if (*pcrl_score & CRL_SCORE_ISSUER_NAME) {\n            *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_ISSUER_CERT;\n            *pissuer = crl_issuer;\n            return;\n        }\n    }\n    for (cidx++; cidx < sk_X509_num(ctx->chain); cidx++) {\n        crl_issuer = sk_X509_value(ctx->chain, cidx);\n        if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))\n            continue;\n        if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {\n            *pcrl_score |= CRL_SCORE_AKID | CRL_SCORE_SAME_PATH;\n            *pissuer = crl_issuer;\n            return;\n        }\n    }\n    if (!(ctx->param->flags & X509_V_FLAG_EXTENDED_CRL_SUPPORT))\n        return;\n    for (i = 0; i < sk_X509_num(ctx->untrusted); i++) {\n        crl_issuer = sk_X509_value(ctx->untrusted, i);\n        if (X509_NAME_cmp(X509_get_subject_name(crl_issuer), cnm))\n            continue;\n        if (X509_check_akid(crl_issuer, crl->akid) == X509_V_OK) {\n            *pissuer = crl_issuer;\n            *pcrl_score |= CRL_SCORE_AKID;\n            return;\n        }\n    }\n}', 'X509_NAME *X509_CRL_get_issuer(const X509_CRL *crl)\n{\n    return crl->crl.issuer;\n}', 'DEFINE_STACK_OF(X509)', 'int OPENSSL_sk_num(const OPENSSL_STACK *st)\n{\n    if (st == NULL)\n        return -1;\n    return st->num;\n}', 'void *OPENSSL_sk_value(const OPENSSL_STACK *st, int i)\n{\n    if (st == NULL || i < 0 || i >= st->num)\n        return NULL;\n    return (void *)st->data[i];\n}', 'int X509_check_akid(X509 *issuer, AUTHORITY_KEYID *akid)\n{\n    if (!akid)\n        return X509_V_OK;\n    if (akid->keyid && issuer->skid &&\n        ASN1_OCTET_STRING_cmp(akid->keyid, issuer->skid))\n        return X509_V_ERR_AKID_SKID_MISMATCH;\n    if (akid->serial &&\n        ASN1_INTEGER_cmp(X509_get_serialNumber(issuer), akid->serial))\n        return X509_V_ERR_AKID_ISSUER_SERIAL_MISMATCH;\n    if (akid->issuer) {\n        GENERAL_NAMES *gens;\n        GENERAL_NAME *gen;\n        X509_NAME *nm = NULL;\n        int i;\n        gens = akid->issuer;\n        for (i = 0; i < sk_GENERAL_NAME_num(gens); i++) {\n            gen = sk_GENERAL_NAME_value(gens, i);\n            if (gen->type == GEN_DIRNAME) {\n                nm = gen->d.dirn;\n                break;\n            }\n        }\n        if (nm && X509_NAME_cmp(nm, X509_get_issuer_name(issuer)))\n            return X509_V_ERR_AKID_ISSUER_SERIAL_MISMATCH;\n    }\n    return X509_V_OK;\n}', 'X509_NAME *X509_get_subject_name(const X509 *a)\n{\n    return (a->cert_info.subject);\n}']