fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/31968 ) Change subject: l1ctl_proto: fix unpacked struct in l1ctl_gprs_dl_block_ind ...................................................................... l1ctl_proto: fix unpacked struct in l1ctl_gprs_dl_block_ind The __attribute__((packed)) does not apply to inner structures. I found this problem when testing the new TTCN-3 message definitions. Change-Id: I90e74330073a9090c2638fc3175ff0216b0da5ec Related: OS#5500 --- M include/l1ctl_proto.h 1 file changed, 14 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve dexter: Looks good to me, approved diff --git a/include/l1ctl_proto.h b/include/l1ctl_proto.h index 9bf40d2..3512424 100644 --- a/include/l1ctl_proto.h +++ b/include/l1ctl_proto.h @@ -376,7 +376,7 @@ /* payload of L1CTL_GPRS_DL_BLOCK_IND */ struct l1ctl_gprs_dl_block_ind { struct l1ctl_gprs_block_hdr hdr; - struct { + struct __attribute__((packed)) { uint16_t ber10k; /* Bit Error Rate */ int16_t ci_cb; /* C/I in centiBels */ uint8_t rx_lev; /* RxLev 0..63 */ -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31968 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I90e74330073a9090c2638fc3175ff0216b0da5ec Gerrit-Change-Number: 31968 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <pmaier@sysmocom.de> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged