<p>pespin <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/23860">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">gtp: Rework parsing logic of UpdatePdpCtxResponse<br><br>The previous order of parsing lead to non-optimal information gathering<br>when pushing events to upper layers.<br><br>This patch rearranges parsing of packet data to always gather as much<br>info as possible for the benefit of the upper layer. This way it can<br>gather information such as the cause, which is important in the case of<br>"Non-existent", since user should then drop the context.<br><br>First we want to parse the recovery state, but delay cb to upper layers<br>until we tried to gather the pdp ctx (meaning all except that pdp ctx<br>should be freed).<br>Second, we want to parse the cause, in order to know if there's an<br>associated pdp ctx we can gather from TEID.<br>Third, once we know if we should expect a meaningul TEID, parse it.<br><br>Related: SYS#5435<br>Change-Id: Idd10b494e8fbac8703c49ecd8f9bbe4246e51c57<br>---<br>M gtp/gtp.c<br>1 file changed, 30 insertions(+), 19 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ggsn refs/changes/60/23860/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/23860">change 23860</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-ggsn/+/23860"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ggsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Idd10b494e8fbac8703c49ecd8f9bbe4246e51c57 </div>
<div style="display:none"> Gerrit-Change-Number: 23860 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>