Hi Hal You missed the line that asserts on null p_next_step just before the code you changed ...
EZ 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:[EMAIL PROTECTED] > Sent: Wednesday, May 24, 2006 4:34 PM > To: [email protected] > Cc: Eitan Zahavi > Subject: [PATCH] OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_tFix > NULL ptr issue > > OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_t Fix NULL ptr > issue > > Signed-off-by: Hal Rosenstock <[EMAIL PROTECTED]> > > Index: opensm/osm_ucast_updn.c > =================================================================== > --- opensm/osm_ucast_updn.c (revision 7435) > +++ opensm/osm_ucast_updn.c (working copy) > @@ -121,10 +121,12 @@ __updn_create_updn_next_step_t(IN updn_s > 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; > - > } > > /********************************************************************** > _______________________________________________ openib-general mailing list [email protected] http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
