<br><br>
<div class="gmail_quote">On Tue, Sep 22, 2009 at 10:53 PM, Sasha Khapyorsky <span dir="ltr"><<a href="mailto:sashak@voltaire.com" target="_blank">sashak@voltaire.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">
<div>On 16:44 Tue 22 Sep     , Hal Rosenstock wrote:<br>><br>> Yeah, the port lid table will be OK but port's PortInfo won't (so base<br>> LID/LMC will be broken) for this scenario but it wouldn't affect this code<br>
> in this way. </div></blockquote>
<div> </div>
<div> Let me try this again... The port LID table is fine but the lookup is done based on the LID in the received portInfo as it is the result of osm_physp_get_base_lid() (osm_link_mgr.c:link_mgr_get_smsl line 83). In the case of failed Sets, this is invalid so LID 0 is used and that's what causes the NULL p_src_port which in turn causes the seg fault.</div>

<div> </div>
<div>So I'm back to:</div>
<div> </div>
<div>I can see two ways to fix this:<br>1. Replace with port GUID search<br>2. Have osm_get_lash_sl handle NULL for p_src_port<br>Maybe you see other ways to deal with this.</div>
<div> </div>
<div>Do you have a preferred approach ?</div>
<div> </div>
<div>-- Hal</div>
<div> </div></div>