crypto/x509/x509_cmp.c:393: error: NULL_DEREFERENCE
  pointer `xk` last assigned on line 392 could be null and is dereferenced by call to `EVP_PKEY_cmp()` at line 393, column 10.
Showing all 86 steps of the trace


crypto/x509/x509_cmp.c:387:1: start of procedure X509_check_private_key()
385.   	}
386.   
387. > int X509_check_private_key(X509 *x, EVP_PKEY *k)
388.   	{
389.   	EVP_PKEY *xk=NULL;

crypto/x509/x509_cmp.c:389:2: 
387.   int X509_check_private_key(X509 *x, EVP_PKEY *k)
388.   	{
389. > 	EVP_PKEY *xk=NULL;
390.   	int ok=0;
391.   

crypto/x509/x509_cmp.c:390:2: 
388.   	{
389.   	EVP_PKEY *xk=NULL;
390. > 	int ok=0;
391.   
392.   	xk=X509_get_pubkey(x);

crypto/x509/x509_cmp.c:392:2: 
390.   	int ok=0;
391.   
392. > 	xk=X509_get_pubkey(x);
393.   	switch (EVP_PKEY_cmp(xk, k))
394.   		{

crypto/x509/x509_cmp.c:374:1: start of procedure X509_get_pubkey()
  372.   	}
  373.   
  374. > EVP_PKEY *X509_get_pubkey(X509 *x)
  375.   	{
  376.   	if ((x == NULL) || (x->cert_info == NULL))

crypto/x509/x509_cmp.c:376:7: Taking false branch
  374.   EVP_PKEY *X509_get_pubkey(X509 *x)
  375.   	{
  376.   	if ((x == NULL) || (x->cert_info == NULL))
               ^
  377.   		return(NULL);
  378.   	return(X509_PUBKEY_get(x->cert_info->key));

crypto/x509/x509_cmp.c:376:22: Taking false branch
  374.   EVP_PKEY *X509_get_pubkey(X509 *x)
  375.   	{
  376.   	if ((x == NULL) || (x->cert_info == NULL))
                              ^
  377.   		return(NULL);
  378.   	return(X509_PUBKEY_get(x->cert_info->key));

crypto/x509/x509_cmp.c:378:2: 
  376.   	if ((x == NULL) || (x->cert_info == NULL))
  377.   		return(NULL);
  378. > 	return(X509_PUBKEY_get(x->cert_info->key));
  379.   	}
  380.   

crypto/asn1/x_pubkey.c:254:1: start of procedure X509_PUBKEY_get()
    252.   	}
    253.   
    254. > EVP_PKEY *X509_PUBKEY_get(X509_PUBKEY *key)
    255.   	{
    256.   	EVP_PKEY *ret=NULL;

crypto/asn1/x_pubkey.c:256:2: 
    254.   EVP_PKEY *X509_PUBKEY_get(X509_PUBKEY *key)
    255.   	{
    256. > 	EVP_PKEY *ret=NULL;
    257.   	long j;
    258.   	int type;

crypto/asn1/x_pubkey.c:265:6: Taking false branch
    263.   #endif
    264.   
    265.   	if (key == NULL) goto err;
                ^
    266.   
    267.   	if (key->pkey != NULL)

crypto/asn1/x_pubkey.c:267:6: Taking false branch
    265.   	if (key == NULL) goto err;
    266.   
    267.   	if (key->pkey != NULL)
                ^
    268.   		{
    269.   		CRYPTO_add(&key->pkey->references, 1, CRYPTO_LOCK_EVP_PKEY);

crypto/asn1/x_pubkey.c:273:6: Taking false branch
    271.   		}
    272.   
    273.   	if (key->public_key == NULL) goto err;
                ^
    274.   
    275.   	type=OBJ_obj2nid(key->algor->algorithm);

crypto/asn1/x_pubkey.c:275:2: 
    273.   	if (key->public_key == NULL) goto err;
    274.   
    275. > 	type=OBJ_obj2nid(key->algor->algorithm);
    276.   	if ((ret = EVP_PKEY_new()) == NULL)
    277.   		{

crypto/objects/obj_dat.c:368:1: start of procedure OBJ_obj2nid()
      366.   	}
      367.   
      368. > int OBJ_obj2nid(const ASN1_OBJECT *a)
      369.   	{
      370.   	ASN1_OBJECT **op;

crypto/objects/obj_dat.c:373:6: Taking false branch
      371.   	ADDED_OBJ ad,*adp;
      372.   
      373.   	if (a == NULL)
                  ^
      374.   		return(NID_undef);
      375.   	if (a->nid != 0)

crypto/objects/obj_dat.c:375:6: Taking false branch
      373.   	if (a == NULL)
      374.   		return(NID_undef);
      375.   	if (a->nid != 0)
                  ^
      376.   		return(a->nid);
      377.   

crypto/objects/obj_dat.c:378:6: Taking true branch
      376.   		return(a->nid);
      377.   
      378.   	if (added != NULL)
                  ^
      379.   		{
      380.   		ad.type=ADDED_DATA;

crypto/objects/obj_dat.c:380:3: 
      378.   	if (added != NULL)
      379.   		{
      380. > 		ad.type=ADDED_DATA;
      381.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
      382.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);

crypto/objects/obj_dat.c:381:3: 
      379.   		{
      380.   		ad.type=ADDED_DATA;
      381. > 		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
      382.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
      383.   		if (adp != NULL) return (adp->obj->nid);

crypto/objects/obj_dat.c:382:3: 
      380.   		ad.type=ADDED_DATA;
      381.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
      382. > 		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
      383.   		if (adp != NULL) return (adp->obj->nid);
      384.   		}

crypto/lhash/lhash.c:248:1: start of procedure lh_retrieve()
        246.   	}
        247.   
        248. > void *lh_retrieve(LHASH *lh, const void *data)
        249.   	{
        250.   	unsigned long hash;

crypto/lhash/lhash.c:254:2: 
        252.   	void *ret;
        253.   
        254. > 	lh->error=0;
        255.   	rn=getrn(lh,data,&hash);
        256.   

crypto/lhash/lhash.c:255:2: Skipping getrn(): empty list of specs
        253.   
        254.   	lh->error=0;
        255.   	rn=getrn(lh,data,&hash);
                ^
        256.   
        257.   	if (*rn == NULL)

crypto/lhash/lhash.c:257:6: Taking false branch
        255.   	rn=getrn(lh,data,&hash);
        256.   
        257.   	if (*rn == NULL)
                    ^
        258.   		{
        259.   		lh->num_retrieve_miss++;

crypto/lhash/lhash.c:264:3: 
        262.   	else
        263.   		{
        264. > 		ret= (*rn)->data;
        265.   		lh->num_retrieve++;
        266.   		}

crypto/lhash/lhash.c:265:3: 
        263.   		{
        264.   		ret= (*rn)->data;
        265. > 		lh->num_retrieve++;
        266.   		}
        267.   	return(ret);

crypto/lhash/lhash.c:267:2: 
        265.   		lh->num_retrieve++;
        266.   		}
        267. > 	return(ret);
        268.   	}
        269.   

crypto/lhash/lhash.c:268:2: return from a call to lh_retrieve
        266.   		}
        267.   	return(ret);
        268.   	}
                ^
        269.   
        270.   static void doall_util_fn(LHASH *lh, int use_arg, LHASH_DOALL_FN_TYPE func,

crypto/objects/obj_dat.c:383:7: Taking true branch
      381.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
      382.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
      383.   		if (adp != NULL) return (adp->obj->nid);
                   ^
      384.   		}
      385.   	op=(ASN1_OBJECT **)OBJ_bsearch((const char *)&a,(const char *)obj_objs,

crypto/objects/obj_dat.c:383:20: 
      381.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
      382.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
      383. > 		if (adp != NULL) return (adp->obj->nid);
      384.   		}
      385.   	op=(ASN1_OBJECT **)OBJ_bsearch((const char *)&a,(const char *)obj_objs,

crypto/objects/obj_dat.c:390:2: return from a call to OBJ_obj2nid
      388.   		return(NID_undef);
      389.   	return((*op)->nid);
      390.   	}
              ^
      391.   
      392.   /* Convert an object name into an ASN1_OBJECT

crypto/asn1/x_pubkey.c:276:6: 
    274.   
    275.   	type=OBJ_obj2nid(key->algor->algorithm);
    276. > 	if ((ret = EVP_PKEY_new()) == NULL)
    277.   		{
    278.   		X509err(X509_F_X509_PUBKEY_GET, ERR_R_MALLOC_FAILURE);

crypto/evp/p_lib.c:307:1: start of procedure EVP_PKEY_new()
      305.   	}
      306.   
      307. > EVP_PKEY *EVP_PKEY_new(void)
      308.   	{
      309.   	EVP_PKEY *ret;

crypto/evp/p_lib.c:311:2: 
      309.   	EVP_PKEY *ret;
      310.   
      311. > 	ret=(EVP_PKEY *)OPENSSL_malloc(sizeof(EVP_PKEY));
      312.   	if (ret == NULL)
      313.   		{

crypto/mem.c:291:1: start of procedure CRYPTO_malloc()
        289.   	}
        290.   
        291. > void *CRYPTO_malloc(int num, const char *file, int line)
        292.   	{
        293.   	void *ret = NULL;

crypto/mem.c:293:2: 
        291.   void *CRYPTO_malloc(int num, const char *file, int line)
        292.   	{
        293. > 	void *ret = NULL;
        294.   	extern unsigned char cleanse_ctr;
        295.   

crypto/mem.c:296:6: Taking false branch
        294.   	extern unsigned char cleanse_ctr;
        295.   
        296.   	if (num <= 0) return NULL;
                    ^
        297.   
        298.   	allow_customize = 0;

crypto/mem.c:298:2: 
        296.   	if (num <= 0) return NULL;
        297.   
        298. > 	allow_customize = 0;
        299.   	if (malloc_debug_func != NULL)
        300.   		{

crypto/mem.c:299:6: Taking true branch
        297.   
        298.   	allow_customize = 0;
        299.   	if (malloc_debug_func != NULL)
                    ^
        300.   		{
        301.   		allow_customize_debug = 0;

crypto/mem.c:301:3: 
        299.   	if (malloc_debug_func != NULL)
        300.   		{
        301. > 		allow_customize_debug = 0;
        302.   		malloc_debug_func(NULL, num, file, line, 0);
        303.   		}

crypto/mem.c:302:3: Skipping __function_pointer__(): unresolved function pointer
        300.   		{
        301.   		allow_customize_debug = 0;
        302.   		malloc_debug_func(NULL, num, file, line, 0);
                 ^
        303.   		}
        304.   	ret = malloc_ex_func(num,file,line);

crypto/mem.c:304:2: Skipping __function_pointer__(): unresolved function pointer
        302.   		malloc_debug_func(NULL, num, file, line, 0);
        303.   		}
        304.   	ret = malloc_ex_func(num,file,line);
                ^
        305.   #ifdef LEVITTE_DEBUG_MEM
        306.   	fprintf(stderr, "LEVITTE_DEBUG_MEM:         > 0x%p (%d)\n", ret, num);

crypto/mem.c:308:6: Taking true branch
        306.   	fprintf(stderr, "LEVITTE_DEBUG_MEM:         > 0x%p (%d)\n", ret, num);
        307.   #endif
        308.   	if (malloc_debug_func != NULL)
                    ^
        309.   		malloc_debug_func(ret, num, file, line, 1);
        310.   

crypto/mem.c:309:3: Skipping __function_pointer__(): unresolved function pointer
        307.   #endif
        308.   	if (malloc_debug_func != NULL)
        309.   		malloc_debug_func(ret, num, file, line, 1);
                 ^
        310.   
        311.           /* Create a dependency on the value of 'cleanse_ctr' so our memory

crypto/mem.c:314:12: Taking false branch
        312.            * sanitisation function can't be optimised out. NB: We only do
        313.            * this for >2Kb so the overhead doesn't bother us. */
        314.           if(ret && (num > 2048))
                          ^
        315.                   ((unsigned char *)ret)[0] = cleanse_ctr;
        316.   

crypto/mem.c:317:2: 
        315.                   ((unsigned char *)ret)[0] = cleanse_ctr;
        316.   
        317. > 	return ret;
        318.   	}
        319.   

crypto/mem.c:318:2: return from a call to CRYPTO_malloc
        316.   
        317.   	return ret;
        318.   	}
                ^
        319.   
        320.   void *CRYPTO_realloc(void *str, int num, const char *file, int line)

crypto/evp/p_lib.c:312:6: Taking true branch
      310.   
      311.   	ret=(EVP_PKEY *)OPENSSL_malloc(sizeof(EVP_PKEY));
      312.   	if (ret == NULL)
                  ^
      313.   		{
      314.   		EVPerr(EVP_F_EVP_PKEY_NEW,ERR_R_MALLOC_FAILURE);

crypto/evp/p_lib.c:314:3: 
      312.   	if (ret == NULL)
      313.   		{
      314. > 		EVPerr(EVP_F_EVP_PKEY_NEW,ERR_R_MALLOC_FAILURE);
      315.   		return(NULL);
      316.   		}

crypto/err/err.c:666:1: start of procedure ERR_put_error()
        664.   /********************************************************/
        665.   
        666. > void ERR_put_error(int lib, int func, int reason, const char *file,
        667.   	     int line)
        668.   	{

crypto/err/err.c:690:2: Skipping ERR_get_state(): empty list of specs
        688.   	}
        689.   #endif
        690.   	es=ERR_get_state();
                ^
        691.   
        692.   	es->top=(es->top+1)%ERR_NUM_ERRORS;

crypto/err/err.c:692:2: 
        690.   	es=ERR_get_state();
        691.   
        692. > 	es->top=(es->top+1)%ERR_NUM_ERRORS;
        693.   	if (es->top == es->bottom)
        694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;

crypto/err/err.c:693:6: Taking false branch
        691.   
        692.   	es->top=(es->top+1)%ERR_NUM_ERRORS;
        693.   	if (es->top == es->bottom)
                    ^
        694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
        695.   	es->err_flags[es->top]=0;

crypto/err/err.c:695:2: 
        693.   	if (es->top == es->bottom)
        694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
        695. > 	es->err_flags[es->top]=0;
        696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
        697.   	es->err_file[es->top]=file;

crypto/err/err.c:696:2: 
        694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
        695.   	es->err_flags[es->top]=0;
        696. > 	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
        697.   	es->err_file[es->top]=file;
        698.   	es->err_line[es->top]=line;

crypto/err/err.c:697:2: 
        695.   	es->err_flags[es->top]=0;
        696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
        697. > 	es->err_file[es->top]=file;
        698.   	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);

crypto/err/err.c:698:2: 
        696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
        697.   	es->err_file[es->top]=file;
        698. > 	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);
        700.   	}

crypto/err/err.c:699:2: Taking true branch
        697.   	es->err_file[es->top]=file;
        698.   	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);
                ^
        700.   	}
        701.   

