<p><a href="https://gerrit.osmocom.org/c/osmo-msc/+/26411">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/osmo-msc/+/26411/1/src/libmsc/gsm_04_11.c">File src/libmsc/gsm_04_11.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-msc/+/26411/1/src/libmsc/gsm_04_11.c@633">Patch Set #1, Line 633:</a> <code style="font-family:monospace,monospace"> if (gsms->smpp.esme) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Are you sure it is leaked if it got routed? It could be that the sms is added to some wqueue or simi […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Nope, I spent quite some time checking the code branches. The given 'gsms' is merely used to compose an SMPP message, but never gets free()d or queued. And actually we did free() it before Ic34d398e0a850856e20380ae35e5c2ae5e3c539b was merged.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-msc/+/26411/1/src/libmsc/gsm_04_11.c@644">Patch Set #1, Line 644:</a> <code style="font-family:monospace,monospace">      if (rc == GSM411_RP_CAUSE_MO_NET_OUT_OF_ORDER)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Will you upload another patchset?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">https://gerrit.osmocom.org/c/osmo-msc/+/26413</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-msc/+/26411">change 26411</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-msc/+/26411"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-msc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I8a745d747f06baa7109418ffe600b27b3c0a5228 </div>
<div style="display:none"> Gerrit-Change-Number: 26411 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Pablo Neira Ayuso <pablo@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 29 Nov 2021 12:46:46 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>