OSDN Git Service

add argument check in setenv()
authorXishi Qiu <qiuxishi@huawei.com>
Tue, 4 Nov 2014 11:26:28 +0000 (19:26 +0800)
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Fri, 23 Jan 2015 19:59:27 +0000 (20:59 +0100)
setenv() in glibc/eglibc will check the argument, like this,
  ...
  if (name == NULL || *name == '\0' || strchr (name, '=') != NULL)
    {
      __set_errno (EINVAL);
      return -1;
    }
  ...
So add argument check in uclibc's setenv() too.

Signed-off-by: Xishi Qiu <qiuxishi@huawei.com>
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
libc/stdlib/setenv.c

index f3b53b6..ecc3025 100644 (file)
@@ -41,7 +41,7 @@ static char **last_environ;
    to reuse values once generated for a `setenv' call since we can never
    free the strings. [in uclibc, we do not]  */
 static int __add_to_environ(const char *name, const char *value,
-               int replace)
+               int replace)
 {
        register char **ep;
        register size_t size;
@@ -116,6 +116,11 @@ static int __add_to_environ(const char *name, const char *value,
 
 int setenv(const char *name, const char *value, int replace)
 {
+       if (name == NULL || *name == '\0' || strchr (name, '=') != NULL) {
+               __set_errno(EINVAL);
+               return -1;
+       }
+
        /* NB: setenv("VAR", NULL, 1) inserts "VAR=" string */
        return __add_to_environ(name, value ? value : "", replace);
 }