osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

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>
This commit is contained in:
Martin Kaiser 2017-09-19 23:16:44 +02:00 committed by Michael Mann
parent 70263eade4
commit bbbec5552d
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
epan/dissectors/packet-windows-common.c
View File

@ -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;