apps/ca.c:3197: error: NULL_DEREFERENCE
  pointer `tmp` last assigned on line 3195 could be null and is dereferenced by call to `strchr()` at line 3197, column 6.
Showing all 44 steps of the trace


apps/ca.c:3187:1: start of procedure unpack_revinfo()
3185.   	}
3186.   
3187. > int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)
3188.   	{
3189.   	char *tmp = NULL;

apps/ca.c:3189:2: 
3187.   int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)
3188.   	{
3189. > 	char *tmp = NULL;
3190.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3191.   	int reason_code = -1;

apps/ca.c:3190:2: 
3188.   	{
3189.   	char *tmp = NULL;
3190. > 	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3191.   	int reason_code = -1;
3192.   	int i, ret = 0;

apps/ca.c:3191:2: 
3189.   	char *tmp = NULL;
3190.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3191. > 	int reason_code = -1;
3192.   	int i, ret = 0;
3193.   	ASN1_OBJECT *hold = NULL;

apps/ca.c:3192:2: 
3190.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3191.   	int reason_code = -1;
3192. > 	int i, ret = 0;
3193.   	ASN1_OBJECT *hold = NULL;
3194.   	ASN1_GENERALIZEDTIME *comp_time = NULL;

apps/ca.c:3193:2: 
3191.   	int reason_code = -1;
3192.   	int i, ret = 0;
3193. > 	ASN1_OBJECT *hold = NULL;
3194.   	ASN1_GENERALIZEDTIME *comp_time = NULL;
3195.   	tmp = BUF_strdup(str);

apps/ca.c:3194:2: 
3192.   	int i, ret = 0;
3193.   	ASN1_OBJECT *hold = NULL;
3194. > 	ASN1_GENERALIZEDTIME *comp_time = NULL;
3195.   	tmp = BUF_strdup(str);
3196.   

apps/ca.c:3195:2: 
3193.   	ASN1_OBJECT *hold = NULL;
3194.   	ASN1_GENERALIZEDTIME *comp_time = NULL;
3195. > 	tmp = BUF_strdup(str);
3196.   
3197.   	p = strchr(tmp, ',');

crypto/buffer/buffer.c:128:1: start of procedure BUF_strdup()
  126.   	}
  127.   
  128. > char *BUF_strdup(const char *str)
  129.   	{
  130.   	char *ret;

crypto/buffer/buffer.c:133:6: Taking false branch
  131.   	int n;
  132.   
  133.   	if (str == NULL) return(NULL);
              ^
  134.   
  135.   	n=strlen(str);

crypto/buffer/buffer.c:135:2: 
  133.   	if (str == NULL) return(NULL);
  134.   
  135. > 	n=strlen(str);
  136.   	ret=OPENSSL_malloc(n+1);
  137.   	if (ret == NULL) 

crypto/buffer/buffer.c:136:2: 
  134.   
  135.   	n=strlen(str);
  136. > 	ret=OPENSSL_malloc(n+1);
  137.   	if (ret == NULL) 
  138.   		{

crypto/mem.c:282:1: start of procedure CRYPTO_malloc()
    280.   	}
    281.   
    282. > void *CRYPTO_malloc(int num, const char *file, int line)
    283.   	{
    284.   	void *ret = NULL;

crypto/mem.c:284:2: 
    282.   void *CRYPTO_malloc(int num, const char *file, int line)
    283.   	{
    284. > 	void *ret = NULL;
    285.   
    286.   	allow_customize = 0;

crypto/mem.c:286:2: 
    284.   	void *ret = NULL;
    285.   
    286. > 	allow_customize = 0;
    287.   	if (malloc_debug_func != NULL)
    288.   		{

crypto/mem.c:287:6: Taking false branch
    285.   
    286.   	allow_customize = 0;
    287.   	if (malloc_debug_func != NULL)
                ^
    288.   		{
    289.   		allow_customize_debug = 0;

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

crypto/mem.c:296:6: Taking false branch
    294.   	fprintf(stderr, "LEVITTE_DEBUG_MEM:         > 0x%p (%d)\n", ret, num);
    295.   #endif
    296.   	if (malloc_debug_func != NULL)
                ^
    297.   		malloc_debug_func(ret, num, file, line, 1);
    298.   

crypto/mem.c:299:2: 
    297.   		malloc_debug_func(ret, num, file, line, 1);
    298.   
    299. > 	return ret;
    300.   	}
    301.   

crypto/mem.c:300:2: return from a call to CRYPTO_malloc
    298.   
    299.   	return ret;
    300.   	}
            ^
    301.   
    302.   void *CRYPTO_realloc(void *str, int num, const char *file, int line)

crypto/buffer/buffer.c:137:6: Taking true branch
  135.   	n=strlen(str);
  136.   	ret=OPENSSL_malloc(n+1);
  137.   	if (ret == NULL) 
              ^
  138.   		{
  139.   		BUFerr(BUF_F_BUF_STRDUP,ERR_R_MALLOC_FAILURE);

crypto/buffer/buffer.c:139:3: 
  137.   	if (ret == NULL) 
  138.   		{
  139. > 		BUFerr(BUF_F_BUF_STRDUP,ERR_R_MALLOC_FAILURE);
  140.   		return(NULL);
  141.   		}

crypto/err/err.c:616:1: start of procedure ERR_put_error()
    614.   /********************************************************/
    615.   
    616. > void ERR_put_error(int lib, int func, int reason, const char *file,
    617.   	     int line)
    618.   	{

crypto/err/err.c:640:2: Skipping ERR_get_state(): empty list of specs
    638.   	}
    639.   #endif
    640.   	es=ERR_get_state();
            ^
    641.   
    642.   	es->top=(es->top+1)%ERR_NUM_ERRORS;

crypto/err/err.c:642:2: 
    640.   	es=ERR_get_state();
    641.   
    642. > 	es->top=(es->top+1)%ERR_NUM_ERRORS;
    643.   	if (es->top == es->bottom)
    644.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;

crypto/err/err.c:643:6: Taking false branch
    641.   
    642.   	es->top=(es->top+1)%ERR_NUM_ERRORS;
    643.   	if (es->top == es->bottom)
                ^
    644.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    645.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);

crypto/err/err.c:645:2: 
    643.   	if (es->top == es->bottom)
    644.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    645. > 	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    646.   	es->err_file[es->top]=file;
    647.   	es->err_line[es->top]=line;

crypto/err/err.c:646:2: 
    644.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    645.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    646. > 	es->err_file[es->top]=file;
    647.   	es->err_line[es->top]=line;
    648.   	err_clear_data(es,es->top);

crypto/err/err.c:647:2: 
    645.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    646.   	es->err_file[es->top]=file;
    647. > 	es->err_line[es->top]=line;
    648.   	err_clear_data(es,es->top);
    649.   	}

crypto/err/err.c:648:2: Taking true branch
    646.   	es->err_file[es->top]=file;
    647.   	es->err_line[es->top]=line;
    648.   	err_clear_data(es,es->top);
            ^
    649.   	}
    650.   

crypto/err/err.c:648:2: Taking true branch
    646.   	es->err_file[es->top]=file;
    647.   	es->err_line[es->top]=line;
    648.   	err_clear_data(es,es->top);
            ^
    649.   	}
    650.   

crypto/mem.c:318:1: start of procedure CRYPTO_free()
      316.   	}
      317.   
      318. > void CRYPTO_free(void *str)
      319.   	{
      320.   	if (free_debug_func != NULL)

crypto/mem.c:320:6: Taking true branch
      318.   void CRYPTO_free(void *str)
      319.   	{
      320.   	if (free_debug_func != NULL)
                  ^
      321.   		free_debug_func(str, 0);
      322.   #ifdef LEVITTE_DEBUG_MEM

crypto/mem.c:321:3: Skipping __function_pointer__(): unresolved function pointer
      319.   	{
      320.   	if (free_debug_func != NULL)
      321.   		free_debug_func(str, 0);
               ^
      322.   #ifdef LEVITTE_DEBUG_MEM
      323.   	fprintf(stderr, "LEVITTE_DEBUG_MEM:         < 0x%p\n", str);

crypto/mem.c:325:2: Skipping __function_pointer__(): unresolved function pointer
      323.   	fprintf(stderr, "LEVITTE_DEBUG_MEM:         < 0x%p\n", str);
      324.   #endif
      325.   	free_func(str);
              ^
      326.   	if (free_debug_func != NULL)
      327.   		free_debug_func(NULL, 1);

crypto/mem.c:326:6: Taking true branch
      324.   #endif
      325.   	free_func(str);
      326.   	if (free_debug_func != NULL)
                  ^
      327.   		free_debug_func(NULL, 1);
      328.   	}

crypto/mem.c:327:3: Skipping __function_pointer__(): unresolved function pointer
      325.   	free_func(str);
      326.   	if (free_debug_func != NULL)
      327.   		free_debug_func(NULL, 1);
               ^
      328.   	}
      329.   

crypto/mem.c:326:2: 
      324.   #endif
      325.   	free_func(str);
      326. > 	if (free_debug_func != NULL)
      327.   		free_debug_func(NULL, 1);
      328.   	}

