<p style="white-space: pre-wrap; word-wrap: break-word;">To me this looks good. I have left two comments, probably everything is fine the way it is. I just wanted to make you aware of this.</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25145">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25145/2/src/gb/gprs_ns2_sns.c">File src/gb/gprs_ns2_sns.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/libosmocore/+/25145/2/src/gb/gprs_ns2_sns.c@2074">Patch Set #2, Line 2074:</a> <code style="font-family:monospace,monospace">                gprs_ns2_free_nsvc(nsvc);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I have seen that the freeing is done differently in this patch: https://gerrit.osmocom.org/c/libosmocore/+/25147/2/src/gb/gprs_ns2_sns.c - Maybe doing this here in a similar way might be a good idea. However I am currently not aware which patch comes when in this patch. Maybe this is already changed. - I just wanted to make you aware of this. (There would also be a an llist_last to detect the end of the loop. http://git.osmocom.org/libosmocore/tree/include/osmocom/core/linuxlist.h)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/25145/2/src/gb/gprs_ns2_sns.c@2314">Patch Set #2, Line 2314:</a> <code style="font-family:monospace,monospace">                        /* keep the NSVC we need for SNS, but unconfigure it */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Ther is this if (nsvc == gss->sns_nsvc) that guards this in the original version. I don't know if it can happen that gss->sns_nsvc is NULL or so.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25145">change 25145</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/libosmocore/+/25145"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2639345fdf3cd300a934238d676c543065ceaa8b </div>
<div style="display:none"> Gerrit-Change-Number: 25145 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 12 Aug 2021 09:28:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>