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


apps/ca.c:3029:1: start of procedure unpack_revinfo()
3027.   	}
3028.   
3029. > int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)
3030.   	{
3031.   	char *tmp = NULL;

apps/ca.c:3031:2: 
3029.   int unpack_revinfo(ASN1_TIME **prevtm, int *preason, ASN1_OBJECT **phold, ASN1_GENERALIZEDTIME **pinvtm, char *str)
3030.   	{
3031. > 	char *tmp = NULL;
3032.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3033.   	int reason_code = -1;

apps/ca.c:3032:2: 
3030.   	{
3031.   	char *tmp = NULL;
3032. > 	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3033.   	int reason_code = -1;
3034.   	int i, ret = 0;

apps/ca.c:3033:2: 
3031.   	char *tmp = NULL;
3032.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3033. > 	int reason_code = -1;
3034.   	int i, ret = 0;
3035.   	ASN1_OBJECT *hold = NULL;

apps/ca.c:3034:2: 
3032.   	char *rtime_str, *reason_str = NULL, *arg_str = NULL, *p;
3033.   	int reason_code = -1;
3034. > 	int i, ret = 0;
3035.   	ASN1_OBJECT *hold = NULL;
3036.   	ASN1_GENERALIZEDTIME *comp_time = NULL;

apps/ca.c:3035:2: 
3033.   	int reason_code = -1;
3034.   	int i, ret = 0;
3035. > 	ASN1_OBJECT *hold = NULL;
3036.   	ASN1_GENERALIZEDTIME *comp_time = NULL;
3037.   	tmp = BUF_strdup(str);

apps/ca.c:3036:2: 
3034.   	int i, ret = 0;
3035.   	ASN1_OBJECT *hold = NULL;
3036. > 	ASN1_GENERALIZEDTIME *comp_time = NULL;
3037.   	tmp = BUF_strdup(str);
3038.   

apps/ca.c:3037:2: 
3035.   	ASN1_OBJECT *hold = NULL;
3036.   	ASN1_GENERALIZEDTIME *comp_time = NULL;
3037. > 	tmp = BUF_strdup(str);
3038.   
3039.   	p = strchr(tmp, ',');

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

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

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

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

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:136:6: Taking true branch
  134.   	n=strlen(str);
  135.   	ret=OPENSSL_malloc(n+1);
  136.   	if (ret == NULL) 
              ^
  137.   		{
  138.   		BUFerr(BUF_F_BUF_STRDUP,ERR_R_MALLOC_FAILURE);

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

crypto/err/err.c:357:1: start of procedure ERR_put_error()
    355.   /********************************************************/
    356.   
    357. > void ERR_put_error(int lib, int func, int reason, const char *file,
    358.   	     int line)
    359.   	{

crypto/err/err.c:381:2: Skipping ERR_get_state(): empty list of specs
    379.   	}
    380.   #endif
    381.   	es=ERR_get_state();
            ^
    382.   
    383.   	es->top=(es->top+1)%ERR_NUM_ERRORS;

crypto/err/err.c:383:2: 
    381.   	es=ERR_get_state();
    382.   
    383. > 	es->top=(es->top+1)%ERR_NUM_ERRORS;
    384.   	if (es->top == es->bottom)
    385.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;

crypto/err/err.c:384:6: Taking false branch
    382.   
    383.   	es->top=(es->top+1)%ERR_NUM_ERRORS;
    384.   	if (es->top == es->bottom)
                ^
    385.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    386.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);

crypto/err/err.c:386:2: 
    384.   	if (es->top == es->bottom)
    385.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    386. > 	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    387.   	es->err_file[es->top]=file;
    388.   	es->err_line[es->top]=line;

crypto/err/err.c:387:2: 
    385.   		es->bottom=(es->bottom+1)%ERR_NUM_ERRORS;
    386.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    387. > 	es->err_file[es->top]=file;
    388.   	es->err_line[es->top]=line;
    389.   	err_clear_data(es,es->top);

crypto/err/err.c:388:2: 
    386.   	es->err_buffer[es->top]=ERR_PACK(lib,func,reason);
    387.   	es->err_file[es->top]=file;
    388. > 	es->err_line[es->top]=line;
    389.   	err_clear_data(es,es->top);
    390.   	}

crypto/err/err.c:389:2: Taking true branch
    387.   	es->err_file[es->top]=file;
    388.   	es->err_line[es->top]=line;
    389.   	err_clear_data(es,es->top);
            ^
    390.   	}
    391.   

crypto/err/err.c:389:2: Taking true branch
    387.   	es->err_file[es->top]=file;
    388.   	es->err_line[es->top]=line;
    389.   	err_clear_data(es,es->top);
            ^
    390.   	}
    391.   

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:389:2: 
    387.   	es->err_file[es->top]=file;
    388.   	es->err_line[es->top]=line;
    389. > 	err_clear_data(es,es->top);
    390.   	}
    391.   

crypto/err/err.c:390:2: return from a call to ERR_put_error
    388.   	es->err_line[es->top]=line;
    389.   	err_clear_data(es,es->top);
    390.   	}
            ^
    391.   
    392.   void ERR_clear_error(void)

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

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

apps/ca.c:3039:2: 
3037.   	tmp = BUF_strdup(str);
3038.   
3039. > 	p = strchr(tmp, ',');
3040.   
3041.   	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);
	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\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}']