commit 64ae77890c996d03458f0efe084ebb3f69f7e9ff
author Jakub Grajciar <Jakub.Grajciar@pantheon.tech> Tue Nov 14 13:45:04 2017 +0100
committer Damjan Marion <dmarion.lists@gmail.com> Tue Nov 14 16:13:40 2017 +0000
tree 77d13f07a1ac9af571a3b6837e26649d0f0e2fc9
parent fa600c9169c0d7104af7a9be12a0471a8a8c8262

vnet: af_packet_set_l4_cksum_offload device class check

Change-Id: Ie07b71977c46d2f1e030799a08cc5af0fdc397aa
Signed-off-by: Jakub Grajciar <Jakub.Grajciar@pantheon.tech>

src/vnet/devices/af_packet/af_packet.c

diff --git a/src/vnet/devices/af_packet/af_packet.c b/src/vnet/devices/af_packet/af_packet.c
index d0a241e..eec3e4b 100644
--- a/src/vnet/devices/af_packet/af_packet.c
+++ b/src/vnet/devices/af_packet/af_packet.c
@@ -416,6 +416,9 @@
 
   hw = vnet_get_sup_hw_interface (vnm, sw_if_index);
 
+  if (hw->dev_class_index != af_packet_device_class.index)
+    return VNET_API_ERROR_INVALID_INTERFACE;
+
   if (set)
     hw->flags &= ~VNET_HW_INTERFACE_FLAG_SUPPORTS_TX_L4_CKSUM_OFFLOAD;
   else

src/vnet/devices/af_packet/cli.c

diff --git a/src/vnet/devices/af_packet/cli.c b/src/vnet/devices/af_packet/cli.c
index 29c0e06..faa0728 100644
--- a/src/vnet/devices/af_packet/cli.c
+++ b/src/vnet/devices/af_packet/cli.c
@@ -225,7 +225,8 @@
 	}
     }
 
-  af_packet_set_l4_cksum_offload (vm, sw_if_index, set);
+  if (af_packet_set_l4_cksum_offload (vm, sw_if_index, set) < 0)
+    error = clib_error_return (0, "not an af_packet interface");
 
 done:
   unformat_free (line_input);