[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