Bug 2754

Summary: Incorrect arguments are passed Asn1Header::SerializeInteger
Product: ns-3 Reporter: Alexander Krotov <krotov>
Component: lteAssignee: Manuel Requena <manuel.requena>
Status: RESOLVED FIXED    
Severity: major CC: manuel.requena, ns-bugs, tomh
Priority: P3    
Version: ns-3-dev   
Hardware: All   
OS: All   
Attachments: Patch that adds assertion
Fix some related bugs
Patch for ASN.1 encoding of RRC messages

Description Alexander Krotov 2017-06-13 10:23:59 EDT
Created attachment 2870 [details]
Patch that adds assertion

Adding assert (nmin <= n && n <= nmax) to Asn1Header::SerializeInteger causes a lot of tests and examples, maybe even all of them which use LteRrcProtocolReal, to crash.
Comment 1 Alexander Krotov 2017-06-13 10:54:36 EDT
Created attachment 2871 [details]
Fix some related bugs

Here is a patch that fixes some of the bugs but crashes still remain.

lte-mimo crashes because m_rrcTransactionIdentifier is 4, for example
Comment 2 Manuel Requena 2018-02-21 10:21:00 EST
Created attachment 3055 [details]
Patch for ASN.1 encoding of RRC messages

Following patch solves all ASN.1 encoding problems.
It also includes the ASSERT and partial patch by Alexander.
Comment 3 Manuel Requena 2018-02-21 10:36:31 EST
patch commit in changeset:   13300:9c62f5c32658