Project

General

Profile

Actions

Bug #4052

closed

Test uncompressed bitmaps without length

Added by lynxis almost 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Target version:
-
Start date:
06/07/2019
Due date:
% Done:

100%

Spec Reference:
44.060

Description

Check and test uncompressed bitmaps without length.


Related issues

Related to OsmoPCU - Bug #3728: Problem using compressed ACK/NACK bitmaps or uncompressed bitmaps without lengthClosedlynxis12/13/2018

Actions
Actions #1

Updated by lynxis almost 5 years ago

  • Related to Bug #3728: Problem using compressed ACK/NACK bitmaps or uncompressed bitmaps without length added
Actions #2

Updated by lynxis almost 5 years ago

  • Assignee set to lynxis
Actions #3

Updated by lynxis almost 5 years ago

  • Priority changed from Normal to Urgent
Actions #4

Updated by lynxis almost 5 years ago

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

There is at least one corner cases, which I'ven't found in the 44.060: What happens when an EGPRS ACK/NACK does not have a Length field, but the uncompressed RBB is less the remaining rest_bits? This was possible, when less than 8 bits were remaining, so only happens when having huge packet loss.

... This is defined as spec compliant. Out of window bits should be ignored.

I've implemented a fix for this in my branch lynxis/gprs-ack-nack https://gerrit.osmocom.org/plugins/gitiles/osmo-pcu/+/lynxis/gprs-ack-nack
I will push this patch to gerrit and write test cases for the encoding.

However I want also validate the encoding of the URBB is correct at all.

Actions #5

Updated by lynxis almost 5 years ago

  • Spec Reference set to 44.060
Actions #6

Updated by lynxis almost 5 years ago

lynxis wrote:

There is at least one corner cases, which I'ven't found in the 44.060: What happens when an EGPRS ACK/NACK does not have a Length field, but the uncompressed RBB is less the remaining rest_bits? This was possible, when less than 8 bits were remaining, so only happens when having huge packet loss.

... This is defined as spec compliant. Out of window bits should be ignored.

But those bits are still within the window. They won't be ignored. And bits which represent V(R) or higher must be set to 0 (44.060 9.1.10).

Actions #7

Updated by lynxis over 4 years ago

  • Status changed from In Progress to Feedback
  • % Done changed from 20 to 90

The uncompressed bitmap will be removed.

https://gerrit.osmocom.org/c/osmo-pcu/+/14697

Actions #8

Updated by lynxis over 4 years ago

  • Status changed from Feedback to Closed
  • % Done changed from 90 to 100

The patch is now merged.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)