[dpdk-dev] net/mrvl: reset errno before library call
Checks
Commit Message
In case errno contains some non-zero value and call to strtoul() is
successful -2 error will be returned.
Setting errno to zero before calling strtoul() makes sure
errors will be handled correctly.
Fixes: 0ddc9b815b11 ("net/mrvl: add net PMD skeleton")
Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
---
drivers/net/mrvl/mrvl_qos.c | 1 +
1 file changed, 1 insertion(+)
--
2.7.4
Comments
On 11/2/2017 1:37 AM, Tomasz Duszynski wrote:
> In case errno contains some non-zero value and call to strtoul() is
> successful -2 error will be returned.
>
> Setting errno to zero before calling strtoul() makes sure
> errors will be handled correctly.
>
> Fixes: 0ddc9b815b11 ("net/mrvl: add net PMD skeleton")
>
> Signed-off-by: Tomasz Duszynski <tdu@semihalf.com>
Applied to dpdk-next-net/master, thanks.
@@ -122,6 +122,7 @@ get_val_securely(const char *string, uint32_t *val)
if (len == 0)
return -1;
+ errno = 0;
*val = strtoul(string, &endptr, 0);
if (errno != 0 || RTE_PTR_DIFF(endptr, string) != len)
return -2;