Committed by
Gerrit Code Review
Add missing fail() invocation
Added missing fail() call-out in the versatile flow objective handler in OFDPA2Pipeline.java to cover the case when neither a nextId nor a treatment is provided. Change-Id: Ic73e614c141281ff0014a8055d35183b7c3abd4e
Showing
1 changed file
with
1 additions
and
0 deletions
... | @@ -645,6 +645,7 @@ public class OFDPA2Pipeline extends AbstractHandlerBehaviour implements Pipeline | ... | @@ -645,6 +645,7 @@ public class OFDPA2Pipeline extends AbstractHandlerBehaviour implements Pipeline |
645 | if (fwd.nextId() == null && fwd.treatment() == null) { | 645 | if (fwd.nextId() == null && fwd.treatment() == null) { |
646 | log.error("Forwarding objective {} from {} must contain " | 646 | log.error("Forwarding objective {} from {} must contain " |
647 | + "nextId or Treatment", fwd.selector(), fwd.appId()); | 647 | + "nextId or Treatment", fwd.selector(), fwd.appId()); |
648 | + fail(fwd, ObjectiveError.BADPARAMS); | ||
648 | return Collections.emptySet(); | 649 | return Collections.emptySet(); |
649 | } | 650 | } |
650 | 651 | ... | ... |
-
Please register or login to post a comment