tunnel: Do not set padding bits in tunnel mask.
authorBen Pfaff <blp@nicira.com>
Sun, 2 Mar 2014 01:15:00 +0000 (17:15 -0800)
committerBen Pfaff <blp@nicira.com>
Sun, 2 Mar 2014 01:39:24 +0000 (17:39 -0800)
commit42e4cc97a95f2d16dbee11a041dc74b0f1178f7d
tree83cdaa7b3cc714e191025faaef3a8b35dbe79627
parent37dfcd695e5e29e1d6051b0d53385fd7ae686bc5
tunnel: Do not set padding bits in tunnel mask.

On most architectures other than 32-bit x86, struct flow_tnl ends with 4
padding bytes.  Until now, tnl_xlate_init() set those bytes to nonzero
values in the wildcard mask.  When the wildcard mask passed through Netlink
attributes and back to userspace, the padding bytes of course became zero
again, which caused a wildcard mask mismatch and premature deletion of the
flow in revalidation.  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>
ofproto/tunnel.c