On Wed, Nov 21, 2018 at 08:51:56PM -0600, Honnappa Nagarahalli wrote: > Add a debug check for out-of-bound write while freeing the key slot. > > Coverity issue: 325733 > Fixes: e605a1d36ca7 ("hash: add lock-free r/w concurrency") > Cc: honnappa.nagaraha...@arm.com > > Signed-off-by: Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> > Reviewed-by: Steve Capper <steve.cap...@arm.com> > Reviewed-by: Gavin Hu <gavin...@arm.com> > --- > lib/librte_hash/rte_cuckoo_hash.c | 4 ++++ > lib/librte_hash/rte_cuckoo_hash.h | 11 +++++++++++ > 2 files changed, 15 insertions(+) > While I think enqueuing a free slot should ever fail, I don't see an issue with having a check for that.
Acked-by: Bruce Richardson <bruce.richard...@intel.com>