diff options
author | Ben Pfaff <blp@nicira.com> | 2014-03-01 17:11:02 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2014-03-01 17:30:32 -0800 |
commit | 27a8cbbc8e09e66cf4097c40757e797effbdd20c (patch) | |
tree | b665751a5881df4561b7e7680f5bcf99acf30854 /ofproto/tunnel.c | |
parent | 77587e0fef5818b5a873932c573781a14eb9edfb (diff) | |
download | openvswitch-27a8cbbc8e09e66cf4097c40757e797effbdd20c.tar.gz |
odp-util: Include tun_id when nonzero even if "key" flag not set.
When a flow_tnl is being translated to Netlink attributes, the tun_id field
was included only if the FLOW_TNL_F_KEY flag was set. This meant that for
a mask, where one would not necessarily expect that flag to be set even if
there were a key, the tun_id could be omitted even if it were nonzero.
This led to kernel flows that did not match on a field that was required
to be matched (possibly causing incorrect treatment of packets) and
premature deletion of kernel flows due to mask mismatch. This commit
fixes the problem.
Bug #1192516.
Reported-by: Krishna Miriyala <miriyalak@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'ofproto/tunnel.c')
0 files changed, 0 insertions, 0 deletions