Acked-by: Sairam Venugopal <vsai...@vmware.com>
On 9/3/15, 7:01 AM, "Nithin Raju" <nit...@vmware.com> wrote:
>OVS_VPORT_ATTR_OPTIONS being an optional attribute should be
>preceded by a check for valid value before access.
>
>Signed-off-by: Nithin Raju <nit...@vmware.com>
>---
> datapath-windows/ovsext/Vport.c | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/datapath-windows/ovsext/Vport.c
>b/datapath-windows/ovsext/Vport.c
>index 6b74ec2..dd615e4 100644
>--- a/datapath-windows/ovsext/Vport.c
>+++ b/datapath-windows/ovsext/Vport.c
>@@ -2177,10 +2177,12 @@ OvsNewVportCmdHandler(POVS_USER_PARAMS_CONTEXT
>usrParamsCtx,
> break;
> }
>
>- PNL_ATTR attr =
>NlAttrFindNested(vportAttrs[OVS_VPORT_ATTR_OPTIONS],
>- OVS_TUNNEL_ATTR_DST_PORT);
>- if (attr) {
>- transportPortDest = NlAttrGetU16(attr);
>+ if (vportAttrs[OVS_VPORT_ATTR_OPTIONS]) {
>+ PNL_ATTR attr =
>NlAttrFindNested(vportAttrs[OVS_VPORT_ATTR_OPTIONS],
>+
>OVS_TUNNEL_ATTR_DST_PORT);
>+ if (attr) {
>+ transportPortDest = NlAttrGetU16(attr);
>+ }
> }
>
> status = OvsInitTunnelVport(usrParamsCtx,
>--
>1.8.5.6
>
>_______________________________________________
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Dc
>ruz40PROJ40ROzSpxyQSLw6fcrOWpJgEcEmNR3JEQ&m=M6HsLmOuYm_fYoAFouyGOA8j2eLADC
>Xwt6z5nSZ_nig&s=OhvXMw1uly8zT4V_RJmeHY9zTieLgVoooGIzOIN7nRU&e=
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
