[asterisk-commits] res pjsip: Fix 'A = B != C' kind. (asterisk[master])
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Dec 9 05:33:27 CST 2016
Joshua Colp has submitted this change and it was merged. ( https://gerrit.asterisk.org/4601 )
Change subject: res_pjsip: Fix 'A = B != C' kind.
......................................................................
res_pjsip: Fix 'A = B != C' kind.
Consider reviewing the expression of the 'A = B != C' kind.
The expression is calculated as following: 'A = (B != C)'
Change-Id: Ibaa637dfda47d51a20e26069d3103e05ce80003d
---
M res/res_pjsip/pjsip_options.c
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Kevin Harwell: Looks good to me, approved
Richard Mudgett: Looks good to me, but someone else must approve
Anonymous Coward #1000019: Verified
diff --git a/res/res_pjsip/pjsip_options.c b/res/res_pjsip/pjsip_options.c
index 09fe155..24e9272f 100644
--- a/res/res_pjsip/pjsip_options.c
+++ b/res/res_pjsip/pjsip_options.c
@@ -701,7 +701,8 @@
pj_status_t status;
/* Make the response object */
- if ((status = ast_sip_create_response(rdata, code, NULL, &tdata) != PJ_SUCCESS)) {
+ status = ast_sip_create_response(rdata, code, NULL, &tdata);
+ if (status != PJ_SUCCESS) {
ast_log(LOG_ERROR, "Unable to create response (%d)\n", status);
return status;
}
--
To view, visit https://gerrit.asterisk.org/4601
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibaa637dfda47d51a20e26069d3103e05ce80003d
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Badalian Vyacheslav <v.badalyan at open-bs.ru>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
More information about the asterisk-commits
mailing list