<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6828">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Sean Bright: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">chan_sip: Fix output of 'sip set debug off'.<br><br>When sip.conf contains 'sipdebug=yes' it is impossible to disable it<br>using CLI 'sip set debug off'.  This corrects the output of that CLI<br>command to instruct the user to turn sipdebug off in the configuration<br>file.<br><br>ASTERISK-23462 #close<br><br>Change-Id: I1cceade9caa9578e1b060feb832e3495ef5ad318<br>---<br>M channels/chan_sip.c<br>1 file changed, 6 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/channels/chan_sip.c b/channels/chan_sip.c<br>index caf7802..ace1f82 100644<br>--- a/channels/chan_sip.c<br>+++ b/channels/chan_sip.c<br>@@ -22635,7 +22635,12 @@<br>           } else if (!strcasecmp(what, "off")) {<br>                      sipdebug &= ~sip_debug_console;<br>                   sipdebug_text = 0;<br>-                   ast_cli(a->fd, "SIP Debugging Disabled\n");<br>+                     if (sipdebug == sip_debug_none) {<br>+                            ast_cli(a->fd, "SIP Debugging Disabled\n");<br>+                     } else {<br>+                             ast_cli(a->fd, "SIP Debugging still enabled due to configuration.\n");<br>+                          ast_cli(a->fd, "Set sipdebug=no in sip.conf and reload to actually disable.\n");<br>+                        }<br>                     return CLI_SUCCESS;<br>           }<br>     } else if (a->argc == e->args + 1) { /* ip/peer */<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6828">change 6828</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/6828"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I1cceade9caa9578e1b060feb832e3495ef5ad318 </div>
<div style="display:none"> Gerrit-Change-Number: 6828 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>