diff options
author | Stig Bjørlykke <stig@bjorlykke.org> | 2018-09-21 14:09:58 +0200 |
---|---|---|
committer | Stig Bjørlykke <stig@bjorlykke.org> | 2018-09-21 12:52:08 +0000 |
commit | 001a81bace06c8f3c92f0a2c375746312f88dd25 (patch) | |
tree | cb67620c75b745bb860934b8486c6e2a38e4ece8 /epan/dissectors/packet-ieee802154.c | |
parent | 993800b8b97c19e7b2194983a1f97f4884979ec3 (diff) |
ieee802154: Make a variable volatile
This fixes a warning:
[...]/epan/dissectors/packet-ieee802154.c:3196:17: error: variable ‘length’ might
be clobbered by ‘longjmp’ or ‘vfork’ [-Werror=clobbered]
guint16 length = (guint16) (ie_header & IEEE802154_PAYLOAD_IE_LENGTH_MASK);
Change-Id: I556b2199f8ab028e87e4c9d868da9831f7076402
Reviewed-on: https://code.wireshark.org/review/29779
Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
Diffstat (limited to 'epan/dissectors/packet-ieee802154.c')
-rw-r--r-- | epan/dissectors/packet-ieee802154.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ieee802154.c b/epan/dissectors/packet-ieee802154.c index 0fc6b97662..6fdfd1d924 100644 --- a/epan/dissectors/packet-ieee802154.c +++ b/epan/dissectors/packet-ieee802154.c @@ -3193,7 +3193,7 @@ dissect_ieee802154_payload_ie(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree volatile int consumed = 0; guint16 ie_header = tvb_get_letohs(tvb, offset); guint16 id = (guint16) ((ie_header & IEEE802154_PAYLOAD_IE_ID_MASK) >> 11); - guint16 length = (guint16) (ie_header & IEEE802154_PAYLOAD_IE_LENGTH_MASK); + volatile guint16 length = (guint16) (ie_header & IEEE802154_PAYLOAD_IE_LENGTH_MASK); tvbuff_t *ie_tvb = tvb_new_subset_length(tvb, offset, 2 + length); if (id == IEEE802154_PAYLOAD_IE_TERMINATION) { |