On 10/7/25 5:16 PM, Mike Pattrick via dev wrote:
> Currently while handling the 24 bit VNI field in VXLAN code,
> netdev-offload-dpdk will actually read and write 32 bits. The byte that
> is overwritten is reserved and supposed to be set to zero anyways, so
> this is mostly harmless.
> 
> However, Openscanhub correctly identified this as a buffer overrun. Now
> use ovs_16aligned_be32 to access the field as a 32bit integer.
> 
> Reported-at: https://issues.redhat.com/browse/FDP-1122
> Signed-off-by: Mike Pattrick <[email protected]>
> ---
> v3:
>  - Change from memcpy to ovs_16aligned_be32
> v4:
>  - Correct outdated commit message
> v5:
>  - Cast from hdr member
> ---
>  lib/netdev-offload-dpdk.c | 17 ++++++++---------
>  1 file changed, 8 insertions(+), 9 deletions(-)

Thanks, Mike!  Applied and backported down to 3.3.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to