<p style="white-space: pre-wrap; word-wrap: break-word;">I can change the macro the way you propose but it seems rather pointless to me: we're trying to prevent theoretical memleak in an unwritten code which have to be written in particular (and wrong) way for it to happen at all.</p><p><a href="https://gerrit.osmocom.org/12761">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12761/6/src/simple_ctrl.c">File src/simple_ctrl.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/12761/6/src/simple_ctrl.c@42">Patch Set #6, Line 42:</a> <code style="font-family:monospace,monospace">   fprintf(stderr, "CTRL %s error: " fmt, make_authority(sch, &sch->cfg), ##args)</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">memleak of make_authority. At least until you free sch. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The ERR part in macro name refers to "error" - this macro will be called only when we have some errors. Keeping sch despite of the permanent error associated with it is plain wrong. No current user of this code does that.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12761">change 12761</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/12761"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sysmon </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie321655a92cdbefbfaa056ac0d583397c83beccb </div>
<div style="display:none"> Gerrit-Change-Number: 12761 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 11 Feb 2019 14:53:05 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>