Page MenuHomeFreeBSD

ifconfig: fix comparison in printmimo()
ClosedPublic

Authored by melifaro on May 27 2023, 12:40 PM.
Tags
None
Referenced Files
F103747811: D40301.diff
Thu, Nov 28, 9:34 PM
Unknown Object (File)
Oct 29 2024, 4:47 AM
Unknown Object (File)
Oct 22 2024, 11:11 PM
Unknown Object (File)
Oct 3 2024, 10:09 PM
Unknown Object (File)
Oct 3 2024, 7:22 AM
Unknown Object (File)
Oct 2 2024, 8:00 PM
Unknown Object (File)
Oct 1 2024, 8:09 AM
Unknown Object (File)
Sep 30 2024, 7:25 AM
Subscribers

Details

Summary

Currently compiler yells at printmimo() with the following warning:

/usr/home/melifaro/free/head/sbin/ifconfig/ifieee80211.c:3716:17: error: comparison of array 'mi->ch[i].rssi' not equal to a null pointer is always true [-Werror,-Wtautological-pointer-compare]
                if (mi->ch[i].rssi != 0) {
                    ~~~~~~~~~~^~~~    ~

To check:

--- a/sbin/ifconfig/Makefile
+++ b/sbin/ifconfig/Makefile
@@ -80,7 +80,8 @@ CFLAGS+=-DWITHOUT_NETLINK
 MAN=   ifconfig.8

 CFLAGS+= -Wall -Wmissing-prototypes -Wcast-qual -Wwrite-strings -Wnested-externs
-WARNS?=        2
+CFLAGS+=-Wno-error=unused-parameter -Wno-error=shadow
+#WARNS?=       2

I have no domain knowledge whether the fix is valid or not, but the code before 617f8b101e7f checked rssi[0].

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 51734
Build 48625: arc lint + arc unit