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

Hal Rosenstock hal.rosenstock at gmail.com
Tue Jul 14 10:05:32 PDT 2009


On Tue, Jul 14, 2009 at 12:16 PM, Sasha Khapyorsky<sashak at voltaire.com> wrote:
> On 11:52 Tue 14 Jul     , Hal Rosenstock wrote:
>>
>> How about a CL_ASSERT there ?
>
> Not much better than segfault :).

Not much better but still slightly better. If not, then what about all
the other CL_ASSERTs in the code ?

> I think that we don't need to patch
> the main stream code until we understand the problem.

OK.

> Sasha
>
>> > If you are able to reproduce such case try to find why this happens
>> > (crash OpenSM into gdb)
>>
>> I put core into gdb but it doesn't show how p_port became NULL
>
> try 'backtrace'.

Of course I did that.

-- Hal

> Sasha
>



More information about the general mailing list