On 9/4/2017 8:26 PM, De Lara Guarch, Pablo wrote:


-----Original Message-----
From: Hemant Agrawal [mailto:hemant.agra...@nxp.com]
Sent: Wednesday, August 23, 2017 1:24 PM
To: dev@dpdk.org
Cc: Doherty, Declan <declan.dohe...@intel.com>; De Lara Guarch, Pablo
<pablo.de.lara.gua...@intel.com>; akhil.go...@nxp.com
Subject: [PATCH] app/crypto-perf: fix uninitialized errno value

errno should be initialized to 0 before calling strtol

Fixes: f6cefe253cc8 ("app/crypto-perf: add range/list of sizes")
Cc: sta...@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>
---
 app/test-crypto-perf/cperf_options_parsing.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/test-crypto-perf/cperf_options_parsing.c b/app/test-crypto-
perf/cperf_options_parsing.c
index 085aa8f..77c405c 100644
--- a/app/test-crypto-perf/cperf_options_parsing.c
+++ b/app/test-crypto-perf/cperf_options_parsing.c
@@ -137,6 +137,7 @@ parse_range(const char *arg, uint32_t *min,
uint32_t *max, uint32_t *inc)
        if (copy_arg == NULL)
                return -1;

+       errno = 0;
        token = strtok(copy_arg, ":");

        /* Parse minimum value */
--
2.7.4

Could you extend this fix to the parse_list function?

Thanks!
Pablo

sure.

Reply via email to