| commit | eb148ea4503fad0c7feb770561366a3c8f413bfb | [log] [tgz] |
|---|---|---|
| author | Sangho Shin <sangho@onlab.us> | Fri Oct 24 12:44:15 2014 -0700 |
| committer | Sangho Shin <sangho@onlab.us> | Fri Oct 24 12:44:15 2014 -0700 |
| tree | 4256f9f42351afaee8e3c59da1b4d8789bd5f93c | |
| parent | 6232558230a9881a16c2bc5bb8dd3caabcbf4a65 [diff] |
Fix the bug of creating an empty stitching rule when the label stack of the previous sub tunnel is full and there is no more labels to put. Change-Id: I29318ec064e8c619ae900b1ecfb5b0390280d33d
diff --git a/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java b/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java index 90af7cc..492dc33 100644 --- a/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java +++ b/src/main/java/net/onrc/onos/apps/segmentrouting/SegmentRoutingManager.java
@@ -1570,9 +1570,10 @@ } - if (i < MAX_NUM_LABELS+1) { + if (i < MAX_NUM_LABELS+1 && (routeInfo.getFwdSwDpid() != null && + !routeInfo.getFwdSwDpid().isEmpty())) { rules.add(routeInfo); - // NOTE: empty label stack can happen + // NOTE: empty label stack can happen, but forwarding destination should be set } return rules;