mempool: fix stack handler dequeue
authorOlivier Matz <olivier.matz@6wind.com>
Mon, 23 Jan 2017 17:11:03 +0000 (18:11 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Sun, 29 Jan 2017 22:38:33 +0000 (23:38 +0100)
commite09ff22d538e13ead91a5caa2d136e8e065e5ed1
tree25daf0797eeb769131694827d81abbeca2a9d17c
parent3cdfdf2a3303bb50d98c4f1689fc4b3745ea1e4f
mempool: fix stack handler dequeue

The return value of the stack handler is wrong: it should be 0 on
success, not the number of objects dequeued.

This could lead to memory leaks depending on how the caller checks the
return value (ret < 0 or ret != 0). This was also breaking autotests
with debug enabled, because the debug cookies are only updated when the
function returns 0, so the cookies were not updated, leading to
an abort().

Fixes: 295a530b0844 ("mempool: add stack mempool handler")
Cc: stable@dpdk.org
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
lib/librte_mempool/rte_mempool_stack.c