pespin has submitted this change. (
https://gerrit.osmocom.org/c/libosmo-sigtran/+/41404?usp=email )
Change subject: mtp_sap.h: Protect MTP_SIO macro params with parenthesis
......................................................................
mtp_sap.h: Protect MTP_SIO macro params with parenthesis
Change-Id: I0ba8139fd5458ff3637eebaaba0d4c0edd080e6a
---
M include/osmocom/sigtran/mtp_sap.h
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
fixeria: Looks good to me, approved
Jenkins Builder: Verified
laforge: Looks good to me, but someone else must approve
diff --git a/include/osmocom/sigtran/mtp_sap.h
b/include/osmocom/sigtran/mtp_sap.h
index 05308c2..f13d6f1 100644
--- a/include/osmocom/sigtran/mtp_sap.h
+++ b/include/osmocom/sigtran/mtp_sap.h
@@ -32,7 +32,7 @@
};
/* ITU Q.704 14.2 Service information octet. See enum mtp_si_ni00 in mtp.h. */
-#define MTP_SIO(service, net_ind) (((net_ind & 0x3) << 6) | (service &
0xF))
+#define MTP_SIO(service, net_ind) ((((net_ind) & 0x3) << 6) | ((service)
& 0xF))
struct osmo_mtp_transfer_param {
uint32_t opc;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/41404?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I0ba8139fd5458ff3637eebaaba0d4c0edd080e6a
Gerrit-Change-Number: 41404
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
