[openib-general] RE: [PATCHv2] OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_t:Fix NULL ptr issue

[Eitan Zahavi]

Thanks

Eitan Zahavi
Senior Engineering Director, Software Architect
Mellanox Technologies LTD
Tel:+972-4-9097208
Fax:+972-4-9593245
P.O. Box 586 Yokneam 20692 ISRAEL


> -----Original Message-----
> From: Hal Rosenstock [mailto:halr at voltaire.com]
> Sent: Thursday, May 25, 2006 1:26 PM
> To: openib-general at openib.org
> Cc: Eitan Zahavi
> Subject: [PATCHv2]
> OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_t:Fix NULL ptr
issue
> 
> OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_t: Fix NULL ptr
> issue in non debug builds
> 
> Signed-off-by: Hal Rosenstock <halr at voltaire.com>
> 
> Index: opensm/osm_ucast_updn.c
> ===================================================================
> --- opensm/osm_ucast_updn.c	(revision 7435)
> +++ opensm/osm_ucast_updn.c	(working copy)
> @@ -119,12 +119,12 @@ __updn_create_updn_next_step_t(IN updn_s
>    updn_next_step_t *p_next_step;
> 
>    p_next_step = (updn_next_step_t*) cl_zalloc(sizeof(*p_next_step));
> -  CL_ASSERT (p_next_step != NULL);
> -
> -  p_next_step->state = state;
> -  p_next_step->p_sw = p_sw;
> +  if (p_next_step)
> +  {
> +    p_next_step->state = state;
> +    p_next_step->p_sw = p_sw;
> +  }
>    return p_next_step;
> -
>  }
> 
>
/**********************************************************************
>