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


-----Original Message-----
From: Hemant Agrawal [mailto:[email protected]]
Sent: Wednesday, August 23, 2017 1:24 PM
To: [email protected]
Cc: Doherty, Declan <[email protected]>; De Lara Guarch, Pablo
<[email protected]>; [email protected]
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: [email protected]

Signed-off-by: Hemant Agrawal <[email protected]>
---
 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