[xen master] tools/misc: xenwatchdogd: add parse_secs()

[patchbot@xxxxxxx]

commit f0fa75c9ea9f03cf2c6f5e63d07664ca4b224dd6
Author:     Leigh Brown <leigh@xxxxxxxxxxxxx>
AuthorDate: Tue Apr 23 14:09:50 2024 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Tue Apr 23 14:09:50 2024 +0200

    tools/misc: xenwatchdogd: add parse_secs()
    
    Create a new parse_secs() function to parse the timeout and sleep
    parameters. This ensures that non-numeric parameters are not
    accidentally treated as numbers.
    
    Signed-off-by: Leigh Brown <leigh@xxxxxxxxxxxxx>
    Reviewed-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
---
 tools/misc/xenwatchdogd.c | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/tools/misc/xenwatchdogd.c b/tools/misc/xenwatchdogd.c
index 112b706357..9fa772e49f 100644
--- a/tools/misc/xenwatchdogd.c
+++ b/tools/misc/xenwatchdogd.c
@@ -49,6 +49,18 @@ static void catch_usr1(int sig)
     done = true;
 }
 
+static int parse_secs(const char *arg, const char *what)
+{
+    char *endptr;
+    unsigned long val;
+
+    val = strtoul(arg, &endptr, 0);
+    if (val > INT_MAX || *endptr)
+       errx(EXIT_FAILURE, "invalid %s: '%s'", what, arg);
+
+    return val;
+}
+
 int main(int argc, char **argv)
 {
     int id;
@@ -64,16 +76,11 @@ int main(int argc, char **argv)
     if (h == NULL)
        err(EXIT_FAILURE, "xc_interface_open");
 
-    t = strtoul(argv[1], NULL, 0);
-    if (t == ULONG_MAX)
-       err(EXIT_FAILURE, "strtoul");
+    t = parse_secs(argv[1], "timeout");
 
     s = t / 2;
-    if (argc == 3) {
-       s = strtoul(argv[2], NULL, 0);
-       if (s == ULONG_MAX)
-           err(EXIT_FAILURE, "strtoul");
-    }
+    if (argc == 3)
+       s = parse_secs(argv[2], "sleep");
 
     if (signal(SIGHUP, &catch_exit) == SIG_ERR)
        err(EXIT_FAILURE, "signal");
--
generated by git-patchbot for /home/xen/git/xen.git#master