kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-17 10:26:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_xmpp: fix inverted return code check in OAuth

Browse Source 
fetch_access_token calls func_curl via ast_func_read. The latter returns 0 upon
success and -1 if the function is not available.
This commit inverts the return code check so that an error is printed if the
module is not loaded and not if it is loaded.

ASTERISK-27207 #close

Change-Id: I9ef903f80702d1218e8701f65a4e5e918e6548fb
This commit is contained in:
Michael Kuron
2017-08-20 15:15:37 +02:00
parent 12e63bdefc
commit 83b81d1f8d
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
res/res_xmpp.c
View File

@@ -3915,7 +3915,7 @@ static int fetch_access_token(struct ast_xmpp_client_config *cfg)
ast_debug(2, "Performing OAuth 2.0 authentication for client '%s' using command: %s\n",
cfg->name, cmd);
if (!ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1)) {
if (ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1)) {
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",
cfg->name);
return -1;