From 2b4a67a412ace101f2e3fe893f8b2724c6fd5062 Mon Sep 17 00:00:00 2001 From: Max Date: Fri, 2 Nov 2018 16:12:03 +0100 Subject: Update cipher reject function * add note about (yet) unsupported standard feature * use enum constant instead of hex value Change-Id: I5cc5fc1ae8d5474dbc8d3385a493adf6420a3c98 Related: OS#3187 --- src/gsm/gsm0808.c | 1 + tests/gsm0808/gsm0808_test.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c index 9ccca4d6..e5c3e30a 100644 --- a/src/gsm/gsm0808.c +++ b/src/gsm/gsm0808.c @@ -275,6 +275,7 @@ struct msgb *gsm0808_create_cipher_reject(uint8_t cause) return NULL; msgb_v_put(msg, BSS_MAP_MSG_CIPHER_MODE_REJECT); + /* FIXME: support 2-byte cause value as per 3GPP TS 08.08 ยง3.2.2.5 */ msgb_tlv_put(msg, GSM0808_IE_CAUSE, 1, &cause); msg->l3h = msgb_tv_push(msg, BSSAP_MSG_BSS_MANAGEMENT, msgb_length(msg)); diff --git a/tests/gsm0808/gsm0808_test.c b/tests/gsm0808/gsm0808_test.c index 0b2794f1..7111db9b 100644 --- a/tests/gsm0808/gsm0808_test.c +++ b/tests/gsm0808/gsm0808_test.c @@ -248,7 +248,7 @@ static void test_create_cipher_reject() struct msgb *msg; printf("Testing creating Cipher Reject\n"); - msg = gsm0808_create_cipher_reject(0x23); + msg = gsm0808_create_cipher_reject(GSM0808_CAUSE_CCCH_OVERLOAD); VERIFY(msg, res, ARRAY_SIZE(res)); msgb_free(msg); } -- cgit v1.2.3