commit 1145fd263ce4d7c3d62639c6fce2513e7e0c3d14
author Vincent Helfre <vincenthelfre@googlemail.com> Tue Jan 28 18:27:49 2020 +0100
committer Vadim Yanitskiy <axilirator@gmail.com> Sat Mar 07 05:04:40 2020 +0700
tree 4808730723f2a13c9752fd3395c57d646294e717
parent 2679ec0a9f2fab76c8943adfd588e558688fecc4

gsm_rlcmac: improve dissection of MS RA Capability IE

Port from wireshark.git de028e81c53f9c45ccc5adb3bffd2f16ae2017bf

This commit breaks transcoding of the test vectors containing
the MS RA Capability IE due to the reasons explained in [1].

The more fields we add, the longer gets the output of the CSN.1
encoder. This is not critical, since we never need to encode
messages containing the MS RA Capability IE on practice.

[1] Ibb4cbd3f5865415fd547e95fc24ff31df1aed4c0

Ported-by: Pau Espin Pedrol <pespin@sysmocom.de>
Change-Id: Ibb4cbd3f5865415fd547e95fc24ff31df1aed4c0

tests/rlcmac/RLCMACTest.ok

diff --git a/tests/rlcmac/RLCMACTest.ok b/tests/rlcmac/RLCMACTest.ok
index fdf4ff0..438417f 100644
--- a/tests/rlcmac/RLCMACTest.ok
+++ b/tests/rlcmac/RLCMACTest.ok
@@ -104,15 +104,15 @@
 =========Start ENCODE=============
 +++++++++Finish ENCODE (0)+++++++++++
 vector1 = 40 16 71 3d c0 94 27 0c a2 ae 57 ef 90 90 06 aa 0f c0 00 1f 80 22 2b 
-vector2 = 40 16 71 3d c0 94 27 0c a2 ae 57 ef 90 90 06 aa 0f c0 00 1f 80 22 2b 
-vector1 == vector2 : TRUE
+vector2 = 40 16 71 3d c0 94 27 48 a2 ae 57 ef 90 90 06 aa 00 00 1f 80 00 3f 00 
+vector1 == vector2 : FALSE
 vector1 = 401673c87f24af2632b25964200600000091000b780080
 =========Start DECODE===========
 +++++++++Finish DECODE (0)++++++++++
 =========Start ENCODE=============
 +++++++++Finish ENCODE (0)+++++++++++
 vector1 = 40 16 73 c8 7f 24 af 26 32 b2 59 64 20 06 00 00 00 91 00 0b 78 00 80 
-vector2 = 40 16 73 c8 7f 24 ae d2 32 b2 59 64 20 06 12 20 01 6f 00 10 0b 2b 2b 
+vector2 = 40 16 73 c8 7f 24 af 0e 32 b2 59 64 20 06 00 00 24 40 02 de 00 20 2b 
 vector1 == vector2 : FALSE
 vector1 = 40200ffc0021ec010b2b2b2b2b2b2b2b2b2b2b2b2b2b2b
 =========Start DECODE===========
@@ -138,6 +138,6 @@
 === Test encoding of MS RA Capability ===
 encode_gsm_ra_cap() returns 0
 vector1 (len_ind=29) = 13 a5 14 62 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 
-vector2 (len_ind=50) = 16 45 14 62 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 
+vector2 (len_ind=65) = 18 25 14 62 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 
 === Test decoding of a malformed vector (short length indicator) ===
 decode_gsm_ra_cap() returns -5