diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2017-09-19 23:16:44 +0200 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-09-19 23:37:42 +0000 |
commit | bbbec5552d44fde57e57f52946059182482ab987 (patch) | |
tree | 081c2a2e4142f1faa999d67848fca050c67809fa /epan/dissectors/packet-windows-common.c | |
parent | 70263eade4cadd157f0322d6df3dd79bfec26242 (diff) |
windows-common: remove volatile qualifier from num_aces
A variable in a dissector must only be volatile if it's used inside a
CATCH block. This is not the case for num_aces.
Change-Id: I3aedb9ad2f8b96f56c9be2120276b552602658a1
Reviewed-on: https://code.wireshark.org/review/23624
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Martin Kaiser <wireshark@kaiser.cx>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-windows-common.c')
-rw-r--r-- | epan/dissectors/packet-windows-common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-windows-common.c b/epan/dissectors/packet-windows-common.c index 3db38aceaa..501fda22c5 100644 --- a/epan/dissectors/packet-windows-common.c +++ b/epan/dissectors/packet-windows-common.c @@ -2137,7 +2137,7 @@ dissect_nt_acl(tvbuff_t *tvb, int offset_a, packet_info *pinfo, int old_offset = offset_a; int pre_ace_offset; guint16 revision; - volatile guint32 num_aces; + guint32 num_aces; volatile int offset_v = offset_a; volatile gboolean missing_data = FALSE; volatile gboolean bad_ace = FALSE; |