> -----Original Message-----
> From: Suanming Mou <[email protected]>
> Sent: Friday, January 5, 2024 12:03 AM
> To: [email protected]; Power, Ciara <[email protected]>
> Cc: [email protected]
> Subject: [PATCH v2 1/2] app/test-crypto-perf: fix invalid memcmp results
>
> The function memcmp() returns an integer less than, equal to, or greater than
> zero. In current code, if the first memcmp() returns less than zero and the
> second memcmp() returns greater than zero, the sum of results may still be 0
> and indicates verify succussed.
>
> This commit converts the return value to be zero or greater than zero. That
> will
> make sure the sum of results be correct.
>
> Fixes: df52cb3b6e13 ("app/crypto-perf: move verify as single test type")
>
> Signed-off-by: Suanming Mou <[email protected]>
> Acked-by: Anoob Joseph <[email protected]>
> ---
> app/test-crypto-perf/cperf_test_verify.c | 6 +++---
Acked-by: Ciara Power <[email protected]>
