[ofa-general] Re: [PATCH] opensm/osm_ucast_lash.c: Eliminate possible seg fault in get_osm_switch_from_port

[Sasha Khapyorsky]

On 07:03 Tue 14 Jul     , Hal Rosenstock wrote:
> >
> > It is what I asked - why is NULL pointer used there (in SA PathRecord
> > flow) and is is legal usage?
> 
> It shouldn't be a valid use case.

So why do we need this patch then?

> > As far as I can see in osm_sa_path_record.c it should be impossible, but
> > maybe I'm missing something.
> 
> Yes, that's what I thought too but apparently it doesn't appear to be
> the case and I'm not sure how that could occur. Any ideas ?

If you are able to reproduce such case try to find why this happens
(crash OpenSM into gdb) - to hide bugs is not a good idea IMO.

Sasha