From: Eugeniu Rosca <erosca@de.adit-jv.com>
Date: Sat, 1 Jul 2017 12:57:27 +0000 (+0200)
Subject: selftests: watchdog: prefer strtoul() over atoi()
X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=7a5e10d439b162ae76e7d7a1bbd5653e62abb5bf;p=GitHub%2Fmoto-9609%2Fandroid_kernel_motorola_exynos9610.git

selftests: watchdog: prefer strtoul() over atoi()

commit f15d7114bbdd ("Documentation/watchdog: add timeout and ping rate
control to watchdog-test.c") used both atoi() and strtoul() for string
to integer conversion. As usage of atoi() is discouraged in newer code,
replace it with strtoul() for consistency.

Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
---

diff --git a/tools/testing/selftests/watchdog/watchdog-test.c b/tools/testing/selftests/watchdog/watchdog-test.c
index e7c9d3bdae56..5cfa88c73095 100644
--- a/tools/testing/selftests/watchdog/watchdog-test.c
+++ b/tools/testing/selftests/watchdog/watchdog-test.c
@@ -106,7 +106,7 @@ int main(int argc, char *argv[])
 			printf("Watchdog ping rate set to %u seconds.\n", ping_rate);
 			break;
 		case 't':
-			flags = atoi(optarg);
+			flags = strtoul(optarg, NULL, 0);
 			ret = ioctl(fd, WDIOC_SETTIMEOUT, &flags);
 			if (!ret)
 				printf("Watchdog timeout set to %u seconds.\n", flags);