crypto/err/err.c:699:2: Taking false branch
        697.   	es->err_file[es->top]=file;
        698.   	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);
                ^
        700.   	}
        701.   

crypto/err/err.c:699:2: Loop condition is false. Leaving loop
        697.   	es->err_file[es->top]=file;
        698.   	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);
                ^
        700.   	}
        701.   

crypto/err/err.c:700:2: return from a call to ERR_put_error
        698.   	es->err_line[es->top]=line;
        699.   	err_clear_data(es,es->top);
        700.   	}
                ^
        701.   
        702.   void ERR_clear_error(void)

crypto/evp/p_lib.c:315:3: 
      313.   		{
      314.   		EVPerr(EVP_F_EVP_PKEY_NEW,ERR_R_MALLOC_FAILURE);
      315. > 		return(NULL);
      316.   		}
      317.   	ret->type=EVP_PKEY_NONE;

crypto/evp/p_lib.c:323:2: return from a call to EVP_PKEY_new
      321.   	ret->save_parameters=1;
      322.   	return(ret);
      323.   	}
              ^
      324.   
      325.   int EVP_PKEY_assign(EVP_PKEY *pkey, int type, char *key)

crypto/asn1/x_pubkey.c:276:6: Taking true branch
    274.   
    275.   	type=OBJ_obj2nid(key->algor->algorithm);
    276.   	if ((ret = EVP_PKEY_new()) == NULL)
                ^
    277.   		{
    278.   		X509err(X509_F_X509_PUBKEY_GET, ERR_R_MALLOC_FAILURE);

crypto/asn1/x_pubkey.c:278:3: 
    276.   	if ((ret = EVP_PKEY_new()) == NULL)
    277.   		{
    278. > 		X509err(X509_F_X509_PUBKEY_GET, ERR_R_MALLOC_FAILURE);
    279.   		goto err;
    280.   		}

crypto/err/err.c:666:1: start of procedure ERR_put_error()
      664.   /********************************************************/
      665.   
      666. > void ERR_put_error(int lib, int func, int reason, const char *file,
      667.   	     int line)
      668.   	{

crypto/err/err.c:690:2: Skipping ERR_get_state(): empty list of specs
      688.   	}
      689.   #endif
      690.   	es=ERR_get_state();
              ^
      691.   
      692.   	es->top=(es->top+1)%ERR_NUM_ERRORS;

crypto/err/err.c:692:2: 
      690.   	es=ERR_get_state();
      691.   
      692. > 	es->top=(es->top+1)%ERR_NUM_ERRORS;
      693.   	if (es->top == es->bottom)
      694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;

crypto/err/err.c:693:6: Taking false branch
      691.   
      692.   	es->top=(es->top+1)%ERR_NUM_ERRORS;
      693.   	if (es->top == es->bottom)
                  ^
      694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
      695.   	es->err_flags[es->top]=0;

crypto/err/err.c:695:2: 
      693.   	if (es->top == es->bottom)
      694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
      695. > 	es->err_flags[es->top]=0;
      696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
      697.   	es->err_file[es->top]=file;

crypto/err/err.c:696:2: 
      694.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
      695.   	es->err_flags[es->top]=0;
      696. > 	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
      697.   	es->err_file[es->top]=file;
      698.   	es->err_line[es->top]=line;

crypto/err/err.c:697:2: 
      695.   	es->err_flags[es->top]=0;
      696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
      697. > 	es->err_file[es->top]=file;
      698.   	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);

