diff options
author | John Thacker <johnthacker@gmail.com> | 2023-09-01 20:58:01 -0400 |
---|---|---|
committer | AndersBroman <a.broman58@gmail.com> | 2023-09-02 10:19:15 +0000 |
commit | f2f7c7bf1c8ab1611541f2f1f84ee1e64423ba4a (patch) | |
tree | d7d7f602552a5e1d1cf3a1f73eb8a0a592c710e3 /epan/dissectors/packet-gtpv2.c | |
parent | 3679783182d51ef63e4cb0694eaac81e05ea5974 (diff) |
Remove some uses of tvb_new_subset_length_caplen()
These are all very obvious what was intended, and should be
tvb_new_subset_length() or tvb_new_subset_remaining() instead.
A few of them could throw exceptions (and sometimes the wrong
exception) on packets where the captured length was not the full
reported length, but for most of these this change has no effect
(especially after commit 0c4dcc164be292790faa4707ccf06dfb6dc9dcb9)
This makes it easier to find the calls that need fixing.
Diffstat (limited to 'epan/dissectors/packet-gtpv2.c')
-rw-r--r-- | epan/dissectors/packet-gtpv2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-gtpv2.c b/epan/dissectors/packet-gtpv2.c index 59c2bb628f..c6166a9046 100644 --- a/epan/dissectors/packet-gtpv2.c +++ b/epan/dissectors/packet-gtpv2.c @@ -5288,7 +5288,7 @@ dissect_complete_request_msg(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree offset += 1; /* Add the Complete Request Message */ - new_tvb = tvb_new_subset_length_caplen(tvb, offset, length-1, length-1); + new_tvb = tvb_new_subset_length(tvb, offset, length-1); call_dissector(nas_eps_handle, new_tvb, pinfo, tree); } |