crypto/x509/x509_vfy.c:772: error: NULL_DEREFERENCE
  pointer `x` last assigned on line 771 could be null and is dereferenced by call to `lookup_cert_match()` at line 772, column 14.
Showing all 31 steps of the trace


crypto/x509/x509_vfy.c:718:1: start of procedure check_trust()
716.   }
717.   
718. > static int check_trust(X509_STORE_CTX *ctx, int num_untrusted)
719.   {
720.       int i;

crypto/x509/x509_vfy.c:721:5: 
719.   {
720.       int i;
721. >     X509 *x = NULL;
722.       X509 *mx;
723.       SSL_DANE *dane = ctx->dane;

crypto/x509/x509_vfy.c:723:5: 
721.       X509 *x = NULL;
722.       X509 *mx;
723. >     SSL_DANE *dane = ctx->dane;
724.       int num = sk_X509_num(ctx->chain);
725.       int trust;

crypto/x509/x509_vfy.c:724:5: 
722.       X509 *mx;
723.       SSL_DANE *dane = ctx->dane;
724. >     int num = sk_X509_num(ctx->chain);
725.       int trust;
726.   

include/openssl/x509.h:97:1: start of procedure sk_X509_num()
  95.   typedef struct x509_cinf_st X509_CINF;
  96.   
  97. > DEFINE_STACK_OF(X509)
  98.   
  99.   /* 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:97:1: return from a call to sk_X509_num
  95.   typedef struct x509_cinf_st X509_CINF;
  96.   
  97. > DEFINE_STACK_OF(X509)
  98.   
  99.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:731:9: Taking true branch
729.        * match, we're done, otherwise we'll merely record the match depth.
730.        */
731.       if (DANETLS_HAS_TA(dane) && num_untrusted > 0 && num_untrusted < num) {
               ^
732.           switch (trust = check_dane_issuer(ctx, num_untrusted)) {
733.           case X509_TRUST_TRUSTED:

crypto/x509/x509_vfy.c:731:9: Taking false branch
729.        * match, we're done, otherwise we'll merely record the match depth.
730.        */
731.       if (DANETLS_HAS_TA(dane) && num_untrusted > 0 && num_untrusted < num) {
               ^
732.           switch (trust = check_dane_issuer(ctx, num_untrusted)) {
733.           case X509_TRUST_TRUSTED:

crypto/x509/x509_vfy.c:745:10: 
743.        * and wants to incrementally check just any added since.
744.        */
745. >     for (i = num_untrusted; i < num; i++) {
746.           x = sk_X509_value(ctx->chain, i);
747.           trust = X509_check_trust(x, ctx->param->trust, 0);

crypto/x509/x509_vfy.c:745:29: Loop condition is false. Leaving loop
743.        * and wants to incrementally check just any added since.
744.        */
745.       for (i = num_untrusted; i < num; i++) {
                                   ^
746.           x = sk_X509_value(ctx->chain, i);
747.           trust = X509_check_trust(x, ctx->param->trust, 0);

crypto/x509/x509_vfy.c:759:9: Taking false branch
757.        * the chain is PKIX trusted.
758.        */
759.       if (num_untrusted < num) {
               ^
760.           if (ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN)
761.               goto trusted;

crypto/x509/x509_vfy.c:765:9: Taking true branch
763.       }
764.   
765.       if (num_untrusted == num && ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN) {
               ^
766.           /*
767.            * Last-resort call with no new trusted certificates, check the leaf

crypto/x509/x509_vfy.c:765:33: Taking true branch
763.       }
764.   
765.       if (num_untrusted == num && ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN) {
                                       ^
766.           /*
767.            * Last-resort call with no new trusted certificates, check the leaf

crypto/x509/x509_vfy.c:770:9: 
768.            * for a direct trust store match.
769.            */
770. >         i = 0;
771.           x = sk_X509_value(ctx->chain, i);
772.           mx = lookup_cert_match(ctx, x);

crypto/x509/x509_vfy.c:771:9: 
769.            */
770.           i = 0;
771. >         x = sk_X509_value(ctx->chain, i);
772.           mx = lookup_cert_match(ctx, x);
773.           if (!mx)

include/openssl/x509.h:97:1: start of procedure sk_X509_value()
  95.   typedef struct x509_cinf_st X509_CINF;
  96.   
  97. > DEFINE_STACK_OF(X509)
  98.   
  99.   /* 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:97:1: return from a call to sk_X509_value
  95.   typedef struct x509_cinf_st X509_CINF;
  96.   
  97. > DEFINE_STACK_OF(X509)
  98.   
  99.   /* This is used for a table of trust checking functions */

crypto/x509/x509_vfy.c:772:9: 
770.           i = 0;
771.           x = sk_X509_value(ctx->chain, i);
772. >         mx = lookup_cert_match(ctx, x);
773.           if (!mx)
774.               return X509_TRUST_UNTRUSTED;

crypto/x509/x509_vfy.c:125:1: start of procedure lookup_cert_match()
  123.   /* Given a certificate try and find an exact match in the store */
  124.   
  125. > static X509 *lookup_cert_match(X509_STORE_CTX *ctx, X509 *x)
  126.   {
  127.       STACK_OF(X509) *certs;

crypto/x509/x509_vfy.c:128:5: 
  126.   {
  127.       STACK_OF(X509) *certs;
  128. >     X509 *xtmp = NULL;
  129.       int i;
  130.       /* Lookup all certs with matching subject name */

crypto/x509/x509_vfy.c:131:5: 
  129.       int i;
  130.       /* Lookup all certs with matching subject name */
  131. >     certs = ctx->lookup_certs(ctx, X509_get_subject_name(x));
  132.       if (certs == NULL)
  133.           return NULL;

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

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

static int check_trust(X509_STORE_CTX *ctx, int num_untrusted)
{
    int i;
    X509 *x = NULL;
    X509 *mx;
    SSL_DANE *dane = ctx->dane;
    int num = sk_X509_num(ctx->chain);
    int trust;
    if (DANETLS_HAS_TA(dane) && num_untrusted > 0 && num_untrusted < num) {
        switch (trust = check_dane_issuer(ctx, num_untrusted)) {
        case X509_TRUST_TRUSTED:
        case X509_TRUST_REJECTED:
            return trust;
        }
    }
    for (i = num_untrusted; i < num; i++) {
        x = sk_X509_value(ctx->chain, i);
        trust = X509_check_trust(x, ctx->param->trust, 0);
        if (trust == X509_TRUST_TRUSTED)
            goto trusted;
        if (trust == X509_TRUST_REJECTED)
            goto rejected;
    }
    if (num_untrusted < num) {
        if (ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN)
            goto trusted;
        return X509_TRUST_UNTRUSTED;
    }
    if (num_untrusted == num && ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN) {
        i = 0;
        x = sk_X509_value(ctx->chain, i);
        mx = lookup_cert_match(ctx, x);
        if (!mx)
            return X509_TRUST_UNTRUSTED;
        trust = X509_check_trust(mx, ctx->param->trust, 0);
        if (trust == X509_TRUST_REJECTED) {
            X509_free(mx);
            goto rejected;
        }
        (void) sk_X509_set(ctx->chain, 0, mx);
        X509_free(x);
        ctx->num_untrusted = 0;
        goto trusted;
    }
    return X509_TRUST_UNTRUSTED;
 rejected:
    if (!verify_cb_cert(ctx, x, i, X509_V_ERR_CERT_REJECTED))
        return X509_TRUST_REJECTED;
    return X509_TRUST_UNTRUSTED;
 trusted:
    if (!DANETLS_ENABLED(dane))
        return X509_TRUST_TRUSTED;
    if (dane->pdpth < 0)
        dane->pdpth = num_untrusted;
    if (dane->mdpth >= 0)
        return X509_TRUST_TRUSTED;
    return X509_TRUST_UNTRUSTED;
}
['static int check_trust(X509_STORE_CTX *ctx, int num_untrusted)\n{\n    int i;\n    X509 *x = NULL;\n    X509 *mx;\n    SSL_DANE *dane = ctx->dane;\n    int num = sk_X509_num(ctx->chain);\n    int trust;\n    if (DANETLS_HAS_TA(dane) && num_untrusted > 0 && num_untrusted < num) {\n        switch (trust = check_dane_issuer(ctx, num_untrusted)) {\n        case X509_TRUST_TRUSTED:\n        case X509_TRUST_REJECTED:\n            return trust;\n        }\n    }\n    for (i = num_untrusted; i < num; i++) {\n        x = sk_X509_value(ctx->chain, i);\n        trust = X509_check_trust(x, ctx->param->trust, 0);\n        if (trust == X509_TRUST_TRUSTED)\n            goto trusted;\n        if (trust == X509_TRUST_REJECTED)\n            goto rejected;\n    }\n    if (num_untrusted < num) {\n        if (ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN)\n            goto trusted;\n        return X509_TRUST_UNTRUSTED;\n    }\n    if (num_untrusted == num && ctx->param->flags & X509_V_FLAG_PARTIAL_CHAIN) {\n        i = 0;\n        x = sk_X509_value(ctx->chain, i);\n        mx = lookup_cert_match(ctx, x);\n        if (!mx)\n            return X509_TRUST_UNTRUSTED;\n        trust = X509_check_trust(mx, ctx->param->trust, 0);\n        if (trust == X509_TRUST_REJECTED) {\n            X509_free(mx);\n            goto rejected;\n        }\n        (void) sk_X509_set(ctx->chain, 0, mx);\n        X509_free(x);\n        ctx->num_untrusted = 0;\n        goto trusted;\n    }\n    return X509_TRUST_UNTRUSTED;\n rejected:\n    if (!verify_cb_cert(ctx, x, i, X509_V_ERR_CERT_REJECTED))\n        return X509_TRUST_REJECTED;\n    return X509_TRUST_UNTRUSTED;\n trusted:\n    if (!DANETLS_ENABLED(dane))\n        return X509_TRUST_TRUSTED;\n    if (dane->pdpth < 0)\n        dane->pdpth = num_untrusted;\n    if (dane->mdpth >= 0)\n        return X509_TRUST_TRUSTED;\n    return X509_TRUST_UNTRUSTED;\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}', 'static X509 *lookup_cert_match(X509_STORE_CTX *ctx, X509 *x)\n{\n    STACK_OF(X509) *certs;\n    X509 *xtmp = NULL;\n    int i;\n    certs = ctx->lookup_certs(ctx, X509_get_subject_name(x));\n    if (certs == NULL)\n        return NULL;\n    for (i = 0; i < sk_X509_num(certs); i++) {\n        xtmp = sk_X509_value(certs, i);\n        if (!X509_cmp(xtmp, x))\n            break;\n    }\n    if (i < sk_X509_num(certs))\n        X509_up_ref(xtmp);\n    else\n        xtmp = NULL;\n    sk_X509_pop_free(certs, X509_free);\n    return xtmp;\n}', 'X509_NAME *X509_get_subject_name(const X509 *a)\n{\n    return (a->cert_info.subject);\n}']