crypto/err/err.c:698:2: 
      696.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
      697.   	es->err_file[es->top]=file;
      698. > 	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);
      700.   	}

crypto/err/err.c:699:2: Taking true branch
      697.   	es->err_file[es->top]=file;
      698.   	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);
              ^
      700.   	}
      701.   

crypto/err/err.c:699:2: Taking false branch
      697.   	es->err_file[es->top]=file;
      698.   	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);
              ^
      700.   	}
      701.   

crypto/err/err.c:699:2: Loop condition is false. Leaving loop
      697.   	es->err_file[es->top]=file;
      698.   	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);
              ^
      700.   	}
      701.   

crypto/err/err.c:700:2: return from a call to ERR_put_error
      698.   	es->err_line[es->top]=line;
      699.   	err_clear_data(es,es->top);
      700.   	}
              ^
      701.   
      702.   void ERR_clear_error(void)

crypto/asn1/x_pubkey.c:369:1: 
    367.   	CRYPTO_add(&ret->references, 1, CRYPTO_LOCK_EVP_PKEY);
    368.   	return(ret);
    369. > err:
    370.   	if (ret != NULL)
    371.   		EVP_PKEY_free(ret);

crypto/asn1/x_pubkey.c:370:6: Taking false branch
    368.   	return(ret);
    369.   err:
    370.   	if (ret != NULL)
                ^
    371.   		EVP_PKEY_free(ret);
    372.   	return(NULL);

