diff options
| author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2014-10-09 17:46:55 +0000 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2014-10-09 15:47:29 -0700 |
| commit | 29a5b29f09b06205a4b5f2d3701cbd18a70ae226 (patch) | |
| tree | 40a3b6dacf298973d367ecf2535e9acf6c0c2eb7 /datapath-windows/ovsext/Vport.c | |
| parent | 9930aae56893b87b90b32e3a444e73b715167bf9 (diff) | |
| download | openvswitch-29a5b29f09b06205a4b5f2d3701cbd18a70ae226.tar.gz | |
datapath-windows: Define OVS_DPPORT_NUMBER_INVALID
The fields externalVport and internalVport of the OVS_SWITCH_CONTEXT
struct are currently defined as PVOID. However, all over the code they
are used as POVS_VPORT_ENTRY. In order to improve clarity and reduce the
need for useless casts to POVS_VPORT_ENTRY, this patch changes the type
from PVOID to POVS_VPORT_ENTRY.
This patch does not cleanup the code that already uses casts to
POVS_VPORT_ENTRY. This cleanup can be done later on as well.
Signed-off-by: Samuel Ghinet <sghinet@cloudbasesolutions.com>
Co-authored-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Acked-by: Ankur Sharma <ankursharma@vmware.com>
Acked-by: Eitan Eliahu <eliahue@vmware.com>
Acked-by: Nithin Raju <nithin@vmware.com>
Tested-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'datapath-windows/ovsext/Vport.c')
| -rw-r--r-- | datapath-windows/ovsext/Vport.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c index 942bad00d..98ec439d4 100644 --- a/datapath-windows/ovsext/Vport.c +++ b/datapath-windows/ovsext/Vport.c @@ -718,7 +718,7 @@ POVS_VPORT_ENTRY vport) vport->nicIndex != 0) { vport->portNo = OvsGetVportNo(switchContext, vport->nicIndex, vport->ovsType, vport->portType == NdisSwitchPortTypeExternal); - if (vport->portNo == 0) { + if (vport->portNo == OVS_DPPORT_NUMBER_INVALID) { return NDIS_STATUS_RESOURCES; } ASSERT(OVS_IS_VPORT_ENTRY_NULL(switchContext, |