<p><a href="https://gerrit.osmocom.org/c/osmo-smlc/+/20470">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-smlc/+/20470/6/src/osmo-smlc/lb_peer.c">File src/osmo-smlc/lb_peer.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-smlc/+/20470/6/src/osmo-smlc/lb_peer.c@463">Patch Set #6, Line 463:</a> <code style="font-family:monospace,monospace">                                    " Dropping message.\n");</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't recall all the details of the protocol spec. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">by spec, a connection request can be answered with either a conn confirm or a conn refused.<br>however this here is a weird special case: a conn is already established for a given conn id, but we are receiving another Connection Request for the same conn id. what would that mean?</p><ul><li>if it is the same peer sending it, that's a bug / protocol error.</li><li>assume the peer has restarted and tries to re-use a previous conn id, it would make sense to discard the previous conn. But then again, there should have been a RESET from that peer that has done just that.</li><li>If it is a different peer using the same conn id ... can that happen? in that case a Conn Refused might be in order.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">so, if it is the same peer, it's a weird corner case that shouldn't happen in practice, not really sure how to deal with it. i think dropping the message like in this patch is the easiest / least invasive way until we see a real situation that suggests a useful way to respond ... ?</p><p style="white-space: pre-wrap; word-wrap: break-word;">but let's clarify: is a different peer colliding with another peer's conn id possible? if yes, would it even propagate to this layer?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-smlc/+/20470">change 20470</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-smlc/+/20470"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-smlc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I917ba8fc51a1f1150be77ae01e12a7b16a853052 </div>
<div style="display:none"> Gerrit-Change-Number: 20470 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 16 Oct 2020 13:36:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>