<p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/22314">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/22314/2/src/gbproxy/gb_proxy.c">File src/gbproxy/gb_proxy.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-sgsn/+/22314/2/src/gbproxy/gb_proxy.c@953">Patch Set #2, Line 953:</a> <code style="font-family:monospace,monospace">insure</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">ensure</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/22314/2/src/gbproxy/gb_proxy.c@969">Patch Set #2, Line 969:</a> <code style="font-family:monospace,monospace">            }</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Yeah, but we might talk to a (3rd party) SGSN that supports other protocols. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Definitely, gbproxy should simply forward anything != GERAN to the SGSN and let the SGSN deal with it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/22314/2/src/gbproxy/gb_proxy.c@1275">Patch Set #2, Line 1275:</a> <code style="font-family:monospace,monospace">insure</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"ensure".  And why would we want to check the RIM Source address here?  If it's from the SGSN, we don't care which is the source address (not even whether it's GERAN).  We just care about the destination, which must be within the cells covered by this gbproxy.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/22314">change 22314</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-sgsn/+/22314"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I3403736eed9d6819634c26db0b90393ac6f4416a </div>
<div style="display:none"> Gerrit-Change-Number: 22314 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 29 Jan 2021 21:13:59 +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: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>