net: shell: ping: streamline parse_arg function by using shell_strtol

Switch from using direct `strtol` calls to `shell_strtol`.
This change leverages the extensive error handling provided
by `shell_strtol`.

Signed-off-by: Pisit Sawangvonganan <[email protected]>

Author: ndrs-pst

subsys/net/lib/shell/ping.c

@@ -199,9 +199,10 @@ static int handle_ipv4_echo_reply(struct net_icmp_ctx *ctx,
 
 static int parse_arg(size_t *i, size_t argc, char *argv[])
 {
-	int res = -1;
+	int res;
+	int err;
+	int base;
 	const char *str = argv[*i] + 2;
-	char *endptr;
 
 	if (*str == 0) {
 		if (*i + 1 >= argc) {
@@ -212,14 +213,15 @@ static int parse_arg(size_t *i, size_t argc, char *argv[])
 		str = argv[*i];
 	}
 
-	errno = 0;
 	if (strncmp(str, "0x", 2) == 0) {
-		res = strtol(str, &endptr, 16);
+		base = 16;
 	} else {
-		res = strtol(str, &endptr, 10);
+		base = 10;
 	}
 
-	if (errno || (endptr == str)) {
+	err = 0;
+	res = shell_strtol(str, base, &err);
+	if (err != 0) {
 		return -1;
 	}