Re: [PATCH] examples/ipsec-secgw: fix partial overflow

[Thomas Monjalon <thomas@monjalon.net>]

> > Case of partial overflow detected with ASan. Added extra padding to cdev_key
> > structure.
> > 
> > This structure is used for the key in hash table.
> > Padding is added to force the struct to use 8 bytes, to ensure memory is
> > notread past this structs boundary (the hash key calculation reads 8 bytes if
> > this struct is size 5 bytes).
> > The padding should be zeroed.
> > If fields are modified in this struct, the padding must be updated to ensure
> > multiple of 8 bytes size overall.
> > 
> > Fixes: d299106e8e31 ("examples/ipsec-secgw: add IPsec sample application")
> > Cc: sergio.gonzalez.monroy@intel.com
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: Brian Dooley <brian.dooley@intel.com>
> 
> Acked-by: Ciara Power <ciara.power@intel.com>

Applied and made the comment simpler with this:

  uint8_t padding[3]; /* padding to 8-byte size should be zeroed */