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


engines/e_sureware.c:962:1: start of procedure surewarehk_dsa_do_sign()
960.   #ifndef OPENSSL_NO_DSA
961.   /* DSA sign and verify */
962. > static	DSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)
963.   {
964.   	int ret=0;

engines/e_sureware.c:964:2: 
962.   static	DSA_SIG * surewarehk_dsa_do_sign(const unsigned char *from, int flen, DSA *dsa)
963.   {
964. > 	int ret=0;
965.   	char *hptr=NULL;
966.   	DSA_SIG *psign=NULL;

engines/e_sureware.c:965:2: 
963.   {
964.   	int ret=0;
965. > 	char *hptr=NULL;
966.   	DSA_SIG *psign=NULL;
967.   	char msg[64]="ENGINE_dsa_do_sign";

engines/e_sureware.c:966:2: 
964.   	int ret=0;
965.   	char *hptr=NULL;
966. > 	DSA_SIG *psign=NULL;
967.   	char msg[64]="ENGINE_dsa_do_sign";
968.   	if (!p_surewarehk_Dsa_Sign)

engines/e_sureware.c:967:2: 
965.   	char *hptr=NULL;
966.   	DSA_SIG *psign=NULL;
967. > 	char msg[64]="ENGINE_dsa_do_sign";
968.   	if (!p_surewarehk_Dsa_Sign)
969.   	{

engines/e_sureware.c:968:7: Taking false branch
966.   	DSA_SIG *psign=NULL;
967.   	char msg[64]="ENGINE_dsa_do_sign";
968.   	if (!p_surewarehk_Dsa_Sign)
             ^
969.   	{
970.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ENGINE_R_NOT_INITIALISED);

engines/e_sureware.c:973:13: 
971.   	}
972.   	/* extract ref to private key */
973. > 	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
974.   	{
975.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);

crypto/dsa/dsa_lib.c:268:1: start of procedure DSA_get_ex_data()
  266.   	}
  267.   
  268. > void *DSA_get_ex_data(DSA *d, int idx)
  269.   	{
  270.   	return(CRYPTO_get_ex_data(&d->ex_data,idx));

crypto/dsa/dsa_lib.c:270:2: 
  268.   void *DSA_get_ex_data(DSA *d, int idx)
  269.   	{
  270. > 	return(CRYPTO_get_ex_data(&d->ex_data,idx));
  271.   	}
  272.   

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

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

crypto/ex_data.c:626:11: 
    624.   	if (ad->sk == NULL)
    625.   		return(0);
    626. > 	else if (idx >= sk_num(ad->sk))
    627.   		return(0);
    628.   	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:626:11: Taking false branch
    624.   	if (ad->sk == NULL)
    625.   		return(0);
    626.   	else if (idx >= sk_num(ad->sk))
                     ^
    627.   		return(0);
    628.   	else

crypto/ex_data.c:629:3: 
    627.   		return(0);
    628.   	else
    629. > 		return(sk_value(ad->sk,idx));
    630.   	}
    631.   

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/ex_data.c:630:2: return from a call to CRYPTO_get_ex_data
    628.   	else
    629.   		return(sk_value(ad->sk,idx));
    630.   	}
            ^
    631.   
    632.   IMPLEMENT_STACK_OF(CRYPTO_EX_DATA_FUNCS)

crypto/dsa/dsa_lib.c:271:2: return from a call to DSA_get_ex_data
  269.   	{
  270.   	return(CRYPTO_get_ex_data(&d->ex_data,idx));
  271.   	}
          ^
  272.   
  273.   #ifndef OPENSSL_NO_DH

engines/e_sureware.c:973:13: Taking true branch
971.   	}
972.   	/* extract ref to private key */
973.   	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
                   ^
974.   	{
975.   		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);

engines/e_sureware.c:975:3: 
973.   	else if (!(hptr=DSA_get_ex_data(dsa, dsaHndidx)))
974.   	{
975. > 		SUREWAREerr(SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,SUREWARE_R_MISSING_KEY_COMPONENTS);
976.   	}
977.   	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:1024:1: start of procedure ERR_get_next_error_library()
    1022.   	}
    1023.   
    1024. > int ERR_get_next_error_library(void)
    1025.   	{
    1026.   	err_fns_check();

crypto/err/err.c:1026:2: 
    1024.   int ERR_get_next_error_library(void)
    1025.   	{
    1026. > 	err_fns_check();
    1027.   	return ERRFN(get_next_lib)();
    1028.   	}

crypto/err/err.c:283:1: start of procedure err_fns_check()
      281.   /* Internal function that checks whether "err_fns" is set and if not, sets it to
      282.    * the defaults. */
      283. > static void err_fns_check(void)
      284.   	{
      285.   	if (err_fns) return;

crypto/err/err.c:285:6: Taking true branch
      283.   static void err_fns_check(void)
      284.   	{
      285.   	if (err_fns) return;
                  ^
      286.   	
      287.   	CRYPTO_w_lock(CRYPTO_LOCK_ERR);

crypto/err/err.c:285:15: 
      283.   static void err_fns_check(void)
      284.   	{
      285. > 	if (err_fns) return;
      286.   	
      287.   	CRYPTO_w_lock(CRYPTO_LOCK_ERR);

crypto/err/err.c:291:2: return from a call to err_fns_check
      289.   		err_fns = &err_defaults;
      290.   	CRYPTO_w_unlock(CRYPTO_LOCK_ERR);
      291.   	}
              ^
      292.   
      293.   /* API functions to get or set the underlying ERR functions. */

crypto/err/err.c:1027:2: Skipping __function_pointer__(): unresolved function pointer
    1025.   	{
    1026.   	err_fns_check();
    1027.   	return ERRFN(get_next_lib)();
             ^
    1028.   	}
    1029.   

crypto/err/err.c:1028:2: return from a call to ERR_get_next_error_library
    1026.   	err_fns_check();
    1027.   	return ERRFN(get_next_lib)();
    1028.   	}
             ^
    1029.   
    1030.   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:664:1: start of procedure ERR_put_error()
    662.   /********************************************************/
    663.   
    664. > void ERR_put_error(int lib, int func, int reason, const char *file,
    665.   	     int line)
    666.   	{

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

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

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

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

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

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

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

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

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

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

crypto/err/err.c:698:2: return from a call to ERR_put_error
    696.   	es->err_line[es->top]=line;
    697.   	err_clear_data(es,es->top);
    698.   	}
            ^
    699.   
    700.   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:997:2: 
995.   		surewarehk_error_handling(msg,SUREWARE_F_SUREWAREHK_DSA_DO_SIGN,ret);
996.   	}
997. > 	psign->r->top=20/sizeof(BN_ULONG);
998.   	bn_fix_top(psign->r);
999.   	psign->s->top=20/sizeof(BN_ULONG);
