diff options
| author | Jaap Keuter <jaap.keuter@xs4all.nl> | 2016-03-01 20:14:10 +0100 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-03-02 17:22:13 +0000 |
| commit | aa52850ad259926a12724f25f499b607e9a2b1be (patch) | |
| tree | d915a62a3fe1b42c5969e6351dffdd18452ea4f2 /epan | |
| parent | 9b3c5202e959ef05fc6df62c40adec5d87a03590 (diff) | |
Remove useless pointer checks (CID 281378)
rpc_call cannot be NULL at this point, so don't check for it,
especially not incorrectly.
Change-Id: I90fcb064ce479d71edf3b4cb0ebea9a5ab623119
Reviewed-on: https://code.wireshark.org/review/14268
Petri-Dish: Jaap Keuter <jaap.keuter@xs4all.nl>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-rpc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c index 0e015fbbf7..afcfdc41fe 100644 --- a/epan/dissectors/packet-rpc.c +++ b/epan/dissectors/packet-rpc.c @@ -2638,7 +2638,7 @@ dissect_rpc_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, offset += 4; /* Indicate the frame to which this is a reply. */ - if(rpc_call && rpc_call->req_num){ + if (rpc_call->req_num) { proto_item *tmp_item; tmp_item=proto_tree_add_uint_format(rpc_tree, hf_rpc_repframe, @@ -2656,7 +2656,7 @@ dissect_rpc_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, } - if ((!rpc_call) || (rpc_call->rep_num == 0)) { + if (rpc_call->rep_num == 0) { /* We have not yet seen a reply to that call, so this must be the first reply; remember its frame number. */ |