crypto/pkcs7/pk7_doit.c:1112: error: NULL_DEREFERENCE
  pointer `ri` last assigned on line 1111 could be null and is dereferenced at line 1112, column 8.
Showing all 44 steps of the trace


crypto/pkcs7/pk7_doit.c:1095:1: start of procedure PKCS7_get_issuer_and_serial()
1093.   	}
1094.   
1095. > PKCS7_ISSUER_AND_SERIAL *PKCS7_get_issuer_and_serial(PKCS7 *p7, int idx)
1096.   	{
1097.   	STACK_OF(PKCS7_RECIP_INFO) *rsk;

crypto/pkcs7/pk7_doit.c:1101:2: 
1099.   	int i;
1100.   
1101. > 	i=OBJ_obj2nid(p7->type);
1102.   	if (i != NID_pkcs7_signedAndEnveloped)
1103.   		return NULL;

crypto/objects/obj_dat.c:387:1: start of procedure OBJ_obj2nid()
  385.   	}
  386.   
  387. > int OBJ_obj2nid(const ASN1_OBJECT *a)
  388.   	{
  389.   	ASN1_OBJECT **op;

crypto/objects/obj_dat.c:392:6: Taking false branch
  390.   	ADDED_OBJ ad,*adp;
  391.   
  392.   	if (a == NULL)
              ^
  393.   		return(NID_undef);
  394.   	if (a->nid != 0)

crypto/objects/obj_dat.c:394:6: Taking false branch
  392.   	if (a == NULL)
  393.   		return(NID_undef);
  394.   	if (a->nid != 0)
              ^
  395.   		return(a->nid);
  396.   

crypto/objects/obj_dat.c:397:6: Taking true branch
  395.   		return(a->nid);
  396.   
  397.   	if (added != NULL)
              ^
  398.   		{
  399.   		ad.type=ADDED_DATA;

crypto/objects/obj_dat.c:399:3: 
  397.   	if (added != NULL)
  398.   		{
  399. > 		ad.type=ADDED_DATA;
  400.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
  401.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);

crypto/objects/obj_dat.c:400:3: 
  398.   		{
  399.   		ad.type=ADDED_DATA;
  400. > 		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
  401.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
  402.   		if (adp != NULL) return (adp->obj->nid);

crypto/objects/obj_dat.c:401:3: 
  399.   		ad.type=ADDED_DATA;
  400.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
  401. > 		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
  402.   		if (adp != NULL) return (adp->obj->nid);
  403.   		}

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:402:7: Taking true branch
  400.   		ad.obj=(ASN1_OBJECT *)a; /* XXX: ugly but harmless */
  401.   		adp=(ADDED_OBJ *)lh_retrieve(added,&ad);
  402.   		if (adp != NULL) return (adp->obj->nid);
               ^
  403.   		}
  404.   	op=(ASN1_OBJECT **)OBJ_bsearch((const char *)&a,(const char *)obj_objs,

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

crypto/objects/obj_dat.c:409:2: return from a call to OBJ_obj2nid
  407.   		return(NID_undef);
  408.   	return((*op)->nid);
  409.   	}
          ^
  410.   
  411.   /* Convert an object name into an ASN1_OBJECT

crypto/pkcs7/pk7_doit.c:1102:6: Taking false branch
1100.   
1101.   	i=OBJ_obj2nid(p7->type);
1102.   	if (i != NID_pkcs7_signedAndEnveloped)
             ^
1103.   		return NULL;
1104.   	if (p7->d.signed_and_enveloped == NULL)

crypto/pkcs7/pk7_doit.c:1104:6: Taking false branch
1102.   	if (i != NID_pkcs7_signedAndEnveloped)
1103.   		return NULL;
1104.   	if (p7->d.signed_and_enveloped == NULL)
             ^
1105.   		return NULL;
1106.   	rsk=p7->d.signed_and_enveloped->recipientinfo;

crypto/pkcs7/pk7_doit.c:1106:2: 
1104.   	if (p7->d.signed_and_enveloped == NULL)
1105.   		return NULL;
1106. > 	rsk=p7->d.signed_and_enveloped->recipientinfo;
1107.   	if (rsk == NULL)
1108.   		return NULL;

crypto/pkcs7/pk7_doit.c:1107:6: Taking false branch
1105.   		return NULL;
1106.   	rsk=p7->d.signed_and_enveloped->recipientinfo;
1107.   	if (rsk == NULL)
             ^
1108.   		return NULL;
1109.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);

crypto/pkcs7/pk7_doit.c:1109:2: 
1107.   	if (rsk == NULL)
1108.   		return NULL;
1109. > 	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);
1110.   	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
1111.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);

crypto/stack/stack.c:307:1: start of procedure sk_value()
  305.   }
  306.   
  307. > char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;

crypto/stack/stack.c:309:6: Taking false branch
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
              ^
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:309:13: Taking false branch
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
                     ^
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:309:24: Taking false branch
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
                                ^
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:310:2: 
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
  310. > 	return st->data[i];
  311.   }
  312.   

crypto/stack/stack.c:311:1: return from a call to sk_value
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
  310.   	return st->data[i];
  311. > }
  312.   
  313.   char *sk_set(STACK *st, int i, char *value)

crypto/pkcs7/pk7_doit.c:1110:6: 
1108.   		return NULL;
1109.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);
1110. > 	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
1111.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);
1112.   	return(ri->issuer_and_serial);

crypto/stack/stack.c:301:1: start of procedure sk_num()
  299.   	}
  300.   
  301. > int sk_num(const STACK *st)
  302.   {
  303.   	if(st == NULL) return -1;

crypto/stack/stack.c:303:5: Taking false branch
  301.   int sk_num(const STACK *st)
  302.   {
  303.   	if(st == NULL) return -1;
             ^
  304.   	return st->num;
  305.   }

crypto/stack/stack.c:304:2: 
  302.   {
  303.   	if(st == NULL) return -1;
  304. > 	return st->num;
  305.   }
  306.   

crypto/stack/stack.c:305:1: return from a call to sk_num
  303.   	if(st == NULL) return -1;
  304.   	return st->num;
  305. > }
  306.   
  307.   char *sk_value(const STACK *st, int i)

crypto/pkcs7/pk7_doit.c:1110:6: Taking false branch
1108.   		return NULL;
1109.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);
1110.   	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
             ^
1111.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);
1112.   	return(ri->issuer_and_serial);

crypto/pkcs7/pk7_doit.c:1111:2: 
1109.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);
1110.   	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
1111. > 	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);
1112.   	return(ri->issuer_and_serial);
1113.   	}

crypto/stack/stack.c:307:1: start of procedure sk_value()
  305.   }
  306.   
  307. > char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;

crypto/stack/stack.c:309:6: Taking false branch
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
              ^
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:309:13: Taking true branch
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
                     ^
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:309:39: 
  307.   char *sk_value(const STACK *st, int i)
  308.   {
  309. > 	if(!st || (i < 0) || (i >= st->num)) return NULL;
  310.   	return st->data[i];
  311.   }

crypto/stack/stack.c:311:1: return from a call to sk_value
  309.   	if(!st || (i < 0) || (i >= st->num)) return NULL;
  310.   	return st->data[i];
  311. > }
  312.   
  313.   char *sk_set(STACK *st, int i, char *value)

crypto/pkcs7/pk7_doit.c:1112:2: 
1110.   	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
1111.   	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);
1112. > 	return(ri->issuer_and_serial);
1113.   	}
1114.   

PKCS7_ISSUER_AND_SERIAL *PKCS7_get_issuer_and_serial(PKCS7 *p7, int idx)
	{
	STACK_OF(PKCS7_RECIP_INFO) *rsk;
	PKCS7_RECIP_INFO *ri;
	int i;
	i=OBJ_obj2nid(p7->type);
	if (i != NID_pkcs7_signedAndEnveloped)
		return NULL;
	if (p7->d.signed_and_enveloped == NULL)
		return NULL;
	rsk=p7->d.signed_and_enveloped->recipientinfo;
	if (rsk == NULL)
		return NULL;
	ri=sk_PKCS7_RECIP_INFO_value(rsk,0);
	if (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);
	ri=sk_PKCS7_RECIP_INFO_value(rsk,idx);
	return(ri->issuer_and_serial);
	}
['PKCS7_ISSUER_AND_SERIAL *PKCS7_get_issuer_and_serial(PKCS7 *p7, int idx)\n\t{\n\tSTACK_OF(PKCS7_RECIP_INFO) *rsk;\n\tPKCS7_RECIP_INFO *ri;\n\tint i;\n\ti=OBJ_obj2nid(p7->type);\n\tif (i != NID_pkcs7_signedAndEnveloped)\n\t\treturn NULL;\n\tif (p7->d.signed_and_enveloped == NULL)\n\t\treturn NULL;\n\trsk=p7->d.signed_and_enveloped->recipientinfo;\n\tif (rsk == NULL)\n\t\treturn NULL;\n\tri=sk_PKCS7_RECIP_INFO_value(rsk,0);\n\tif (sk_PKCS7_RECIP_INFO_num(rsk) <= idx) return(NULL);\n\tri=sk_PKCS7_RECIP_INFO_value(rsk,idx);\n\treturn(ri->issuer_and_serial);\n\t}', 'int OBJ_obj2nid(const ASN1_OBJECT *a)\n\t{\n\tASN1_OBJECT **op;\n\tADDED_OBJ ad,*adp;\n\tif (a == NULL)\n\t\treturn(NID_undef);\n\tif (a->nid != 0)\n\t\treturn(a->nid);\n\tif (added != NULL)\n\t\t{\n\t\tad.type=ADDED_DATA;\n\t\tad.obj=(ASN1_OBJECT *)a;\n\t\tadp=(ADDED_OBJ *)lh_retrieve(added,&ad);\n\t\tif (adp != NULL) return (adp->obj->nid);\n\t\t}\n\top=(ASN1_OBJECT **)OBJ_bsearch((const char *)&a,(const char *)obj_objs,\n\t\tNUM_OBJ, sizeof(ASN1_OBJECT *),obj_cmp);\n\tif (op == NULL)\n\t\treturn(NID_undef);\n\treturn((*op)->nid);\n\t}', 'void *lh_retrieve(LHASH *lh, const void *data)\n\t{\n\tunsigned long hash;\n\tLHASH_NODE **rn;\n\tvoid *ret;\n\tlh->error=0;\n\trn=getrn(lh,data,&hash);\n\tif (*rn == NULL)\n\t\t{\n\t\tlh->num_retrieve_miss++;\n\t\treturn(NULL);\n\t\t}\n\telse\n\t\t{\n\t\tret= (*rn)->data;\n\t\tlh->num_retrieve++;\n\t\t}\n\treturn(ret);\n\t}', 'char *sk_value(const STACK *st, int i)\n{\n\tif(!st || (i < 0) || (i >= st->num)) return NULL;\n\treturn st->data[i];\n}', 'int sk_num(const STACK *st)\n{\n\tif(st == NULL) return -1;\n\treturn st->num;\n}']