Re: BUG: AddressSanitizer reports a buffer-overflow on rte_hash_lookup

2023-02-05 Thread Stephen Hemminger
On Sun, 5 Feb 2023 22:14:28 +0200 Isaac Boukris wrote: > > > > This code is using the common optimization of doing a full 32 bit access > > and masking the result. This will read past the end of the passed input > > but ignore the extra bytes. It won't be a problem unless the application > > goes

Re: BUG: AddressSanitizer reports a buffer-overflow on rte_hash_lookup

2023-02-05 Thread Isaac Boukris
On Sun, Feb 5, 2023 at 9:49 PM Stephen Hemminger wrote: > > On Sun, 5 Feb 2023 18:54:20 +0200 > Isaac Boukris wrote: > > > Hi, > > > > I managed to reproduce it by modifying the helloworld app (see > > attached). The report seem correct, as in case of 10 byte key the code > > tries to look at the

Re: BUG: AddressSanitizer reports a buffer-overflow on rte_hash_lookup

2023-02-05 Thread Stephen Hemminger
On Sun, 5 Feb 2023 18:54:20 +0200 Isaac Boukris wrote: > Hi, > > I managed to reproduce it by modifying the helloworld app (see > attached). The report seem correct, as in case of 10 byte key the code > tries to look at the key as uint32 array and access k[2] which is two > bytes over, see: > ht

BUG: AddressSanitizer reports a buffer-overflow on rte_hash_lookup

2023-02-05 Thread Isaac Boukris
Hi, I managed to reproduce it by modifying the helloworld app (see attached). The report seem correct, as in case of 10 byte key the code tries to look at the key as uint32 array and access k[2] which is two bytes over, see: https://github.com/DPDK/dpdk/blob/0bf583971a0154c9150d4a7a4b82ecbc9dd