diff options
| author | Jarno Rajahalme <jrajahalme@nicira.com> | 2015-06-09 15:24:33 -0700 |
|---|---|---|
| committer | Jarno Rajahalme <jrajahalme@nicira.com> | 2015-06-09 17:08:26 -0700 |
| commit | 19ba142220fb109583c181039fb37d8f7aa6ff67 (patch) | |
| tree | 9d8ebe9102a65ce2d8377df6d6b11b1a3e659f11 /lib/ofp-parse.c | |
| parent | 3139b8e909d4fdd0a32fa8e3c556966bc4f63ddc (diff) | |
| download | openvswitch-19ba142220fb109583c181039fb37d8f7aa6ff67.tar.gz | |
ofp-parse: Use F_OUT_PORT when parsing.
We set this field flag for the cases when an out_port should be
parsed, but failed to make use of it.
Two test cases needed to be updated due to use of out_port in flow
add, while out_port is legal for flow deletes only.
Suggested-by: Ben Pfaff <blp@nicira.com>
Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/ofp-parse.c')
| -rw-r--r-- | lib/ofp-parse.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ofp-parse.c b/lib/ofp-parse.c index 856044dbb..6125f27b6 100644 --- a/lib/ofp-parse.c +++ b/lib/ofp-parse.c @@ -353,7 +353,7 @@ parse_ofp_str__(struct ofputil_flow_mod *fm, int command, char *string, if (fm->table_id != 0xff) { *usable_protocols &= OFPUTIL_P_TID; } - } else if (!strcmp(name, "out_port")) { + } else if (fields & F_OUT_PORT && !strcmp(name, "out_port")) { if (!ofputil_port_from_string(value, &fm->out_port)) { error = xasprintf("%s is not a valid OpenFlow port", value); |