Validation Issue with DlSrc,DlDst MacAddress 46/3546/1
authorGaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Sat, 7 Dec 2013 11:34:16 +0000 (17:04 +0530)
committerGaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Sat, 7 Dec 2013 11:44:00 +0000 (17:14 +0530)
Signed-off-by: Gaurav Bhagwani <gaurav.bhagwani@ericsson.com>
Change-Id: Ib02ea2488a290f399e666122821e84dc6364eca5

opendaylight/md-sal/forwardingrules-manager/src/main/java/org/opendaylight/controller/forwardingrulesmanager/consumer/impl/FRMUtil.java

index 24bfc4fdf4c09079230189c43e9d33a428ab0cbf..84b3e53e065cfea03f225cc1c6fe0f8ebaa27582 100644 (file)
@@ -260,13 +260,13 @@ public class FRMUtil {
                 }
             } else if (action instanceof SetDlDstAction) {
                 MacAddress address = ((SetDlDstAction) action).getAddress();
                 }
             } else if (action instanceof SetDlDstAction) {
                 MacAddress address = ((SetDlDstAction) action).getAddress();
-                if (address != null && !isL2AddressValid(address.toString())) {
+                if (address != null && !isL2AddressValid(address.getValue())) {
                     logger.error("SetDlDstAction: Address not valid");
                     return false;
                 }
             } else if (action instanceof SetDlSrcAction) {
                 MacAddress address = ((SetDlSrcAction) action).getAddress();
                     logger.error("SetDlDstAction: Address not valid");
                     return false;
                 }
             } else if (action instanceof SetDlSrcAction) {
                 MacAddress address = ((SetDlSrcAction) action).getAddress();
-                if (address != null && !isL2AddressValid(address.toString())) {
+                if (address != null && !isL2AddressValid(address.getValue())) {
                     logger.error("SetDlSrcAction: Address not valid");
                     return false;
                 }
                     logger.error("SetDlSrcAction: Address not valid");
                     return false;
                 }