From 9e71090b9a6ddd3873a581f47423f86b492384c3 Mon Sep 17 00:00:00 2001 From: Jeff Johnston Date: Fri, 26 Sep 2008 16:23:58 +0000 Subject: 2008-09-26 Craig Howland * libc/stdlib/getenv_r.c (_getenv_r): Modify to not match if name contains an equal sign. --- newlib/libc/stdlib/getenv_r.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) (limited to 'newlib/libc') diff --git a/newlib/libc/stdlib/getenv_r.c b/newlib/libc/stdlib/getenv_r.c index bfa9682f2..272b097ad 100644 --- a/newlib/libc/stdlib/getenv_r.c +++ b/newlib/libc/stdlib/getenv_r.c @@ -29,7 +29,8 @@ A pointer to the (string) value of the environment variable, or PORTABILITY <<_getenv_r>> is not ANSI; the rules for properly forming names of environment -variables vary from one system to another. +variables vary from one system to another. This implementation does not +permit '=' to be in identifiers. <<_getenv_r>> requires a global pointer <>. */ @@ -98,17 +99,22 @@ _DEFUN (_findenv_r, (reent_ptr, name, offset), return NULL; } - len = strlen(name); - c = name + len; - - for (p = *p_environ; *p; ++p) - if (!strncmp (*p, name, len)) - if (*(c = *p + len) == '=') + c = name; + while (*c && *c != '=') c++; + + /* Identifiers may not contain an '=', so cannot match if does */ + if(*c != '=') + { + len = c - name; + for (p = *p_environ; *p; ++p) + if (!strncmp (*p, name, len)) + if (*(c = *p + len) == '=') { *offset = p - *p_environ; - ENV_UNLOCK; + ENV_UNLOCK; return (char *) (++c); } + } ENV_UNLOCK; return NULL; } -- cgit v1.2.3