engines/e_sureware.c:989: error: NULL_DEREFERENCE
  pointer `psign` last assigned on line 958 could be null and is dereferenced at line 989, column 2.
Showing all 50 steps of the trace


engines/e_sureware.c:954:1: start of procedure surewarehk_dsa_do_sign()
952.   #ifndef OPENSSL_NO_DSA
953.   /* DSA sign and verify */
954. > static	DSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)
955.   {
956.   	int ret=0;

engines/e_sureware.c:956:2: 
954.   static	DSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)
955.   {
956. > 	int ret=0;
957.   	char *hptr=NULL;
958.   	DSA_SIG *psign=NULL;

engines/e_sureware.c:957:2: 
955.   {
956.   	int ret=0;
957. > 	char *hptr=NULL;
958.   	DSA_SIG *psign=NULL;
959.   	char msg[64]="ENGINE_dsa_do_sign";

engines/e_sureware.c:958:2: 
956.   	int ret=0;
957.   	char *hptr=NULL;
958. > 	DSA_SIG *psign=NULL;
959.   	char msg[64]="ENGINE_dsa_do_sign";
960.   	if (!p_surewarehk_Dsa_Sign)

engines/e_sureware.c:959:2: 
957.   	char *hptr=NULL;
958.   	DSA_SIG *psign=NULL;
959. > 	char msg[64]="ENGINE_dsa_do_sign";
960.   	if (!p_surewarehk_Dsa_Sign)
961.   	{

engines/e_sureware.c:960:7: Taking false branch
958.   	DSA_SIG *psign=NULL;
959.   	char msg[64]="ENGINE_dsa_do_sign";
960.   	if (!p_surewarehk_Dsa_Sign)
             ^
961.   	{
962.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ENGINE_R_NOT_INITIALISED);

engines/e_sureware.c:965:13: 
963.   	}
964.   	/* extract ref to private key */
965. > 	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
966.   	{
967.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);

crypto/dsa/dsa_lib.c:257:1: start of procedure DSA_get_ex_data()
  255.   	}
  256.   
  257. > void *DSA_get_ex_data(DSA *d, int idx)
  258.   	{
  259.   	return(CRYPTO_get_ex_data(&d->ex_data,idx));

crypto/dsa/dsa_lib.c:259:2: 
  257.   void *DSA_get_ex_data(DSA *d, int idx)
  258.   	{
  259. > 	return(CRYPTO_get_ex_data(&d->ex_data,idx));
  260.   	}
  261.   

crypto/ex_data.c:626:1: start of procedure CRYPTO_get_ex_data()
    624.   /* For a given CRYPTO_EX_DATA_ variable, get the value corresponding to a
    625.    * particular index in the class used by this variable */
    626. > void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)
    627.   	{
    628.   	if (ad->sk == NULL)

crypto/ex_data.c:628:6: Taking false branch
    626.   void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)
    627.   	{
    628.   	if (ad->sk == NULL)
                ^
    629.   		return(0);
    630.   	else if (idx >= sk_num(ad->sk))

crypto/ex_data.c:630:11: 
    628.   	if (ad->sk == NULL)
    629.   		return(0);
    630. > 	else if (idx >= sk_num(ad->sk))
    631.   		return(0);
    632.   	else

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/ex_data.c:630:11: Taking false branch
    628.   	if (ad->sk == NULL)
    629.   		return(0);
    630.   	else if (idx >= sk_num(ad->sk))
                     ^
    631.   		return(0);
    632.   	else

crypto/ex_data.c:633:3: 
    631.   		return(0);
    632.   	else
    633. > 		return(sk_value(ad->sk,idx));
    634.   	}
    635.   

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 == NULL) return NULL;

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

crypto/stack/stack.c:310:2: 
      308.   {
      309.   	if(st == NULL) 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 == NULL) return NULL;
      310.   	return st->data[i];
      311. > }
      312.   
      313.   char *sk_set(STACK *st, int i, char *value)

crypto/ex_data.c:634:2: return from a call to CRYPTO_get_ex_data
    632.   	else
    633.   		return(sk_value(ad->sk,idx));
    634.   	}
            ^
    635.   
    636.   IMPLEMENT_STACK_OF(CRYPTO_EX_DATA_FUNCS)

crypto/dsa/dsa_lib.c:260:2: return from a call to DSA_get_ex_data
  258.   	{
  259.   	return(CRYPTO_get_ex_data(&d->ex_data,idx));
  260.   	}
          ^
  261.   
  262.   #ifndef OPENSSL_NO_DH

