I'd like to keep the explicit check here, but since underlying type of
enum is implementation defined, theres some inconsistency using -Wall
-Werror in older compilers here
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
int vrf_configure_backend(enum vrf_backend_type backend)
{
- if (backend < 0 || backend >= VRF_BACKEND_MAX)
+ /* Work around issue in old gcc */
+ switch (backend) {
+ case VRF_BACKEND_UNKNOWN:
+ case VRF_BACKEND_NETNS:
+ case VRF_BACKEND_VRF_LITE:
+ break;
+ default:
return -1;
+ }
vrf_backend = backend;
vrf_backend_configured = 1;