[Asterisk-code-review] res xmpp: fix inverted return code check (asterisk[13])

[Sean Bright]

Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/6249 )

Change subject: res_xmpp: fix inverted return code check
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/6249/1/res/res_xmpp.c
File res/res_xmpp.c:

https://gerrit.asterisk.org/#/c/6249/1/res/res_xmpp.c@3920
PS1, Line 3920: 	if (ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1) != 0) {
Just do:

  if (ast_func_read(NULL, cmd, cBuf, sizeof(cBuf) - 1)) {



-- 
To view, visit https://gerrit.asterisk.org/6249
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: I9ef903f80702d1218e8701f65a4e5e918e6548fb
Gerrit-Change-Number: 6249
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Kuron <m.kuron at gmx.de>
Gerrit-Reviewer: Andrey <andr06 at gmail.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Michael Kuron <m.kuron at gmx.de>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Mon, 21 Aug 2017 21:18:29 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170821/f1256863/attachment.html>