diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-08-21 22:34:50 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-08-21 22:37:22 +0000 |
commit | c11ceb3a0da74711f8fd9fa6598d5d6fab5ff57b (patch) | |
tree | 40621794389fbb24089fa78453045d2cef814ff9 /epan/dissectors/packet-rpc.c | |
parent | a0b2a5989d80486916438a42364f50b46d1b0645 (diff) |
rpc: remove unnecessary NULL check
call_dissect_function() does already check internally
if dissect_function is NULL
Change-Id: I4780733ee38bcde74b498bedb1031eefa3c07da4
Reviewed-on: https://code.wireshark.org/review/23159
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 | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c index eb5fefcc65..6e9d104b3d 100644 --- a/epan/dissectors/packet-rpc.c +++ b/epan/dissectors/packet-rpc.c @@ -1586,11 +1586,9 @@ dissect_rpc_authgss_integ_data(tvbuff_t *tvb, packet_info *pinfo, tvb, offset+4, 4, seq); offset += 8; - if (dissect_function != NULL) { - /* offset = */ - call_dissect_function(tvb, pinfo, gtree, offset, - dissect_function, progname, rpc_call); - } + /* offset = */ + call_dissect_function(tvb, pinfo, gtree, offset, dissect_function, progname, rpc_call); + offset += rounded_length - 4; offset = dissect_rpc_authgss_token(tvb, tree, offset, pinfo, hf_rpc_authgss_checksum); |