<p> Attention is currently required from: N A. </p>
<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4; color: #000000;">Code-Review -1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/19305">View Change</a></p><p>7 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File res/res_pjsip_geolocation.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/asterisk/+/19305/comment/ef0627ac_fa62f6b6">Patch Set #1, Line 148:</a> <code style="font-family:monospace,monospace">     RAII_VAR(struct ast_str *, buf, ast_str_create(1024), ast_free);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">While you're in this module could you fix this as well?  I shouldn't be allocating the buffer here when 99% of the incoming invites won't need it.  </p><p style="white-space: pre-wrap; word-wrap: break-word;">Change this to...<br>```<br>RAII_VAR(struct ast_str *, buf, NULL, ast_free);<br>```<br>and add the create below where indicated.  Thanks!</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/bf22d8b1_9821f91c">Patch Set #1, Line 194:</a> <code style="font-family:monospace,monospace">                 SCOPE_EXIT_LOG_RTN_VALUE(0, LOG_DEBUG, "%s: Endpoint has no geoloc_incoming_call_profile. "</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Something odd here is if that if you use 1 instead 0, things behave as if an ast_channel_unref was forgotten somewhere.... actually what happens is the debug is executed but nothing ever happens on the channel afterwards and it gets "stuck" there.</p><p style="white-space: pre-wrap; word-wrap: break-word;">0 works fine, but then the debug ALWAYS gets outputted even if core debug level is 0, and I don't think that is appropriate either.</p><p style="white-space: pre-wrap; word-wrap: break-word;">What I'm trying to do is the debug level with at least sub level 1.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">If you return "1", res_pjsip_session:handle_incoming_request stops processing additional supplements which may explain that behavior. Also, all of the SCOPE_EXIT* macros automatically call ast_debug.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Instead, call SCOPE_EXIT_RTN_VALUE (no LOG).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/ad7c6737_c2fa097e">Patch Set #1, Line 211:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Insert here...</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">```<br>buf = ast_str_create(1024);<br>if (!buf) {<br>      SCOPE_EXIT_LOG_RTN_VALUE(0, LOG_WARNING, "%s: Unable to allocate buf\n");<br>}<br>```</pre></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/d76efef3_cfabe7d1">Patch Set #1, Line 472:</a> <code style="font-family:monospace,monospace">        RAII_VAR(struct ast_str *, buf, ast_str_create(1024), ast_free);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Change here as well.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/a66f6e28_d9f10327">Patch Set #1, Line 481:</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 (!buf) {<br>           SCOPE_EXIT_LOG_RTN(LOG_WARNING, "%s: Unable to allocate buf\n",<br>                     session_name);<br>        }<br><br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">and move this below where indicated.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/e8159e95_a34f3cc6">Patch Set #1, Line 497:</a> <code style="font-family:monospace,monospace">SCOPE_EXIT_LOG_RTN</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You can just all SCOPE_EXIT_RTN without the LOG.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19305/comment/916109a6_99eefc6b">Patch Set #1, Line 580:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Add the ast_str_create and move the error check here.  Thanks!</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19305">change 19305</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/+/19305"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I656b355d812f67cc0f0fdf09b00b0e1458598bb4 </div>
<div style="display:none"> Gerrit-Change-Number: 19305 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 26 Sep 2022 11:55:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>