forked from torvalds/linux
-
Notifications
You must be signed in to change notification settings - Fork 16
Enable -Wuninitialized #381
Copy link
Copy link
Closed
Labels
-Wsometimes-uninitialized-Wuninitialized[BUG] linuxA bug that should be fixed in the mainline kernel.A bug that should be fixed in the mainline kernel.[FIXED][LINUX] 5.3This bug was fixed in Linux 5.3This bug was fixed in Linux 5.3
Metadata
Metadata
Assignees
Labels
-Wsometimes-uninitialized-Wuninitialized[BUG] linuxA bug that should be fixed in the mainline kernel.A bug that should be fixed in the mainline kernel.[FIXED][LINUX] 5.3This bug was fixed in Linux 5.3This bug was fixed in Linux 5.3
29b00e6 would have been caught with Clang in its default configuration (and not by GCC due to a bug), as pointed out in this thread: https://lore.kernel.org/lkml/[email protected]/
However,
-Wuninitializedalso disables-Wsometimes-uninitializedso this was missed by both compilers :(I am going to test the following diff then send it along right away if there are no additional warnings:
Side note, looks like this might be a good time to tell Linus about the KernelCI work that is going on: https://lore.kernel.org/lkml/CAHk-=wggjLsi-1BmDHqWAJPzBvTD_-MQNo5qQ9WCuncnyWPROg@mail.gmail.com/