crypto/mem.c:328:2: return from a call to CRYPTO_free
      326.   	if (free_debug_func != NULL)
      327.   		free_debug_func(NULL, 1);
      328.   	}
              ^
      329.   
      330.   void *CRYPTO_remalloc(void *a, int num, const char *file, int line)

crypto/err/err.c:648:2: 
    646.   	es->err_file[es->top]=file;
    647.   	es->err_line[es->top]=line;
    648. > 	err_clear_data(es,es->top);
    649.   	}
    650.   

crypto/err/err.c:649:2: return from a call to ERR_put_error
    647.   	es->err_line[es->top]=line;
    648.   	err_clear_data(es,es->top);
    649.   	}
            ^
    650.   
    651.   void ERR_clear_error(void)

crypto/buffer/buffer.c:140:3: 
  138.   		{
  139.   		BUFerr(BUF_F_BUF_STRDUP,ERR_R_MALLOC_FAILURE);
  140. > 		return(NULL);
  141.   		}
  142.   	memcpy(ret,str,n+1);

crypto/buffer/buffer.c:144:2: return from a call to BUF_strdup
  142.   	memcpy(ret,str,n+1);
  143.   	return(ret);
  144.   	}
          ^
  145.   

apps/ca.c:3197:2: 
3195.   	tmp = BUF_strdup(str);
3196.   
3197. > 	p = strchr(tmp, ',');
3198.   
3199.   	rtime_str = tmp;

