From: Daisuke Kotani <kotani@net.ist.i.kyoto-u.ac.jp>
Date: Mon, 23 Dec 2013 09:19:48 +0000 (+0900)
Subject: ofp-parse: Check port number only after parsing it in parse_output().
X-Git-Tag: v2.1.0~54
X-Git-Url: http://git.cascardo.eti.br/?p=cascardo%2Fovs.git;a=commitdiff_plain;h=f0e047db8c208d85f97d2674bd2e6b2b30342c09

ofp-parse: Check port number only after parsing it in parse_output().

This patch allows to set max_len to UINT16_MAX in parse_output
if output port is OFPP_CONTROLLER.

Signed-off-by: Daisuke Kotani <kotani@net.ist.i.kyoto-u.ac.jp>
Signed-off-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/lib/ofp-parse.c b/lib/ofp-parse.c
index 6b69ecd8e..251adfac8 100644
--- a/lib/ofp-parse.c
+++ b/lib/ofp-parse.c
@@ -212,10 +212,10 @@ parse_output(const char *arg, struct ofpbuf *ofpacts)
         struct ofpact_output *output;
 
         output = ofpact_put_OUTPUT(ofpacts);
-        output->max_len = output->port == OFPP_CONTROLLER ? UINT16_MAX : 0;
         if (!ofputil_port_from_string(arg, &output->port)) {
             return xasprintf("%s: output to unknown port", arg);
         }
+        output->max_len = output->port == OFPP_CONTROLLER ? UINT16_MAX : 0;
         return NULL;
     }
 }