Change in osmo-bts[master]: rsl: use tlvp_val16be() in rsl_rx_ipac_XXcx()

fixeria gerrit-no-reply at lists.osmocom.org
Sat Jul 3 23:18:34 UTC 2021


fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/24837 )


Change subject: rsl: use tlvp_val16be() in rsl_rx_ipac_XXcx()
......................................................................

rsl: use tlvp_val16be() in rsl_rx_ipac_XXcx()

Instead of using tlvp_val16_unal() and then doing ntohs() here
and there, convert the remote port to the host byte order once.

Change-Id: Id883a976a03fd3022ed9d66f703b01244df2d9db
---
M src/common/rsl.c
1 file changed, 4 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/37/24837/1

diff --git a/src/common/rsl.c b/src/common/rsl.c
index 010b9d3..29388a1 100644
--- a/src/common/rsl.c
+++ b/src/common/rsl.c
@@ -2615,9 +2615,8 @@
 	}
 
 	if (TLVP_PRES_LEN(&tp, RSL_IE_IPAC_REMOTE_PORT, 2)) {
-		connect_port = tlvp_val16_unal(&tp, RSL_IE_IPAC_REMOTE_PORT);
-		LOGPC(DRSL, LOGL_DEBUG, "connect_port=%u ",
-		      ntohs(connect_port));
+		connect_port = tlvp_val16be(&tp, RSL_IE_IPAC_REMOTE_PORT);
+		LOGPC(DRSL, LOGL_DEBUG, "connect_port=%u ", connect_port);
 	}
 
 	speech_mode = TLVP_VAL(&tp, RSL_IE_IPAC_SPEECH_MODE);
@@ -2733,7 +2732,7 @@
 	} else
 		ia.s_addr = connect_ip;
 	rc = osmo_rtp_socket_connect(lchan->abis_ip.rtp_socket,
-				     inet_ntoa(ia), ntohs(connect_port));
+				     inet_ntoa(ia), connect_port);
 	if (rc < 0) {
 		LOGPLCHAN(lchan, DRTP, LOGL_ERROR, "Failed to connect RTP/RTCP sockets\n");
 		osmo_rtp_socket_free(lchan->abis_ip.rtp_socket);
@@ -2744,7 +2743,7 @@
 	}
 	/* save IP address and port number */
 	lchan->abis_ip.connect_ip = ntohl(ia.s_addr);
-	lchan->abis_ip.connect_port = ntohs(connect_port);
+	lchan->abis_ip.connect_port = connect_port;
 
 	rc = osmo_rtp_get_bound_ip_port(lchan->abis_ip.rtp_socket,
 					&lchan->abis_ip.bound_ip,

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/24837
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Id883a976a03fd3022ed9d66f703b01244df2d9db
Gerrit-Change-Number: 24837
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210703/0b979967/attachment.htm>


More information about the gerrit-log mailing list