diff options
author | Bill Meier <wmeier@newsguy.com> | 2015-01-23 00:24:46 -0500 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2015-01-23 05:30:55 +0000 |
commit | 8d1a9848e3acc266447e1ccc032f377ba559ca50 (patch) | |
tree | a9bcbf39ec065f2ef3e37140abd3ce7b36855e40 /epan/dissectors/packet-ismacryp.c | |
parent | 5a23a02ab8ae59c451db76bd9304ee420f3e3be5 (diff) |
Remove incorrect 'if (tree)'
Incorrect because of one or both of the following:
- col_...()/expert...() called under 'if (tree)'
- vars set under 'if (tree)' used later (not under 'if (tree)'
as args to col_...()/expert_...()
Change-Id: I89f7d453f2d6eaa40d51cbd794ed2c9be7e549de
Reviewed-on: https://code.wireshark.org/review/6754
Reviewed-by: Bill Meier <wmeier@newsguy.com>
Diffstat (limited to 'epan/dissectors/packet-ismacryp.c')
-rw-r--r-- | epan/dissectors/packet-ismacryp.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/dissectors/packet-ismacryp.c b/epan/dissectors/packet-ismacryp.c index c23cb9a632..9e3d52a15a 100644 --- a/epan/dissectors/packet-ismacryp.c +++ b/epan/dissectors/packet-ismacryp.c @@ -318,10 +318,8 @@ static void dissect_ismacryp_common(tvbuff_t *tvb, packet_info *pinfo, proto_tre } /* end if user_mode == FALSE */ /* navigate through buffer */ - if (tree) + /* if (tree) */ { - /* we are being asked for details */ - guint16 au_headers_length = 0; /* total length of AU headers */ guint16 totalbits =0; /* keeps track of total number of AU header bits treated (used to determine end of AU headers) */ int deltabits = -1; /* keeps track of extra bits per AU header treated (used to determine end of AU heafers ) */ |