whereis: use xstrncpy()

[thirdparty/util-linux.git] / lib / cpuset.c

diff --git a/lib/cpuset.c b/lib/cpuset.c
index fbe99e2a55b90873988fb9ae97dd836f874a3349..2847db8530324925972ba96a4bf3eb049ed6c443 100644 (file)
--- a/lib/cpuset.c
+++ b/lib/cpuset.c
@@ -29,23 +29,21 @@ static inline int val_to_char(int v)
 {
        if (v >= 0 && v < 10)
                return '0' + v;
-       else if (v >= 10 && v < 16)
+       if (v >= 10 && v < 16)
                return ('a' - 10) + v;
-       else
-               return -1;
+       return -1;
 }
 
 static inline int char_to_val(int c)
 {
        int cl;
 
-       cl = tolower(c);
        if (c >= '0' && c <= '9')
                return c - '0';
-       else if (cl >= 'a' && cl <= 'f')
+       cl = tolower(c);
+       if (cl >= 'a' && cl <= 'f')
                return cl + (10 - 'a');
-       else
-               return -1;
+       return -1;
 }
 
 static const char *nexttoken(const char *q,  int sep)
@@ -255,7 +253,6 @@ int cpumask_parse(const char *str, cpu_set_t *set, size_t setsize)
                        CPU_SET_S(cpu + 2, setsize, set);
                if (val & 8)
                        CPU_SET_S(cpu + 3, setsize, set);
-               len--;
                ptr--;
                cpu += 4;
        }
@@ -263,6 +260,19 @@ int cpumask_parse(const char *str, cpu_set_t *set, size_t setsize)
        return 0;
 }
 
+static int nextnumber(const char *str, char **end, unsigned int *result)
+{
+       errno = 0;
+       if (str == NULL || *str == '\0' || !isdigit(*str))
+               return -EINVAL;
+       *result = (unsigned int) strtoul(str, end, 10);
+       if (errno)
+               return -errno;
+       if (str == *end)
+               return -EINVAL;
+       return 0;
+}
+
 /*
  * Parses string with list of CPU ranges.
  * Returns 0 on success.
@@ -275,7 +285,7 @@ int cpulist_parse(const char *str, cpu_set_t *set, size_t setsize, int fail)
 {
        size_t max = cpuset_nbits(setsize);
        const char *p, *q;
-       int r = 0;
+       char *end = NULL;
 
        q = str;
        CPU_ZERO_S(setsize, set);
@@ -285,21 +295,24 @@ int cpulist_parse(const char *str, cpu_set_t *set, size_t setsize, int fail)
                unsigned int b; /* end of range */
                unsigned int s; /* stride */
                const char *c1, *c2;
-               char c;
 
-               if ((r = sscanf(p, "%u%c", &a, &c)) < 1)
+               if (nextnumber(p, &end, &a) != 0)
                        return 1;
                b = a;
                s = 1;
+               p = end;
 
                c1 = nexttoken(p, '-');
                c2 = nexttoken(p, ',');
+
                if (c1 != NULL && (c2 == NULL || c1 < c2)) {
-                       if ((r = sscanf(c1, "%u%c", &b, &c)) < 1)
+                       if (nextnumber(c1, &end, &b) != 0)
                                return 1;
-                       c1 = nexttoken(c1, ':');
+
+                       c1 = end && *end ? nexttoken(end, ':') : NULL;
+
                        if (c1 != NULL && (c2 == NULL || c1 < c2)) {
-                               if ((r = sscanf(c1, "%u%c", &s, &c)) < 1)
+                               if (nextnumber(c1, &end, &s) != 0)
                                        return 1;
                                if (s == 0)
                                        return 1;
@@ -316,7 +329,7 @@ int cpulist_parse(const char *str, cpu_set_t *set, size_t setsize, int fail)
                }
        }
 
-       if (r == 2)
+       if (end && *end)
                return 1;
        return 0;
 }
@@ -393,7 +406,7 @@ int main(int argc, char *argv[])
 
 usage_err:
        fprintf(stderr,
-               "usage: %s [--ncpus <num>] --mask <mask> | --range <list>",
+               "usage: %s [--ncpus <num>] --mask <mask> | --range <list>\n",
                program_invocation_short_name);
        exit(EXIT_FAILURE);
 }