]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
res_pjsip_session: Correct inverted test in session_outgoing_nat_hook
authorGeorge Joseph <gjoseph@digium.com>
Wed, 14 Jun 2017 13:29:00 +0000 (07:29 -0600)
committerSean Bright <sean.bright@gmail.com>
Wed, 14 Jun 2017 16:06:18 +0000 (11:06 -0500)
There was a typo introduced in commit 776ffd77 which was preventing
the transport's external media address from being used.

ASTERISK-27024 #close
Reported-by: Christopher van de Sande
patches:
patch.diff submitted by Florian Floimair (license 6892)

Change-Id: I7ec617171eaa2d86d2680b00cf37d5088adafc27

res/res_pjsip_session.c

index 1fec089f8efbdd067f8650bb94dfa3e865796c8b..59a8915e1b3e8727ae5b9e4311dfc27a8fb3e721 100644 (file)
@@ -3123,8 +3123,7 @@ static void session_outgoing_nat_hook(pjsip_tx_data *tdata, struct ast_sip_trans
                ast_sockaddr_parse(&addr, host, PARSE_PORT_FORBID);
 
                if (!transport_state->localnet
-                       || (transport_state->localnet
-                               && ast_apply_ha(transport_state->localnet, &addr) == AST_SENSE_ALLOW)) {
+                       || ast_apply_ha(transport_state->localnet, &addr) != AST_SENSE_ALLOW) {
                        ast_debug(5, "Setting external media address to %s\n", transport->external_media_address);
                        pj_strdup2(tdata->pool, &sdp->conn->addr, transport->external_media_address);
                }