[openib-general] RE: [PATCH] OpenSM/osm_ucast_updn.c::__updn_create_updn_next_step_tFix NULL ptr issue

Hal Rosenstock halr at voltaire.com
Wed May 24 08:31:33 PDT 2006


On Wed, 2006-05-24 at 11:33, Eitan Zahavi wrote:
> Hi Hal
> 
> You missed the line that asserts on null p_next_step just before the 
> code you changed .

But isn't CL_ASSERT a debug compile time thing so it's needed when it's
built without debug ?

-- Hal

> .
> 
> 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:halr at voltaire.com]
> > Sent: Wednesday, May 24, 2006 4:34 PM
> > To: openib-general at openib.org
> > 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 <halr at voltaire.com>
> > 
> > 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;
> > -
> >  }
> > 
> >
> /**********************************************************************
> > 
> 




More information about the general mailing list