<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7022">View Change</a></p><p>Patch set 1:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7022/1/main/stasis_bridges.c">File main/stasis_bridges.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7022/1/main/stasis_bridges.c@883">Patch Set #1, Line 883:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">   if (!out) {<br>           ast_json_unref(json_target);<br>          ast_json_unref(json_transferee);<br>              return NULL;<br>  }<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">If an error occurs while packing it does not appear to free any objects tha</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">If the pack fails it does clean up using the internal unref function for all the refs it has already stolen.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, there is absolutely no way to know which refs it has already stolen.  The pack may or may not have already stolen the ref.  We don't know where in the packing that the error happened.  We cannot use the 'O' option which gets its own ref rather than stealing it either as the json library ref/unref internal functions are not thread safe so we have to use the ref stealing 'o' option.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7022">change 7022</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.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.asterisk.org/7022"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib1181a36b317c86bff1ef2e44a17a0b1c73cfdc8 </div>
<div style="display:none"> Gerrit-Change-Number: 7022 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 06 Nov 2017 23:33:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>