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>

Reply via email to