diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-08-23 22:16:08 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-08-30 22:57:36 +0000 |
commit | 51789050633fc4554413a6f79439c9162c1c8cfb (patch) | |
tree | 10a00e1e3f1d17d5fd2ab986829af03af6405972 /epan/dissectors/packet-rpc.c | |
parent | 63b66cc7c8a6ec4e28eabd0bea135a70ed707754 (diff) |
rpc: remove two unnecessary DISSECTOR_ASSERT()s
Their conditions are always true. Both tvb_captured_length_remaining()
and tvb_reported_length_remaining() return a gint which is always >= 0.
Change-Id: Ic0c13d17361389053466573fafff44b2c2b478ed
Reviewed-on: https://code.wireshark.org/review/23186
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-rpc.c')
-rw-r--r-- | epan/dissectors/packet-rpc.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c index 111c0b7e7b..f00c88296a 100644 --- a/epan/dissectors/packet-rpc.c +++ b/epan/dissectors/packet-rpc.c @@ -1349,8 +1349,6 @@ dissect_rpc_authgss_token(tvbuff_t* tvb, proto_tree* tree, int offset, if (opaque_length != 0) { length = tvb_captured_length_remaining(tvb, offset); reported_length = tvb_reported_length_remaining(tvb, offset); - DISSECTOR_ASSERT(length >= 0); - DISSECTOR_ASSERT(reported_length >= 0); if (length > reported_length) length = reported_length; if ((guint32)length > opaque_length) |