<div dir="ltr">Got to it sooner than I thought I would.  It works fine.  Thanks, this looks a lot cleaner!  BTW, on a somewhat unrelated note, I just sent an update to bug <a href="http://wiki.strongswan.org/issues/289">http://wiki.strongswan.org/issues/289</a> with another fix I made.  Is it better to send these to the mailing list rather than comment on bugs?  I can open a new thread in that case.<div>
<br></div><div>--</div><div style>Paul</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Mar 9, 2013 at 5:45 PM, Paul Stewart <span dir="ltr"><<a href="mailto:pstew@chromium.org" target="_blank">pstew@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I'll give it a shot at latest Monday.  Thanks!<div><br></div><div>--<br>Paul</div></div><div class="HOEnZb">
<div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 7, 2013 at 1:02 AM, Martin Willi <span dir="ltr"><<a href="mailto:martin@strongswan.org" target="_blank">martin@strongswan.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Paul,<br>
<br>
> [...] which I tracked down to the tsi of the Cisco peer not returning a<br>
<div>> port number in its reply.<br>
<br>
</div>I see.<br>
<div><br>
> Using the patch below, I was able to accommodate this omission.  Does<br>
> this seem like a reasonable change, perhaps behind a configuration<br>
> flag?<br>
<br>
</div>Thanks for the patch, looks reasonable. I don't think a configuration<br>
option is necessary, as long as we install the more restrictive<br>
selector.<br>
<br>
Instead of just checking the port, I think we can handle this in a more<br>
generic way by selecting the subset of the proposed and the returned<br>
selector. This should work in any case, in is actually even simpler.<br>
Please try the attached patch, if that works, I can push it to master.<br>
<br>
Best regards<br>
<span><font color="#888888">Martin<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>