engines/e_sureware.c:965:13: Taking true branch
963.   	}
964.   	/* extract ref to private key */
965.   	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
                   ^
966.   	{
967.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);

engines/e_sureware.c:967:3: 
965.   	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
966.   	{
967. > 		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);
968.   	}
969.   	else

engines/e_sureware_err.c:145:1: start of procedure ERR_SUREWARE_error()
  143.   	}
  144.   
  145. > static void ERR_SUREWARE_error(int function, int reason, char *file, int line)
  146.   	{
  147.   	if (SUREWARE_lib_error_code == 0)

engines/e_sureware_err.c:147:6: Taking true branch
  145.   static void ERR_SUREWARE_error(int function, int reason, char *file, int line)
  146.   	{
  147.   	if (SUREWARE_lib_error_code == 0)
              ^
  148.   		SUREWARE_lib_error_code=ERR_get_next_error_library();
  149.   	ERR_PUT_error(SUREWARE_lib_error_code,function,reason,file,line);

engines/e_sureware_err.c:148:3: 
  146.   	{
  147.   	if (SUREWARE_lib_error_code == 0)
  148. > 		SUREWARE_lib_error_code=ERR_get_next_error_library();
  149.   	ERR_PUT_error(SUREWARE_lib_error_code,function,reason,file,line);
  150.   	}

crypto/err/err.c:973:1: start of procedure ERR_get_next_error_library()
    971.   	}
    972.   
    973. > int ERR_get_next_error_library(void)
    974.   	{
    975.   	err_fns_check();

crypto/err/err.c:975:2: 
    973.   int ERR_get_next_error_library(void)
    974.   	{
    975. > 	err_fns_check();
    976.   	return ERRFN(get_next_lib)();
    977.   	}

crypto/err/err.c:278:1: start of procedure err_fns_check()
      276.   /* Internal function that checks whether "err_fns" is set and if not, sets it to
      277.    * the defaults. */
      278. > static void err_fns_check(void)
      279.   	{
      280.   	if (err_fns) return;

crypto/err/err.c:280:6: Taking true branch
      278.   static void err_fns_check(void)
      279.   	{
      280.   	if (err_fns) return;
                  ^
      281.   	
      282.   	CRYPTO_w_lock(CRYPTO_LOCK_ERR);

crypto/err/err.c:280:15: 
      278.   static void err_fns_check(void)
      279.   	{
      280. > 	if (err_fns) return;
      281.   	
      282.   	CRYPTO_w_lock(CRYPTO_LOCK_ERR);

crypto/err/err.c:286:2: return from a call to err_fns_check
      284.   		err_fns = &err_defaults;
      285.   	CRYPTO_w_unlock(CRYPTO_LOCK_ERR);
      286.   	}
              ^
      287.   
      288.   /* API functions to get or set the underlying ERR functions. */

crypto/err/err.c:976:2: Skipping __function_pointer__(): unresolved function pointer
    974.   	{
    975.   	err_fns_check();
    976.   	return ERRFN(get_next_lib)();
            ^
    977.   	}
    978.   

crypto/err/err.c:977:2: return from a call to ERR_get_next_error_library
    975.   	err_fns_check();
    976.   	return ERRFN(get_next_lib)();
    977.   	}
            ^
    978.   
    979.   void ERR_set_error_data(char *data, int flags)

engines/e_sureware_err.c:149:2: 
  147.   	if (SUREWARE_lib_error_code == 0)
  148.   		SUREWARE_lib_error_code=ERR_get_next_error_library();
  149. > 	ERR_PUT_error(SUREWARE_lib_error_code,function,reason,file,line);
  150.   	}

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

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

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

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

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

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

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

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

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

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

engines/e_sureware_err.c:150:2: return from a call to ERR_SUREWARE_error
  148.   		SUREWARE_lib_error_code=ERR_get_next_error_library();
  149.   	ERR_PUT_error(SUREWARE_lib_error_code,function,reason,file,line);
  150.   	}
          ^

engines/e_sureware.c:989:2: 
987.   		surewarehk_error_handling(msg,SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ret);
988.   	}
989. > 	psign->r->top=20/sizeof(BN_ULONG);
990.   	bn_fix_top(psign->r);
991.   	psign->s->top=20/sizeof(BN_ULONG);

