diff options
author | Vadim Yanitskiy <axilirator@gmail.com> | 2019-09-10 18:11:52 +0200 |
---|---|---|
committer | Vadim Yanitskiy <axilirator@gmail.com> | 2019-09-11 14:04:07 +0200 |
commit | d8f28e6d19563fc629e2e242e7c4e3cdb7515dcb (patch) | |
tree | 66d14aeec30e72360f0253f989a8720ee8c4e599 | |
parent | e5f4a91576bf979109efba55c053a85819871850 (diff) |
Revert "library/PCUIF_Types.ttcn: inform RAW codec about PADDING in PCUIF_data"
This reverts commit dd6d5d1baa62dc9d50fa90ef01fccb8a10284d53.
The PADDING seems to be a very experimental feature of TITAN. It works
very well for decoding of messages, so the padding bytes are getting
recognized as expected, but encoding is broken. Not only the data
buffer and its length are encoded in a wrong way, but other fields too.
Change-Id: Ieefa61232eb215a19a02e490255332e28e23b8f8
-rw-r--r-- | library/PCUIF_Types.ttcn | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/library/PCUIF_Types.ttcn b/library/PCUIF_Types.ttcn index 721eb649..27d9b92e 100644 --- a/library/PCUIF_Types.ttcn +++ b/library/PCUIF_Types.ttcn @@ -77,15 +77,10 @@ type record PCUIF_txt_ind { PCUIF_Text text } with { variant "" }; -/* This is a bad side of the current protocol design: fixed-size buffer is located - * in between the other fields of the message, so if the payload is smaller than - * 162 octets (1296 bits), we need to fill the remaining space with padding. */ -type octetstring PCUIF_DataPad with { variant "ALIGN(left), PADDING(1296)" }; - type record PCUIF_data { PCUIF_Sapi sapi, uint8_t len, - PCUIF_DataPad data, + octetstring data length(162), uint32_t fn, uint16_t arfcn, uint8_t trx_nr, @@ -95,7 +90,7 @@ type record PCUIF_data { uint16_t ber10k, int16_t ta_offs_qbits, int16_t lqual_cb -} with { variant (len) "LENGTHTO(data)" }; +} with { variant (data) "FIELDLENGTH(162), ALIGN(left)" }; type record PCUIF_data_cnf_dt { PCUIF_Sapi sapi, |