Project

General

Profile

Bug #1528

Find a real fix for EGPRS_AckNack_w_len_t decoding

Added by zecke almost 4 years ago. Updated almost 2 years ago.

Status:
New
Priority:
Low
Assignee:
sysmocom
Target version:
-
Start date:
02/22/2016
Due date:
% Done:

0%

Spec Reference:

Description

Refers to commit: 'edge: Workaround to fix decoding of EGPRS_AckNack_w_len_t'

The presence of the LENGTH field adds an additional offset which breaks the related M_SERIALIZE in gsm_rlcmac.cpp. In that case, the Desc in EGPRS_AckNack_t is being cast to EGPRS_AckNack_w_len_t and then ((EGPRS_AckNack_w_len_t *)Desc)->Desc is filled with the
parsed data, which is a platform dependant number of bytes apart the real Desc struct.

That commit removes the LENGTH field from EGPRS_AckNack_w_len_t so that the Desc field is the first field.

Note that this is not a real fix. The rlcmac wireshark dissector
still has the same declaration but doesn't seem to suffer from this
problem. So there is probably some other bug.

History

#1 Updated by zecke almost 4 years ago

  • Priority changed from Normal to Low

#2 Updated by laforge almost 2 years ago

  • Assignee deleted (msuraev)

#3 Updated by laforge almost 2 years ago

  • Assignee set to sysmocom

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)