diff options
| author | Dimitry Andric <dim@FreeBSD.org> | 2017-01-07 15:57:12 +0000 |
|---|---|---|
| committer | Dimitry Andric <dim@FreeBSD.org> | 2017-01-07 15:57:12 +0000 |
| commit | 00742f6ced54eb680e7cd1dffc7c73e703958ca6 (patch) | |
| tree | e5162168b3f59f64344ec009f197048f08334abb /contrib/ngatm/snmp_atm | |
| parent | d4cd50b494d679d68851a8a02e22cd9a7cd11f7b (diff) | |
Fix the following clang 4.0.0 warning in ngatm's snmp_atm.c:
contrib/ngatm/snmp_atm/snmp_atm.c:173:6: error: logical not is only
applied to the left hand side of this bitwise operator
[-Werror,-Wlogical-not-parentheses]
if (!ifmr.ifm_status & IFM_AVALID) {
^ ~
Obviously, the masking needs to be done before the logical not
operation. Add parentheses to make it so.
MFC after: 3 days
Notes
Notes:
svn path=/head/; revision=311649
Diffstat (limited to 'contrib/ngatm/snmp_atm')
| -rw-r--r-- | contrib/ngatm/snmp_atm/snmp_atm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/ngatm/snmp_atm/snmp_atm.c b/contrib/ngatm/snmp_atm/snmp_atm.c index c922c6f3dd7a..a2590aeb4dd4 100644 --- a/contrib/ngatm/snmp_atm/snmp_atm.c +++ b/contrib/ngatm/snmp_atm/snmp_atm.c @@ -170,7 +170,7 @@ atmif_check_carrier(struct atmif_priv *aif) aif->pub.carrier = ATMIF_CARRIER_UNKNOWN; return; } - if (!ifmr.ifm_status & IFM_AVALID) { + if (!(ifmr.ifm_status & IFM_AVALID)) { aif->pub.carrier = ATMIF_CARRIER_UNKNOWN; return; } |