Project

General

Profile

Bug #1640

Missing release field/length in MS RA capability container

Added by prasadkg over 2 years ago. Updated 2 days ago.

Status:
Stalled
Priority:
High
Assignee:
Target version:
-
Start date:
03/10/2016
Due date:
% Done:

10%

Spec Reference:

Description

Description
In case of MS RA Capability container "Content_t", there is no dedicated field to denote the existence of 3GPP release in the message. The encoder goes out of max length allowed for release supported and encodes incorrectly.
Example vector with failure:
vector1 = 40165e000000268ca2a050740440000000300b2b2b2b2b
vector2 = 40165e00000026d0a2a0507400000220000000180b2b2b
vector1 == vector2 : FALSE

Proposal:
For this, we shall have precomputed value of length to terminate the container encoding before calling Content_Dissector in csn1.cpp file. The pre computing length logic routine shall consider the presence of different 3GPP releases.

For example in case of release 5 RA capability message , encoder should be able to code only till release 5 fields . In case of release 6 RA capability, encoder must be able to code till release 6. This release information must be available to the encoder to calculate length. Decoder will set the release field based on the fields present in the message.

History

#1 Updated by laforge almost 2 years ago

the proposal seems fine to me. will your team be working on this?

#2 Updated by arvind.sirsikar almost 2 years ago

  • Status changed from New to Stalled

Currently no impact on the test scenarios. Hence low priority.

#3 Updated by laforge about 1 year ago

  • Assignee set to sysmocom

#4 Updated by laforge about 1 year ago

  • Status changed from Stalled to New
  • Assignee changed from sysmocom to msuraev

#5 Updated by laforge about 1 year ago

  • Priority changed from Normal to High

#6 Updated by msuraev 9 months ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 10

#7 Updated by msuraev 9 months ago

  • Status changed from In Progress to Stalled

#8 Updated by laforge 8 months ago

  • Assignee changed from msuraev to sysmocom

#9 Updated by laforge 6 months ago

  • Assignee changed from sysmocom to lynxis

#10 Updated by laforge 2 months ago

  • Priority changed from High to Immediate

#11 Updated by laforge 2 months ago

  • Priority changed from Immediate to High

#12 Updated by laforge 17 days ago

#13 Updated by laforge 2 days ago

  • Assignee changed from lynxis to msuraev

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)