Merge pull request #160 from tmobile/tmo-CFSPDK-1058-Fix-ping-with-count-zero

johnlange2 · web-flow · commit bffcfe09ac52 · 2023-05-23T14:07:09.000-06:00
Fixed crash on ping count zero
diff --git a/samples/tmo_shell/src/tmo_ping.c b/samples/tmo_shell/src/tmo_ping.c
@@ -86,6 +86,7 @@ int cmd_ping(const struct shell *shell, size_t argc, char **argv)
     if (ping_cnt <= 0) {
         shell_error(shell, "Invalid ping count %d", ping_cnt);
         print_usage(shell);
+        return EINVAL;
     }
     if (!net_ipaddr_parse(host, strlen(host), &dst)) 
     {

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,7 @@ int cmd_ping(const struct shell shell, size_t argc, char *argv)`
`86`	`86`	`if (ping_cnt <= 0) {`
`87`	`87`	`shell_error(shell, "Invalid ping count %d", ping_cnt);`
`88`	`88`	`print_usage(shell);`
	`89`	`+ return EINVAL;`
`89`	`90`	`}`
`90`	`91`	`if (!net_ipaddr_parse(host, strlen(host), &dst))`
`91`	`92`	`{`