static	DSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)
{
	int ret=0;
	char *hptr=NULL;
	DSA_SIG *psign=NULL;
	char msg[64]="ENGINE_dsa_do_sign";
	if (!p_surewarehk_Dsa_Sign)
	{
		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ENGINE_R_NOT_INITIALISED);
	}
	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
	{
		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);
	}
	else
	{
		if((psign = DSA_SIG_new()) == NULL)
		{
			SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ERR_R_MALLOC_FAILURE);
			goto err;
		}
		psign->r=BN_new();
		psign->s=BN_new();
		bn_expand2(psign->r, 20/sizeof(BN_ULONG));
		bn_expand2(psign->s, 20/sizeof(BN_ULONG));
		if (!psign->r || psign->r->dmax!=20/sizeof(BN_ULONG) ||
			!psign->s || psign->s->dmax!=20/sizeof(BN_ULONG))
			goto err;
		ret=p_surewarehk_Dsa_Sign(msg,flen,from,
					  (unsigned long *)psign->r->d,
					  (unsigned long *)psign->s->d,
					  hptr);
		surewarehk_error_handling(msg,SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ret);
	}
	psign->r->top=20/sizeof(BN_ULONG);
	bn_fix_top(psign->r);
	psign->s->top=20/sizeof(BN_ULONG);
	bn_fix_top(psign->s);
err:
	if (psign)
	{
		DSA_SIG_free(psign);
		psign=NULL;
	}
	return psign;
}
['static\tDSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)\n{\n\tint ret=0;\n\tchar *hptr=NULL;\n\tDSA_SIG *psign=NULL;\n\tchar msg[64]="ENGINE_dsa_do_sign";\n\tif (!p_surewarehk_Dsa_Sign)\n\t{\n\t\tSUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ENGINE_R_NOT_INITIALISED);\n\t}\n\telse if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))\n\t{\n\t\tSUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);\n\t}\n\telse\n\t{\n\t\tif((psign = DSA_SIG_new()) == NULL)\n\t\t{\n\t\t\tSUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ERR_R_MALLOC_FAILURE);\n\t\t\tgoto err;\n\t\t}\n\t\tpsign->r=BN_new();\n\t\tpsign->s=BN_new();\n\t\tbn_expand2(psign->r, 20/sizeof(BN_ULONG));\n\t\tbn_expand2(psign->s, 20/sizeof(BN_ULONG));\n\t\tif (!psign->r || psign->r->dmax!=20/sizeof(BN_ULONG) ||\n\t\t\t!psign->s || psign->s->dmax!=20/sizeof(BN_ULONG))\n\t\t\tgoto err;\n\t\tret=p_surewarehk_Dsa_Sign(msg,flen,from,\n\t\t\t\t\t  (unsigned long *)psign->r->d,\n\t\t\t\t\t  (unsigned long *)psign->s->d,\n\t\t\t\t\t  hptr);\n\t\tsurewarehk_error_handling(msg,SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ret);\n\t}\n\tpsign->r->top=20/sizeof(BN_ULONG);\n\tbn_fix_top(psign->r);\n\tpsign->s->top=20/sizeof(BN_ULONG);\n\tbn_fix_top(psign->s);\nerr:\n\tif (psign)\n\t{\n\t\tDSA_SIG_free(psign);\n\t\tpsign=NULL;\n\t}\n\treturn psign;\n}', 'void *DSA_get_ex_data(DSA *d, int idx)\n\t{\n\treturn(CRYPTO_get_ex_data(&d->ex_data,idx));\n\t}', 'void *CRYPTO_get_ex_data(const CRYPTO_EX_DATA *ad, int idx)\n\t{\n\tif (ad->sk == NULL)\n\t\treturn(0);\n\telse if (idx >= sk_num(ad->sk))\n\t\treturn(0);\n\telse\n\t\treturn(sk_value(ad->sk,idx));\n\t}', 'int sk_num(const STACK *st)\n{\n\tif(st == NULL) return -1;\n\treturn st->num;\n}', 'char *sk_value(const STACK *st, int i)\n{\n\tif(st == NULL) return NULL;\n\treturn st->data[i];\n}', 'static void ERR_SUREWARE_error(int function, int reason, char *file, int line)\n\t{\n\tif (SUREWARE_lib_error_code == 0)\n\t\tSUREWARE_lib_error_code=ERR_get_next_error_library();\n\tERR_PUT_error(SUREWARE_lib_error_code,function,reason,file,line);\n\t}', 'int ERR_get_next_error_library(void)\n\t{\n\terr_fns_check();\n\treturn ERRFN(get_next_lib)();\n\t}', 'static void err_fns_check(void)\n\t{\n\tif (err_fns) return;\n\tCRYPTO_w_lock(CRYPTO_LOCK_ERR);\n\tif (!err_fns)\n\t\terr_fns = &err_defaults;\n\tCRYPTO_w_unlock(CRYPTO_LOCK_ERR);\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}']