aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOliver Smith <osmith@sysmocom.de>2021-02-02 16:20:57 +0100
committerOliver Smith <osmith@sysmocom.de>2021-02-02 16:22:01 +0100
commit20642478f3488f12d650f7d0f1b26f58388f15d8 (patch)
treec2a59cc5f4a5ea321cff785d8fe4d7113c097fe1
parented11cc8385353de33c82093d2e33579d8c7a30d0 (diff)
xua_test: fix gcc 4 + -std=gnu11 error
Fix 'error: initializer element is not constant' with debian 8's gcc 4.9.2, triggered by XUA_HDR. Create a new _XUA_HDR without the type cast, and use it inside of const struct definitions in xua_test.c. The new macro is needed, because removing the type cast from the original XUA_HDR would break other uses. Related: OS#5004 Change-Id: I890432ee976043d012b01023f7dd2cfecf79d115
-rw-r--r--tests/xua/xua_test.c25
1 files changed, 14 insertions, 11 deletions
diff --git a/tests/xua/xua_test.c b/tests/xua/xua_test.c
index 77daa56..6096807 100644
--- a/tests/xua/xua_test.c
+++ b/tests/xua/xua_test.c
@@ -369,6 +369,9 @@ struct sccp2sua_testcase {
#define PANDSIZ(x) { x, ARRAY_SIZE(x) }
#define PARTU32(x, data) { .tag = x, .len = 4, .dat = (uint8_t *) data }
#define PARTARR(x, data) { .tag = x, .len = ARRAY_SIZE(data), .dat = (uint8_t *) data }
+/* GCC-4 errors with 'initializer element is not constant' if using XUA_HDR
+ * inside a const struct (OS#5004) */
+#define _XUA_HDR(class, type) { .spare = 0, .msg_class = (class), .msg_type = (type) }
const uint32_t sua_proto_class0 = 0;
const uint32_t sua_proto_class2 = 2;
@@ -384,7 +387,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-RESET",
.sccp = PANDSIZ(bssmap_reset),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
+ .hdr = _XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class0),
PARTARR(SUA_IEI_DEST_ADDR, sua_addr_ssn_bssmap),
@@ -395,7 +398,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-RESET-ACK",
.sccp = PANDSIZ(bssmap_reset_ack),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
+ .hdr = _XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class0),
PARTARR(SUA_IEI_DEST_ADDR, sua_addr_ssn_bssmap_pc1),
@@ -406,7 +409,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-PAGING",
.sccp = PANDSIZ(bssmap_paging),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
+ .hdr = _XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class0),
PARTARR(SUA_IEI_DEST_ADDR, sua_addr_ssn_bssmap_pc1),
@@ -417,7 +420,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-UDT",
.sccp = PANDSIZ(bssmap_udt),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
+ .hdr = _XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class0),
PARTARR(SUA_IEI_DEST_ADDR, sua_addr_ssn_bssmap),
@@ -428,7 +431,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-CR",
.sccp = PANDSIZ(bssmap_cr),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_CORE),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_CORE),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class2),
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_bsc),
@@ -439,7 +442,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-CC",
.sccp = PANDSIZ(bssmap_cc),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_COAK),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_COAK),
.parts = {
PARTU32(SUA_IEI_PROTO_CLASS, &sua_proto_class2),
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_msc),
@@ -450,7 +453,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-DTAP",
.sccp = PANDSIZ(bssmap_dtap),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_CODT),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_CODT),
.parts = {
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_msc),
},
@@ -459,7 +462,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-CLEAR",
.sccp = PANDSIZ(bssmap_clear),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_CODT),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_CODT),
.parts = {
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_msc),
},
@@ -468,7 +471,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-RELEASED",
.sccp = PANDSIZ(bssmap_released),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_RELRE),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_RELRE),
.parts = {
PARTU32(SUA_IEI_DEST_REF, &sua_loc_ref_msc),
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_bsc),
@@ -479,7 +482,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "BSSMAP-RELEASE_COMPLETE",
.sccp = PANDSIZ(bssmap_release_complete),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CO, SUA_CO_RELCO),
+ .hdr = _XUA_HDR(SUA_MSGC_CO, SUA_CO_RELCO),
.parts = {
PARTU32(SUA_IEI_DEST_REF, &sua_loc_ref_bsc),
PARTU32(SUA_IEI_SRC_REF, &sua_loc_ref_msc),
@@ -489,7 +492,7 @@ static const struct sccp2sua_testcase sccp2sua_testcases[] = {
.name = "TCAP",
.sccp = PANDSIZ(tcap_global_title),
.sua = {
- .hdr = XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
+ .hdr = _XUA_HDR(SUA_MSGC_CL, SUA_CL_CLDT),
.parts = {
},
},