crypto/asn1/x_pubkey.c:372:2: 
    370.   	if (ret != NULL)
    371.   		EVP_PKEY_free(ret);
    372. > 	return(NULL);
    373.   	}
    374.   

crypto/asn1/x_pubkey.c:373:2: return from a call to X509_PUBKEY_get
    371.   		EVP_PKEY_free(ret);
    372.   	return(NULL);
    373.   	}
            ^
    374.   
    375.   /* Now two pseudo ASN1 routines that take an EVP_PKEY structure

crypto/x509/x509_cmp.c:379:2: return from a call to X509_get_pubkey
  377.   		return(NULL);
  378.   	return(X509_PUBKEY_get(x->cert_info->key));
  379.   	}
          ^
  380.   
  381.   ASN1_BIT_STRING *X509_get0_pubkey_bitstr(const X509 *x)

crypto/x509/x509_cmp.c:393:2: 
391.   
392.   	xk=X509_get_pubkey(x);
393. > 	switch (EVP_PKEY_cmp(xk, k))
394.   		{
395.   	case 1:

crypto/evp/p_lib.c:255:1: start of procedure EVP_PKEY_cmp()
  253.   	}
  254.   
  255. > int EVP_PKEY_cmp(const EVP_PKEY *a, const EVP_PKEY *b)
  256.   	{
  257.   	if (a->type != b->type)

crypto/evp/p_lib.c:257:6: 
  255.   int EVP_PKEY_cmp(const EVP_PKEY *a, const EVP_PKEY *b)
  256.   	{
  257. > 	if (a->type != b->type)
  258.   		return -1;
  259.   
