We really must handle all possible enumeration values in
authorflorian <florian@openbsd.org>
Mon, 1 Mar 2021 15:56:00 +0000 (15:56 +0000)
committerflorian <florian@openbsd.org>
Mon, 1 Mar 2021 15:56:00 +0000 (15:56 +0000)
commit4b15555f05032b2b476611974f4a51507844874f
tree240225b1d9c937d1f584f84c72848d051c4898d9
parentb7f83d9ab1c58fda54411df2211233004aec6b71
We really must handle all possible enumeration values in
state_transition() and iface_timeout(). Let the compiler help us by
emitting a warning when we missed one (-Wswitch).
Reminded by jsg who pointed out that gcc is quite confused and thinks
there is an out of bounds access in if_state_name[] in the default
case. There is not, if_state_name[] and enum if_state have to be kept
in sync.
(Note that -Wswitch is not a silver bullet, it just happens to work
here.)
sbin/dhcpleased/engine.c