[Asterisk-code-review] chan ooh323: Fix confusing indentation warning (asterisk[14])

Jenkins2 asteriskteam at digium.com
Tue Sep 5 07:01:40 CDT 2017


Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/6377 )

Change subject: chan_ooh323: Fix confusing indentation warning
......................................................................

chan_ooh323: Fix confusing indentation warning

ASTERISK-27177 #close
Reported by: Tzafrir Cohen

Change-Id: I40311c404edb2302a7543ad5ca7a06b2a38f2d97
---
M addons/ooh323c/src/ooSocket.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/addons/ooh323c/src/ooSocket.c b/addons/ooh323c/src/ooSocket.c
index e9087d1..55d7619 100644
--- a/addons/ooh323c/src/ooSocket.c
+++ b/addons/ooh323c/src/ooSocket.c
@@ -386,7 +386,7 @@
    if (*pNewSocket <= 0) return ASN_E_INVSOCKET;
 
    if (destAddr != 0) {
-      if ((host = ast_sockaddr_stringify_addr(&addr)) != NULL);
+      if ((host = ast_sockaddr_stringify_addr(&addr)) != NULL)
       	strncpy(destAddr, host, strlen(host));
    }
    if (destPort != 0)

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

Gerrit-Project: asterisk
Gerrit-Branch: 14
Gerrit-MessageType: merged
Gerrit-Change-Id: I40311c404edb2302a7543ad5ca7a06b2a38f2d97
Gerrit-Change-Number: 6377
Gerrit-PatchSet: 1
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170905/f0eb614a/attachment-0001.html>


More information about the asterisk-code-review mailing list