<p style="white-space: pre-wrap; word-wrap: break-word;">StasisStart gets its timestamp from the start_message_blob, which just obtains it from the current system time and passes it in. Which should still make it unique/different from other events (as I don't see any other event referencing the start_message_blob).<br></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/11133">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.asterisk.org/#/c/11133/1/res/res_stasis.c">File res/res_stasis.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.asterisk.org/#/c/11133/1/res/res_stasis.c@128">Patch Set #1, Line 128:</a> <code style="font-family:monospace,monospace">      msg = ast_json_pack("{s: s, s: O, s: o}",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Bumping the ref here makes sense to me, but (as Josh pointed out) this is now doing something similar to StasisStart however it does not bump the ref. Seems like it would cause a double free potentially unless jansson guards against ref counts below zero or NULL things out. Anyway just thought I'd mention it as I'm not sure what's 100% correct here.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/11133/1/res/res_stasis.c@130">Patch Set #1, Line 130:</a> <code style="font-family:monospace,monospace">           "timestamp", ast_json_object_get(payload->blob, "timestamp"),</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">The blob here gets created in "app_send_end_msg" correct? If so I am not seeing where the timestamp gets created there and set, so this will always be NULL.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/11133">change 11133</a>. To unsubscribe, or for help writing mail filters, 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/c/asterisk/+/11133"/><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-Change-Id: I5eb8380fc472f1100535a6bc4983c64767026116 </div>
<div style="display:none"> Gerrit-Change-Number: 11133 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua C. Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 13 Mar 2019 23:22:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>