<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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'?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure: the only case where they differ is root node allocation inside while(1){...} in main() - and it's deallocated at the end of each iteration.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">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></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I see, that make sense. I've checked all the places where it's called only with dynamically allocated parent so I think it's ok to drop it.</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-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 28 Jan 2019 10:29:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>