diff options
| author | Nithin Raju <nithin@vmware.com> | 2017-06-14 16:01:48 -0700 |
|---|---|---|
| committer | Gurucharan Shetty <guru@ovn.org> | 2017-06-16 09:38:18 -0700 |
| commit | e8808c2249b9d31fc5283f9b69466603ad6c4866 (patch) | |
| tree | 7de0d67437c04a8ca8230ac97f377e711881ec4c /datapath-windows/ovsext/Conntrack.c | |
| parent | cfa98e3878026dd6b206fe1263505e8594846f93 (diff) | |
| download | openvswitch-e8808c2249b9d31fc5283f9b69466603ad6c4866.tar.gz | |
datapath-windows: use NlAttrGet() in Conntrack.c
Couple of minor fixes that got flagged with a static checker.
Signed-off-by: Nithin Raju <nithin@vmware.com>
Acked-by: Sairam Venugopal <vsairam@vmware.com>
Acked-by: Shashank Ram <rams@vmware.com>
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Diffstat (limited to 'datapath-windows/ovsext/Conntrack.c')
| -rw-r--r-- | datapath-windows/ovsext/Conntrack.c | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/datapath-windows/ovsext/Conntrack.c b/datapath-windows/ovsext/Conntrack.c index 68ed39566..07a958371 100644 --- a/datapath-windows/ovsext/Conntrack.c +++ b/datapath-windows/ovsext/Conntrack.c @@ -863,23 +863,13 @@ OvsExecuteConntrackAction(OvsForwardingContext *fwdCtx, ? NAT_ACTION_SRC : NAT_ACTION_DST); break; case OVS_NAT_ATTR_IP_MIN: - if (natAttr->nlaLen < NLA_HDRLEN) { - OVS_LOG_ERROR("Incorrect header length for " - "OVS_NAT_ATTR_IP_MIN message."); - break; - } memcpy(&natActionInfo.minAddr, - NlAttrData(natAttr), natAttr->nlaLen - NLA_HDRLEN); + NlAttrData(natAttr), NlAttrGetSize(natAttr)); hasMinIp = TRUE; break; case OVS_NAT_ATTR_IP_MAX: - if (natAttr->nlaLen < NLA_HDRLEN) { - OVS_LOG_ERROR("Incorrect header length for " - "OVS_NAT_ATTR_IP_MAX message."); - break; - } memcpy(&natActionInfo.maxAddr, - NlAttrData(natAttr), natAttr->nlaLen - NLA_HDRLEN); + NlAttrData(natAttr), NlAttrGetSize(natAttr)); hasMaxIp = TRUE; break; case OVS_NAT_ATTR_PROTO_MIN: |