int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)
	{
	char *tmp = NULL;
	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
	int reason_code = -1;
	int i, ret = 0;
	ASN1_OBJECT *hold = NULL;
	ASN1_GENERALIZEDTIME *comp_time = NULL;
	tmp = BUF_strdup(str);
	p = strchr(tmp, ',');
	rtime_str = tmp;
	if (p)
		{
		*p = '\0';
		p++;
		reason_str = p;
		p = strchr(p, ',');
		if (p)
			{
			*p = '\0';
			arg_str = p + 1;
			}
		}
	if (prevtm)
		{
		*prevtm = ASN1_UTCTIME_new();
		if (!ASN1_UTCTIME_set_string(*prevtm, rtime_str))
			{
			BIO_printf(bio_err, "invalid revocation date %s\n", rtime_str);
			goto err;
			}
		}
	if (reason_str)
		{
		for (i = 0; i < NUM_REASONS; i++)
			{
			if(!strcasecmp(reason_str, crl_reasons[i]))
				{
				reason_code = i;
				break;
				}
			}
		if (reason_code == OCSP_REVOKED_STATUS_NOSTATUS)
			{
			BIO_printf(bio_err, "invalid reason code %s\n", reason_str);
			goto err;
			}
		if (reason_code == 7)
			reason_code = OCSP_REVOKED_STATUS_REMOVEFROMCRL;
		else if (reason_code == 8)
			{
			if (!arg_str)
				{
				BIO_printf(bio_err, "missing hold instruction\n");
				goto err;
				}
			reason_code = OCSP_REVOKED_STATUS_CERTIFICATEHOLD;
			hold = OBJ_txt2obj(arg_str, 0);
			if (!hold)
				{
				BIO_printf(bio_err, "invalid object identifier %s\n", arg_str);
				goto err;
				}
			if (phold) *phold = hold;
			}
		else if ((reason_code == 9) || (reason_code == 10))
			{
			if (!arg_str)
				{
				BIO_printf(bio_err, "missing compromised time\n");
				goto err;
				}
			comp_time = ASN1_GENERALIZEDTIME_new();
			if (!ASN1_GENERALIZEDTIME_set_string(comp_time, arg_str))
				{
				BIO_printf(bio_err, "invalid compromised time %s\n", arg_str);
				goto err;
				}
			if (reason_code == 9)
				reason_code = OCSP_REVOKED_STATUS_KEYCOMPROMISE;
			else
				reason_code = OCSP_REVOKED_STATUS_CACOMPROMISE;
			}
		}
	if (preason) *preason = reason_code;
	if (pinvtm) *pinvtm = comp_time;
	else ASN1_GENERALIZEDTIME_free(comp_time);
	ret = 1;
	err:
	if (tmp) OPENSSL_free(tmp);
	if (!phold) ASN1_OBJECT_free(hold);
	if (!pinvtm) ASN1_GENERALIZEDTIME_free(comp_time);
	return ret;
	}
