Project

General

Profile

Actions

Bug #1528

open

Find a real fix for EGPRS_AckNack_w_len_t decoding

Added by zecke about 8 years ago. Updated about 3 years ago.

Status:
New
Priority:
Low
Assignee:
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.

Actions #1

Updated by zecke about 8 years ago

  • Priority changed from Normal to Low
Actions #2

Updated by laforge about 6 years ago

  • Assignee deleted (msuraev)
Actions #3

Updated by laforge about 6 years ago

  • Assignee set to 4368
Actions #4

Updated by laforge about 3 years ago

  • Assignee changed from 4368 to pespin
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)