From 4d358c00e1b3389291bdb3d5c40d0da8d6cf4d02 Mon Sep 17 00:00:00 2001 From: Neels Hofmeyr Date: Thu, 22 Feb 2018 03:19:05 +0100 Subject: gsm48_ra_id_by_bts(): struct gsm48_ra_id* instead of buf MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Move from using deprecated gsm48_construct_ra(), which uses a buf, to gsm48_encode_ra(), which uses a gsm48_ra_id argument. Pass struct gsm48_ra_id around instead of a buf. struct gsm48_ra_id is the "encoded" representation of the bytes in a typical MCC-MNC-LAC-RAC (04.08 Routing Area Id IE, 3GPP TS 24.008 ยง 10.5.5.15). Using the struct spares using magic numbers for byte offsets. In the process, fix a sanitizer warning for unaligned access by using memcpy() instead of pointer assignment: osmo-bsc/src/libbsc/abis_nm.c:2857:27: runtime error: store to misaligned address 0x7ffe8e0d6695 for type 'uint16_t', which requires 2 byte alignment Note that (besides removing a now unnecessary cast) the gsm0408_test and abis_test tests of RAI / CGI encoding remain stable, which indicates that the new code is still correct. Change-Id: I0d3908fb8ca1e2e669d257b5d59b40675fa85d06 --- tests/gsm0408/gsm0408_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/gsm0408/gsm0408_test.c b/tests/gsm0408/gsm0408_test.c index d1d50f135..aeec56f4c 100644 --- a/tests/gsm0408/gsm0408_test.c +++ b/tests/gsm0408/gsm0408_test.c @@ -801,7 +801,7 @@ static void test_gsm48_ra_id_by_bts() bts.location_area_code = t->lac; bts.gprs.rac = t->rac; - gsm48_ra_id_by_bts((uint8_t*)&result, &bts); + gsm48_ra_id_by_bts(&result, &bts); ok = (t->expect.digits[0] == result.digits[0]) && (t->expect.digits[1] == result.digits[1]) -- cgit v1.2.3