eal: fix strnlen return value with icc
authorCunming Liang <cunming.liang@intel.com>
Tue, 17 Feb 2015 02:08:01 +0000 (10:08 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 24 Feb 2015 19:22:08 +0000 (20:22 +0100)
The problem is that strnlen() here may return invalid value with 32bit icc.
(actually it returns it’s second parameter,e.g: sysconf(_SC_ARG_MAX)).
It starts to manifest hwen max_len parameter is > 2M and using icc –m32 –O2 (or above).

Suggested-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
Signed-off-by: Cunming Liang <cunming.liang@intel.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
lib/librte_eal/common/eal_common_options.c

index c59370b..4319549 100644 (file)
@@ -170,7 +170,7 @@ eal_parse_coremask(const char *coremask)
        if (coremask[0] == '0' && ((coremask[1] == 'x')
                || (coremask[1] == 'X')))
                coremask += 2;
-       i = strnlen(coremask, PATH_MAX);
+       i = strlen(coremask);
        while ((i > 0) && isblank(coremask[i - 1]))
                i--;
        if (i == 0)
@@ -230,7 +230,7 @@ eal_parse_corelist(const char *corelist)
        /* Remove all blank characters ahead and after */
        while (isblank(*corelist))
                corelist++;
-       i = strnlen(corelist, sysconf(_SC_ARG_MAX));
+       i = strlen(corelist);
        while ((i > 0) && isblank(corelist[i - 1]))
                i--;
 
@@ -475,7 +475,7 @@ eal_parse_lcores(const char *lcores)
        /* Remove all blank characters ahead and after */
        while (isblank(*lcores))
                lcores++;
-       i = strnlen(lcores, sysconf(_SC_ARG_MAX));
+       i = strlen(lcores);
        while ((i > 0) && isblank(lcores[i - 1]))
                i--;