['int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)\n\t{\n\tchar *tmp = NULL;\n\tchar *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;\n\tint reason_code = -1;\n\tint i, ret = 0;\n\tASN1_OBJECT *hold = NULL;\n\tASN1_GENERALIZEDTIME *comp_time = NULL;\n\ttmp = BUF_strdup(str);\n\tp = strchr(tmp, \',\');\n\trtime_str = tmp;\n\tif (p)\n\t\t{\n\t\t*p = \'\\0\';\n\t\tp++;\n\t\treason_str = p;\n\t\tp = strchr(p, \',\');\n\t\tif (p)\n\t\t\t{\n\t\t\t*p = \'\\0\';\n\t\t\targ_str = p + 1;\n\t\t\t}\n\t\t}\n\tif (prevtm)\n\t\t{\n\t\t*prevtm = ASN1_UTCTIME_new();\n\t\tif (!ASN1_UTCTIME_set_string(*prevtm, rtime_str))\n\t\t\t{\n\t\t\tBIO_printf(bio_err, "invalid revocation date %s\\n", rtime_str);\n\t\t\tgoto err;\n\t\t\t}\n\t\t}\n\tif (reason_str)\n\t\t{\n\t\tfor (i = 0; i < NUM_REASONS; i++)\n\t\t\t{\n\t\t\tif(!strcasecmp(reason_str, crl_reasons[i]))\n\t\t\t\t{\n\t\t\t\treason_code = i;\n\t\t\t\tbreak;\n\t\t\t\t}\n\t\t\t}\n\t\tif (reason_code == OCSP_REVOKED_STATUS_NOSTATUS)\n\t\t\t{\n\t\t\tBIO_printf(bio_err, "invalid reason code %s\\n", reason_str);\n\t\t\tgoto err;\n\t\t\t}\n\t\tif (reason_code == 7)\n\t\t\treason_code = OCSP_REVOKED_STATUS_REMOVEFROMCRL;\n\t\telse if (reason_code == 8)\n\t\t\t{\n\t\t\tif (!arg_str)\n\t\t\t\t{\n\t\t\t\tBIO_printf(bio_err, "missing hold instruction\\n");\n\t\t\t\tgoto err;\n\t\t\t\t}\n\t\t\treason_code = OCSP_REVOKED_STATUS_CERTIFICATEHOLD;\n\t\t\thold = OBJ_txt2obj(arg_str, 0);\n\t\t\tif (!hold)\n\t\t\t\t{\n\t\t\t\tBIO_printf(bio_err, "invalid object identifier %s\\n", arg_str);\n\t\t\t\tgoto err;\n\t\t\t\t}\n\t\t\tif (phold) *phold = hold;\n\t\t\t}\n\t\telse if ((reason_code == 9) || (reason_code == 10))\n\t\t\t{\n\t\t\tif (!arg_str)\n\t\t\t\t{\n\t\t\t\tBIO_printf(bio_err, "missing compromised time\\n");\n\t\t\t\tgoto err;\n\t\t\t\t}\n\t\t\tcomp_time = ASN1_GENERALIZEDTIME_new();\n\t\t\tif (!ASN1_GENERALIZEDTIME_set_string(comp_time, arg_str))\n\t\t\t\t{\n\t\t\t\tBIO_printf(bio_err, "invalid compromised time %s\\n", arg_str);\n\t\t\t\tgoto err;\n\t\t\t\t}\n\t\t\tif (reason_code == 9)\n\t\t\t\treason_code = OCSP_REVOKED_STATUS_KEYCOMPROMISE;\n\t\t\telse\n\t\t\t\treason_code = OCSP_REVOKED_STATUS_CACOMPROMISE;\n\t\t\t}\n\t\t}\n\tif (preason) *preason = reason_code;\n\tif (pinvtm) *pinvtm = comp_time;\n\telse ASN1_GENERALIZEDTIME_free(comp_time);\n\tret = 1;\n\terr:\n\tif (tmp) OPENSSL_free(tmp);\n\tif (!phold) ASN1_OBJECT_free(hold);\n\tif (!pinvtm) ASN1_GENERALIZEDTIME_free(comp_time);\n\treturn ret;\n\t}', 'char *BUF_strdup(const char *str)\n\t{\n\tchar *ret;\n\tint n;\n\tif (str == NULL) return(NULL);\n\tn=strlen(str);\n\tret=OPENSSL_malloc(n+1);\n\tif (ret == NULL)\n\t\t{\n\t\tBUFerr(BUF_F_BUF_STRDUP,ERR_R_MALLOC_FAILURE);\n\t\treturn(NULL);\n\t\t}\n\tmemcpy(ret,str,n+1);\n\treturn(ret);\n\t}', 'void *CRYPTO_malloc(int num, const char *file, int line)\n\t{\n\tvoid *ret = NULL;\n\tallow_customize = 0;\n\tif (malloc_debug_func != NULL)\n\t\t{\n\t\tallow_customize_debug = 0;\n\t\tmalloc_debug_func(NULL, num, file, line, 0);\n\t\t}\n\tret = malloc_ex_func(num,file,line);\n#ifdef LEVITTE_DEBUG_MEM\n\tfprintf(stderr, "LEVITTE_DEBUG_MEM:         > 0x%p (%d)\\n", ret, num);\n#endif\n\tif (malloc_debug_func != NULL)\n\t\tmalloc_debug_func(ret, num, file, line, 1);\n\treturn ret;\n\t}', 'void ERR_put_error(int lib, int func, int reason, const char *file,\n\t     int line)\n\t{\n\tERR_STATE *es;\n#ifdef _OSD_POSIX\n\tif (strncmp(file,"*POSIX(", sizeof("*POSIX(")-1) == 0) {\n\t\tchar *end;\n\t\tfile += sizeof("*POSIX(")-1;\n\t\tend = &file[strlen(file)-1];\n\t\tif (*end == \')\')\n\t\t\t*end = \'\\0\';\n\t\tif ((end = strrchr(file, \'/\')) != NULL)\n\t\t\tfile = &end[1];\n\t}\n#endif\n\tes=ERR_get_state();\n\tes->top=(es->top+1)%ERR_NUM_ERRORS;\n\tif (es->top == es->bottom)\n\t\tes->bottom=(es->bottom+1)%ERR_NUM_ERRORS;\n\tes->err_buffer[es->top]=ERR_PACK(lib,func,reason);\n\tes->err_file[es->top]=file;\n\tes->err_line[es->top]=line;\n\terr_clear_data(es,es->top);\n\t}', 'void CRYPTO_free(void *str)\n\t{\n\tif (free_debug_func != NULL)\n\t\tfree_debug_func(str, 0);\n#ifdef LEVITTE_DEBUG_MEM\n\tfprintf(stderr, "LEVITTE_DEBUG_MEM:         < 0x%p\\n", str);\n#endif\n\tfree_func(str);\n\tif (free_debug_func != NULL)\n\t\tfree_debug_func(NULL, 1);\n\t}']