interop fix

Signed-off-by: Serguei Bezverkhi <sbezverk@cisco.com>

1 files changed, 2 insertions, 11 deletions

diff --git a/pkg/packet/bgp/bgp.go b/pkg/packet/bgp/bgp.go
index 63082af1..406d024c 100644
--- a/pkg/packet/bgp/bgp.go
+++ b/pkg/packet/bgp/bgp.go
@@ -1621,10 +1621,8 @@ func (l *MPLSLabelStack) DecodeFromBytes(data []byte, options ...*MarshallingOpt
 	foundBottom := false
 	bottomExpected := true
 	if IsAttributePresent(BGP_ATTR_TYPE_PREFIX_SID, options) {
-		// If Update carries Prefix SID attribute then there is no a label stack,
-		// but just 3 bytes which are used to carry the lower portion of Prefix SID.
-		// There is no bottom stack indication in this case. Once 3 bytes are stored
-		// breaking out of the loop.
+		// If Update carries Prefix SID attribute then one should not rely on BoS for label stack processing,
+		// the first label carries transposed variable part of the SRv6 SID.
 		bottomExpected = false
 	}
 	for len(data) >= 3 {
@@ -1665,13 +1663,6 @@ func (l *MPLSLabelStack) Serialize(options ...*MarshallingOption) ([]byte, error
 		buf[i*3+1] = byte((label >> 8) & 0xff)
 		buf[i*3+2] = byte(label & 0xff)
 	}
-	if IsAttributePresent(BGP_ATTR_TYPE_PREFIX_SID, options) {
-		// If Update carries Prefix SID attribute then there is no a label stack,
-		// but just 3 bytes which are used to carry the lower portion of Prefix SID.
-		// No need BoS bit set
-		return buf, nil
-	}
-
 	buf[len(buf)-1] |= 1
 	return buf, nil
 }