diff options
| author | Martin Mathieson <martin.mathieson@keysight.com> | 2020-06-29 22:16:19 +0100 |
|---|---|---|
| committer | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2020-06-30 12:11:15 +0000 |
| commit | 570a0cb30134de8996067981841fb8899fbdecc4 (patch) | |
| tree | 8bdb59546fbc69f950d3bf6dc44f84b8c30114e2 /epan | |
| parent | d70bd4489feed4035c3a35cd2e76cdf877b3f485 (diff) | |
Set member variables in initialiser list (rather than in CTOR bodies).
Also simplify some boolean logic in packet-dcerpc.c.
All reported by cppcheck.
Change-Id: I2075f2ec10dc777ad7635da4ef056d17fc5b0be0
Reviewed-on: https://code.wireshark.org/review/37609
Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-dcerpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-dcerpc.c b/epan/dissectors/packet-dcerpc.c index e0ca35e451..a4058502a5 100644 --- a/epan/dissectors/packet-dcerpc.c +++ b/epan/dissectors/packet-dcerpc.c @@ -4487,7 +4487,7 @@ dissect_dcerpc_cn_stub(tvbuff_t *tvb, int offset, packet_info *pinfo, fd_head = fragment_add_seq_next(&dcerpc_co_reassembly_table, decrypted_tvb, 0, pinfo, frame, NULL, tvb_reported_length(decrypted_tvb), - hdr->flags&PFC_LAST_FRAG ? FALSE : TRUE /* more_frags */); + !(hdr->flags & PFC_LAST_FRAG) /* more_frags */); end_cn_stub: |