osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

rlc: Fix bug in 'RLC over UDP' dissector 

Reassembly of AM/UM packets doesn't work when the URNTI tag is missing and the default value of 0 is kept. This patch makes the default value 1 for those cases.

Bug: 14129
Change-Id: Id13121b5de63da4318214871b8963ceef5d28cf0
Reviewed-on: https://code.wireshark.org/review/23930
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
This commit is contained in:
Darien Spencer 2017-10-15 21:57:06 +03:00 committed by Pascal Quantin
parent 6aba76666d
commit 2663881c37
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
epan/dissectors/packet-umts_rlc.c
View File

@ -2701,6 +2701,10 @@ dissect_rlc_heur(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data
rlcInfoAlreadySet = TRUE;
}
/* Setting non-zero UE-ID for RLC reassembly to work, might be
* overriden if the optional URNTI tag is present */
rlci->ueid[fpi->cur_tb] = 1;
/* Read conditional/optional fields */
while (tag != RLC_PAYLOAD_TAG) {
/* Process next tag */