<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/16189">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/libosmocore/+/16189/3/src/gsm/gsup.c">File src/gsm/gsup.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/+/16189/3/src/gsm/gsup.c@910">Patch Set #3, Line 910:</a> <code style="font-family:monospace,monospace"> * Note: after calling this function, fields in the reply may reference the same memory as rx and are not deep-copied.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I see no deep-copied as a source of problems. Usually you want to queue this response message while freeing the rx message potentially beforehand (because theoretically the socket kernel queue could be full and we have our own wqueue).<br>Am I missing something in my picture?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/16189/3/src/gsm/gsup.c@943">Patch Set #3, Line 943:</a> <code style="font-family:monospace,monospace">        if (!reply->imsi[0])</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'll keep asking to please match against = '\0', it's then clear from reader point of view that this is a string and not some random pointer array.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/16189/3/src/gsm/gsup.c@1040">Patch Set #3, Line 1040:</a> <code style="font-family:monospace,monospace">    OSMO_NAME_C_IMPL(ctx, 64, "ERROR", osmo_gsup_message_name_buf, msg)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">64 looks a bit small here given the amount of text above. 128?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/16189">change 16189</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/+/16189"/><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: Id9692880079ea0f219f52d81b1923a76fc640566 </div>
<div style="display:none"> Gerrit-Change-Number: 16189 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </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-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 27 Nov 2019 07:57:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>