diff options
author | Darien Spencer <cusneud@mail.com> | 2017-10-15 21:57:06 +0300 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2017-10-15 20:26:19 +0000 |
commit | 2663881c378913d5c9cd123c469e1bb29a5e39be (patch) | |
tree | 35b04e6ac5fd7bc644a3f0f6bd83ae648e0c39a0 | |
parent | 6aba76666da060f12b379cf8786686e545007f2a (diff) |
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>
-rw-r--r-- | epan/dissectors/packet-umts_rlc.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/epan/dissectors/packet-umts_rlc.c b/epan/dissectors/packet-umts_rlc.c index 00f553fe90..9d1ab2f0ab 100644 --- a/epan/dissectors/packet-umts_rlc.c +++ b/epan/dissectors/packet-umts_rlc.c @@ -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 */ |