> -----Original Message----- > From: David Marchand <[email protected]> > Sent: Wednesday, October 20, 2021 12:54 PM > To: Wang, Yipeng1 <[email protected]>; Gobriel, Sameh > <[email protected]>; Richardson, Bruce > <[email protected]> > Cc: dev <[email protected]>; dpdk stable <[email protected]>; Medvedkin, > Vladimir <[email protected]> > Subject: Re: [PATCH v3] test/hash: fix buffer overflow > > On Tue, Oct 19, 2021 at 5:58 PM Medvedkin, Vladimir > <[email protected]> wrote: > > > I am not against the fix, but it seems to test something different > > > than what an application using the hash library would do. > > > Or if an application directly calls this hash function, maybe the > > > unit test should not test it via rte_hash_create (which seems to > > > defeat the abstraction). > > > > > > > I'd say that user should not use this hash function with rte_hash. > > Yipeng, Sameh, Bruce, > > what do you think? > > Guys, can we conclude? > Thanks. > > > -- > David Marchand [Wang, Yipeng]
Vladimir, Bruce and I discussed a bit offline and I think we agreed on this fix. But we should be more clear in the comment of rte_hash_create about what hash function the API expected (I.e. key length is byte count). Acked-by: Yipeng Wang <[email protected]>
