<p style="white-space: pre-wrap; word-wrap: break-word;">don't you think the bug is rather that the talloc inside value_node_add is using 'parent' instead of 'ctx'?  I don't recall the details.  My suspicion is that originally not all nodes were dynamically allocated and hence the ctx would need to be specified explicitly.  If that's no longer the case, than it appears we can go for removing the argument like in your patch.</p><p><a href="https://gerrit.osmocom.org/12705">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12705">change 12705</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/12705"/><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: I17c2fed4049e1c83307feda001aefc006dfbe6a3 </div>
<div style="display:none"> Gerrit-Change-Number: 12705 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 27 Jan 2019 13:45:04 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>