dexter has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/37776?usp=email )

Change subject: mgcp_network: use an uint16_t to store the port number
......................................................................

mgcp_network: use an uint16_t to store the port number

The comment in struct mgcp_rtp_end says that the RTCP port number has to
be stored in network byte order. Such a port number is 16 bit long, so
let's use an uint16_t instead of an int here.

Related: OS#6527
Change-Id: I392b07b243389a78d4ad1d784cdfdc28ec59b487
---
M include/osmocom/mgcp/mgcp_network.h
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, approved




diff --git a/include/osmocom/mgcp/mgcp_network.h 
b/include/osmocom/mgcp/mgcp_network.h
index 8f6505c..453fbe9 100644
--- a/include/osmocom/mgcp/mgcp_network.h
+++ b/include/osmocom/mgcp/mgcp_network.h
@@ -94,7 +94,7 @@
        struct osmo_sockaddr addr;

        /* in network byte order */
-       int rtcp_port;
+       uint16_t rtcp_port;

        /* currently selected audio codec */
        struct mgcp_rtp_codec *codec;

--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/37776?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I392b07b243389a78d4ad1d784cdfdc28ec59b487
Gerrit-Change-Number: 37776
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>

Reply via email to