[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH v5 13/14] tools/xenstore: switch get_optval_int() to get_optval_uint()



Let get_optval_int() return an unsigned value and rename it
accordingly.

Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
---
V5:
- new patch, carved out from next patch in series (Julien Grall)
---
 tools/xenstore/xenstored_core.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c
index fce73b883e..86ec7ab446 100644
--- a/tools/xenstore/xenstored_core.c
+++ b/tools/xenstore/xenstored_core.c
@@ -2700,13 +2700,13 @@ int dom0_domid = 0;
 int dom0_event = 0;
 int priv_domid = 0;
 
-static int get_optval_int(const char *arg)
+static unsigned int get_optval_uint(const char *arg)
 {
        char *end;
-       long val;
+       unsigned long val;
 
-       val = strtol(arg, &end, 10);
-       if (!*arg || *end || val < 0 || val > INT_MAX)
+       val = strtoul(arg, &end, 10);
+       if (!*arg || *end || val > INT_MAX)
                barf("invalid parameter value \"%s\"\n", arg);
 
        return val;
@@ -2726,7 +2726,7 @@ static void set_timeout(const char *arg)
 
        if (!eq)
                barf("quotas must be specified via <what>=<seconds>\n");
-       val = get_optval_int(eq + 1);
+       val = get_optval_uint(eq + 1);
        if (what_matches(arg, "watch-event"))
                timeout_watch_event_msec = val * 1000;
        else
@@ -2740,7 +2740,7 @@ static void set_quota(const char *arg, bool soft)
 
        if (!eq)
                barf("quotas must be specified via <what>=<nb>\n");
-       val = get_optval_int(eq + 1);
+       val = get_optval_uint(eq + 1);
        if (what_matches(arg, "outstanding") && !soft)
                quota_req_outstanding = val;
        else if (what_matches(arg, "transaction-nodes") && !soft)
-- 
2.35.3




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.