[Asterisk-code-review] pbx_dundi: Fix PJSIP endpoint configuration check. (asterisk[18])

Friendly Automation asteriskteam at digium.com
Thu Apr 13 16:21:56 CDT 2023


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/20082 )

Change subject: pbx_dundi: Fix PJSIP endpoint configuration check.
......................................................................

pbx_dundi: Fix PJSIP endpoint configuration check.

ASTERISK-28233

Change-Id: I0f11c096b307a6178e22ca49d9c756343f0e1fdc
---
M pbx/pbx_dundi.c
1 file changed, 12 insertions(+), 1 deletion(-)

Approvals:
  N A: Looks good to me, but someone else must approve
  Sean Bright: Looks good to me, approved
  Friendly Automation: Approved for Submit




diff --git a/pbx/pbx_dundi.c b/pbx/pbx_dundi.c
index a353bad..95239d2 100644
--- a/pbx/pbx_dundi.c
+++ b/pbx/pbx_dundi.c
@@ -4896,7 +4896,7 @@
 		/* PJSIP requires an endpoint to be specified explicitly. */
 		if (outgoing_sip_tech == DUNDI_PROTO_PJSIP) {
 			char *number, *ip = ast_strdupa(results[x].dest);
-			if (!ast_strlen_zero(pjsip_outgoing_endpoint)) {
+			if (ast_strlen_zero(pjsip_outgoing_endpoint)) {
 				ast_log(LOG_WARNING, "PJSIP calls require an endpoint to be specified explicitly (use the pjsip_outgoing_endpoint option in dundi.conf)\n");
 				return -1;
 			}

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/20082
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I0f11c096b307a6178e22ca49d9c756343f0e1fdc
Gerrit-Change-Number: 20082
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: N A <asterisk at phreaknet.org>
Gerrit-Reviewer: Sean Bright <sean at seanbright.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20230413/cb4a400b/attachment.html>


More information about the asterisk-code-review mailing list