diff options
author | Gerald Combs <gerald@wireshark.org> | 2015-11-30 14:11:22 -0800 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2015-12-01 01:19:33 +0000 |
commit | b8f90de70efa2d271274fbb48df12737f6eddd12 (patch) | |
tree | 25561156e54bf3185e9d5285b18940761c18cc52 /caputils/ws80211_utils.c | |
parent | d487c8c7e72666d5e926c4c6a499a0821796c694 (diff) |
ws80211: Disable shorten-64-to-32.
Disable shorten-64-to-32 in ws80211_create_on_demand_interface,
which calls NLA_PUT_STRING, which passes the output of strlen to an
int parameter. NLA_PUT_STRING is defined in netlink/attr.h so there's
not much we can do to fix it directly.
Suppress -Wpragmas before suppressing warnings in gcc so that we can
use DIAG_OFF with clang-only warnings.
Change-Id: I1180950edd93c056b8fbfbed164e482024aee90a
Reviewed-on: https://code.wireshark.org/review/12314
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'caputils/ws80211_utils.c')
-rw-r--r-- | caputils/ws80211_utils.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/caputils/ws80211_utils.c b/caputils/ws80211_utils.c index 2db292bf9d..a83f5365a2 100644 --- a/caputils/ws80211_utils.c +++ b/caputils/ws80211_utils.c @@ -680,6 +680,8 @@ out_err: return -1; } +/* Needed for NLA_PUT_STRING, which passes strlen as an int */ +DIAG_OFF(shorten-64-to-32) static int ws80211_create_on_demand_interface(const char *name) { int devidx, phyidx, err; @@ -716,6 +718,7 @@ nla_put_failure: fprintf(stderr, "building message failed\n"); return 2; } +DIAG_ON(shorten-64-to-32) int ws80211_set_freq(const char *name, int freq, int chan_type) { |