crypto/x509/x509_vfy.c:846: error: NULL_DEREFERENCE
  pointer `x` last assigned on line 835 could be null and is dereferenced by call to `get_crl_delta()` at line 846, column 18.
Showing all 26 steps of the trace


crypto/x509/x509_vfy.c:828:1: start of procedure check_cert()
826.   }
827.   
828. > static int check_cert(X509_STORE_CTX *ctx)
829.   {
830.       X509_CRL *crl = NULL, *dcrl = NULL;

crypto/x509/x509_vfy.c:830:5: 
828.   static int check_cert(X509_STORE_CTX *ctx)
829.   {
830. >     X509_CRL *crl = NULL, *dcrl = NULL;
831.       X509 *x;
832.       int ok, cnum;

crypto/x509/x509_vfy.c:834:5: 
832.       int ok, cnum;
833.       unsigned int last_reasons;
834. >     cnum = ctx->error_depth;
835.       x = sk_X509_value(ctx->chain, cnum);
836.       ctx->current_cert = x;

crypto/x509/x509_vfy.c:835:9: Condition is true
833.       unsigned int last_reasons;
834.       cnum = ctx->error_depth;
835.       x = sk_X509_value(ctx->chain, cnum);
               ^
836.       ctx->current_cert = x;
837.       ctx->current_issuer = NULL;

crypto/x509/x509_vfy.c:835:5: 
833.       unsigned int last_reasons;
834.       cnum = ctx->error_depth;
835. >     x = sk_X509_value(ctx->chain, cnum);
836.       ctx->current_cert = x;
837.       ctx->current_issuer = NULL;

crypto/stack/stack.c:326:1: start of procedure sk_value()
  324.   }
  325.   
  326. > void *sk_value(const _STACK *st, int i)
  327.   {
  328.       if (!st || (i < 0) || (i >= st->num))

crypto/stack/stack.c:328:10: Taking false branch
  326.   void *sk_value(const _STACK *st, int i)
  327.   {
  328.       if (!st || (i < 0) || (i >= st->num))
                  ^
  329.           return NULL;
  330.       return st->data[i];

crypto/stack/stack.c:328:17: Taking false branch
  326.   void *sk_value(const _STACK *st, int i)
  327.   {
  328.       if (!st || (i < 0) || (i >= st->num))
                         ^
  329.           return NULL;
  330.       return st->data[i];

crypto/stack/stack.c:328:28: Taking true branch
  326.   void *sk_value(const _STACK *st, int i)
  327.   {
  328.       if (!st || (i < 0) || (i >= st->num))
                                    ^
  329.           return NULL;
  330.       return st->data[i];

crypto/stack/stack.c:329:9: 
  327.   {
  328.       if (!st || (i < 0) || (i >= st->num))
  329. >         return NULL;
  330.       return st->data[i];
  331.   }

crypto/stack/stack.c:331:1: return from a call to sk_value
  329.           return NULL;
  330.       return st->data[i];
  331. > }
  332.   
  333.   void *sk_set(_STACK *st, int i, void *value)

crypto/x509/x509_vfy.c:836:5: 
834.       cnum = ctx->error_depth;
835.       x = sk_X509_value(ctx->chain, cnum);
836. >     ctx->current_cert = x;
837.       ctx->current_issuer = NULL;
838.       ctx->current_crl_score = 0;

crypto/x509/x509_vfy.c:837:5: 
835.       x = sk_X509_value(ctx->chain, cnum);
836.       ctx->current_cert = x;
837. >     ctx->current_issuer = NULL;
838.       ctx->current_crl_score = 0;
839.       ctx->current_reasons = 0;

crypto/x509/x509_vfy.c:838:5: 
836.       ctx->current_cert = x;
837.       ctx->current_issuer = NULL;
838. >     ctx->current_crl_score = 0;
839.       ctx->current_reasons = 0;
840.       while (ctx->current_reasons != CRLDP_ALL_REASONS) {

crypto/x509/x509_vfy.c:839:5: 
837.       ctx->current_issuer = NULL;
838.       ctx->current_crl_score = 0;
839. >     ctx->current_reasons = 0;
840.       while (ctx->current_reasons != CRLDP_ALL_REASONS) {
841.           last_reasons = ctx->current_reasons;

crypto/x509/x509_vfy.c:840:12: Loop condition is true. Entering loop body
838.       ctx->current_crl_score = 0;
839.       ctx->current_reasons = 0;
840.       while (ctx->current_reasons != CRLDP_ALL_REASONS) {
                  ^
841.           last_reasons = ctx->current_reasons;
842.           /* Try to retrieve relevant CRL */

crypto/x509/x509_vfy.c:841:9: 
839.       ctx->current_reasons = 0;
840.       while (ctx->current_reasons != CRLDP_ALL_REASONS) {
841. >         last_reasons = ctx->current_reasons;
842.           /* Try to retrieve relevant CRL */
843.           if (ctx->get_crl)

crypto/x509/x509_vfy.c:843:13: Taking false branch
841.           last_reasons = ctx->current_reasons;
842.           /* Try to retrieve relevant CRL */
843.           if (ctx->get_crl)
                   ^
844.               ok = ctx->get_crl(ctx, &crl, x);
845.           else

crypto/x509/x509_vfy.c:846:13: 
844.               ok = ctx->get_crl(ctx, &crl, x);
845.           else
846. >             ok = get_crl_delta(ctx, &crl, &dcrl, x);
847.           /*
848.            * If error looking up CRL, nothing we can do except notify callback

crypto/x509/x509_vfy.c:1395:1: start of procedure get_crl_delta()
  1393.    */
  1394.   
  1395. > static int get_crl_delta(X509_STORE_CTX *ctx,
  1396.                            X509_CRL **pcrl, X509_CRL **pdcrl, X509 *x)
  1397.   {

crypto/x509/x509_vfy.c:1399:5: 
  1397.   {
  1398.       int ok;
  1399. >     X509 *issuer = NULL;
  1400.       int crl_score = 0;
  1401.       unsigned int reasons;

crypto/x509/x509_vfy.c:1400:5: 
  1398.       int ok;
  1399.       X509 *issuer = NULL;
  1400. >     int crl_score = 0;
  1401.       unsigned int reasons;
  1402.       X509_CRL *crl = NULL, *dcrl = NULL;

crypto/x509/x509_vfy.c:1402:5: 
  1400.       int crl_score = 0;
  1401.       unsigned int reasons;
  1402. >     X509_CRL *crl = NULL, *dcrl = NULL;
  1403.       STACK_OF(X509_CRL) *skcrl;
  1404.       X509_NAME *nm = X509_get_issuer_name(x);

crypto/x509/x509_vfy.c:1404:5: 
  1402.       X509_CRL *crl = NULL, *dcrl = NULL;
  1403.       STACK_OF(X509_CRL) *skcrl;
  1404. >     X509_NAME *nm = X509_get_issuer_name(x);
  1405.       reasons = ctx->current_reasons;
  1406.       ok = get_crl_sk(ctx, &crl, &dcrl,

crypto/x509/x509_cmp.c:130:1: start of procedure X509_get_issuer_name()
    128.   }
    129.   
    130. > X509_NAME *X509_get_issuer_name(X509 *a)
    131.   {
    132.       return (a->cert_info->issuer);

crypto/x509/x509_cmp.c:132:5: 
    130.   X509_NAME *X509_get_issuer_name(X509 *a)
    131.   {
    132. >     return (a->cert_info->issuer);
    133.   }
    134.   

static int check_cert(X509_STORE_CTX *ctx)
{
    X509_CRL *crl = NULL, *dcrl = NULL;
    X509 *x;
    int ok, cnum;
    unsigned int last_reasons;
    cnum = ctx->error_depth;
    x = sk_X509_value(ctx->chain, cnum);
    ctx->current_cert = x;
    ctx->current_issuer = NULL;
    ctx->current_crl_score = 0;
    ctx->current_reasons = 0;
    while (ctx->current_reasons != CRLDP_ALL_REASONS) {
        last_reasons = ctx->current_reasons;
        if (ctx->get_crl)
            ok = ctx->get_crl(ctx, &crl, x);
        else
            ok = get_crl_delta(ctx, &crl, &dcrl, x);
        if (!ok) {
            ctx->error = X509_V_ERR_UNABLE_TO_GET_CRL;
            ok = ctx->verify_cb(0, ctx);
            goto err;
        }
        ctx->current_crl = crl;
        ok = ctx->check_crl(ctx, crl);
        if (!ok)
            goto err;
        if (dcrl) {
            ok = ctx->check_crl(ctx, dcrl);
            if (!ok)
                goto err;
            ok = ctx->cert_crl(ctx, dcrl, x);
            if (!ok)
                goto err;
        } else
            ok = 1;
        if (ok != 2) {
            ok = ctx->cert_crl(ctx, crl, x);
            if (!ok)
                goto err;
        }
        X509_CRL_free(crl);
        X509_CRL_free(dcrl);
        crl = NULL;
        dcrl = NULL;
        if (last_reasons == ctx->current_reasons) {
            ctx->error = X509_V_ERR_UNABLE_TO_GET_CRL;
            ok = ctx->verify_cb(0, ctx);
            goto err;
        }
    }
 err:
    X509_CRL_free(crl);
    X509_CRL_free(dcrl);
    ctx->current_crl = NULL;
    return ok;
}
['static int check_cert(X509_STORE_CTX *ctx)\n{\n    X509_CRL *crl = NULL, *dcrl = NULL;\n    X509 *x;\n    int ok, cnum;\n    unsigned int last_reasons;\n    cnum = ctx->error_depth;\n    x = sk_X509_value(ctx->chain, cnum);\n    ctx->current_cert = x;\n    ctx->current_issuer = NULL;\n    ctx->current_crl_score = 0;\n    ctx->current_reasons = 0;\n    while (ctx->current_reasons != CRLDP_ALL_REASONS) {\n        last_reasons = ctx->current_reasons;\n        if (ctx->get_crl)\n            ok = ctx->get_crl(ctx, &crl, x);\n        else\n            ok = get_crl_delta(ctx, &crl, &dcrl, x);\n        if (!ok) {\n            ctx->error = X509_V_ERR_UNABLE_TO_GET_CRL;\n            ok = ctx->verify_cb(0, ctx);\n            goto err;\n        }\n        ctx->current_crl = crl;\n        ok = ctx->check_crl(ctx, crl);\n        if (!ok)\n            goto err;\n        if (dcrl) {\n            ok = ctx->check_crl(ctx, dcrl);\n            if (!ok)\n                goto err;\n            ok = ctx->cert_crl(ctx, dcrl, x);\n            if (!ok)\n                goto err;\n        } else\n            ok = 1;\n        if (ok != 2) {\n            ok = ctx->cert_crl(ctx, crl, x);\n            if (!ok)\n                goto err;\n        }\n        X509_CRL_free(crl);\n        X509_CRL_free(dcrl);\n        crl = NULL;\n        dcrl = NULL;\n        if (last_reasons == ctx->current_reasons) {\n            ctx->error = X509_V_ERR_UNABLE_TO_GET_CRL;\n            ok = ctx->verify_cb(0, ctx);\n            goto err;\n        }\n    }\n err:\n    X509_CRL_free(crl);\n    X509_CRL_free(dcrl);\n    ctx->current_crl = NULL;\n    return ok;\n}', 'void *sk_value(const _STACK *st, int i)\n{\n    if (!st || (i < 0) || (i >= st->num))\n        return NULL;\n    return st->data[i];\n}', 'static int get_crl_delta(X509_STORE_CTX *ctx,\n                         X509_CRL **pcrl, X509_CRL **pdcrl, X509 *x)\n{\n    int ok;\n    X509 *issuer = NULL;\n    int crl_score = 0;\n    unsigned int reasons;\n    X509_CRL *crl = NULL, *dcrl = NULL;\n    STACK_OF(X509_CRL) *skcrl;\n    X509_NAME *nm = X509_get_issuer_name(x);\n    reasons = ctx->current_reasons;\n    ok = get_crl_sk(ctx, &crl, &dcrl,\n                    &issuer, &crl_score, &reasons, ctx->crls);\n    if (ok)\n        goto done;\n    skcrl = ctx->lookup_crls(ctx, nm);\n    if (!skcrl && crl)\n        goto done;\n    get_crl_sk(ctx, &crl, &dcrl, &issuer, &crl_score, &reasons, skcrl);\n    sk_X509_CRL_pop_free(skcrl, X509_CRL_free);\n done:\n    if (crl) {\n        ctx->current_issuer = issuer;\n        ctx->current_crl_score = crl_score;\n        ctx->current_reasons = reasons;\n        *pcrl = crl;\n        *pdcrl = dcrl;\n        return 1;\n    }\n    return 0;\n}', 'X509_NAME *X509_get_issuer_name(X509 *a)\n{\n    return (a->cert_info->issuer);\n}']