From 31a4cdaa173659023bd0c46ca0da7620110d8a16 Mon Sep 17 00:00:00 2001
From: Divakar <ddivakar@juniper.net>
Date: Tue, 19 May 2015 21:46:58 -0700
Subject: [PATCH] Bum traffic handling in transparent service chain

When the BUM traffic is received on transparent service chain
interface, it has to be dropped to avoid the loops in that
service chain network.
Checking-in on behalf of  Anand

Change-Id: Ic1b1a2f2c5716e2852582012963e6ee9934e9a1a
closes-bug: #1449166
---
 dp-core/vr_datapath.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/dp-core/vr_datapath.c b/dp-core/vr_datapath.c
index 355f9b54d..03eaeff0e 100644
--- a/dp-core/vr_datapath.c
+++ b/dp-core/vr_datapath.c
@@ -495,6 +495,18 @@ vr_virtual_input(unsigned short vrf, struct vr_interface *vif,
         return 0;
     }
 
+    /*
+     * we really do not allow any broadcast packets from interfaces
+     * that are part of transparent service chain, since transparent
+     * service chain bridges packets across vrf (and hence loops can
+     * happen)
+     */
+    if ((pkt->vp_flags & VP_FLAG_MULTICAST) &&
+            (vif_is_service(vif))) {
+        vif_drop_pkt(vif, pkt, 1);
+        return 0;
+    }
+
     if (!vr_flow_forward(pkt->vp_if->vif_router, pkt, &fmd))
         return 0;