Ensuring that SET_DL_* action with openflowj only uses OFP_ETH_ALEN

bytes, just in case SAL passes more than that.

Signed-off-by: Srini Seetharaman <srini.seetharaman@gmail.com>

diff --git a/third-party/openflowj/src/main/java/org/openflow/protocol/action/OFActionDataLayer.java b/third-party/openflowj/src/main/java/org/openflow/protocol/action/OFActionDataLayer.java
index 79edf904f176a6f1c533c1e88f82bee2ff814a92..f637b21e1e8f74975858625ce20e20023a03fc26 100644 (file)
--- a/third-party/openflowj/src/main/java/org/openflow/protocol/action/OFActionDataLayer.java
+++ b/third-party/openflowj/src/main/java/org/openflow/protocol/action/OFActionDataLayer.java
@@ -44,7 +44,7 @@ public abstract class OFActionDataLayer extends OFAction {
     @Override
     public void writeTo(ByteBuffer data) {
         super.writeTo(data);
     @Override
     public void writeTo(ByteBuffer data) {
         super.writeTo(data);

-        data.put(this.dataLayerAddress);
+        data.put(this.dataLayerAddress, 0, OFPhysicalPort.OFP_ETH_ALEN);

         data.putInt(0);
         data.putShort((short) 0);
     }
         data.putInt(0);
         data.putShort((short) 0);
     }


@@ -74,4 +74,4 @@ public abstract class OFActionDataLayer extends OFAction {
         }
         return true;
     }
         }
         return true;
     }

-}
\ No newline at end of file
+}