[ovs-dev] [PATCH] lib/flow.c: Don't include ports of first fragments in hash

[Ben Pfaff]

On Wed, Jun 05, 2019 at 10:36:33PM +0000, Van Bemmel, Jeroen (Nokia - US) wrote:
> For a series of IP fragments, only the first packet includes the transport
> header (TCP/UDP/SCTP) and the src/dst ports. By including these port
> numbers in the hash, it may happen that a first fragment hashes to a
> different value than subsequent packets, causing different packets from
> the same flow to follow different paths. This in turn may result in
> out-of-order delivery or failed reassembly. This patch excludes port
> numbers from the hash calculation in case of IP fragmentation.
> 
> Signed-off-by: Jeroen van Bemmel <jeroen.van_bemmel at nokia.com>

I applied this to master, thanks!