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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_xmpp: fix inverted return code check in OAuth<br><br>fetch_access_token calls func_curl via ast_func_read. The latter returns 0 upon<br>success and -1 if the function is not available.<br>This commit inverts the return code check so that an error is printed if the<br>module is not loaded and not if it is loaded.<br><br>ASTERISK-27207 #close<br><br>Change-Id: I9ef903f80702d1218e8701f65a4e5e918e6548fb<br>---<br>M res/res_xmpp.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_xmpp.c b/res/res_xmpp.c<br>index f8eb505..444eacb 100644<br>--- a/res/res_xmpp.c<br>+++ b/res/res_xmpp.c<br>@@ -3915,7 +3915,7 @@<br>      ast_debug(2, "Performing OAuth 2.0 authentication for client '%s' using command: %s\n",<br>             cfg->name, cmd);<br> <br>-       if (!ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1)) {<br>+     if (ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1)) {<br>               ast_log(LOG_ERROR, "CURL is unavailable. This is required for OAuth 2.0 authentication of XMPP client '%s'. Please ensure it is loaded.\n",<br>                         cfg->name);<br>                return -1;<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6254">change 6254</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/6254"/><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-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I9ef903f80702d1218e8701f65a4e5e918e6548fb </div>
<div style="display:none"> Gerrit-Change-Number: 6254 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Michael Kuron <m.kuron@gmx.de> </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>