Piotr Caban : msvcrt: Improve error handling in _wgetenv_s.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 667d889d3a33bae635b6a63b356fa8485a83e6d2
URL:    https://gitlab.winehq.org/wine/wine/-/commit/667d889d3a33bae635b6a63b356fa84...

Author: Piotr Caban 
Date:   Wed Feb 15 13:35:48 2023 +0100

msvcrt: Improve error handling in _wgetenv_s.

---

dlls/msvcrt/environ.c | 41 ++++++++++++++++++++++-------------------
 1 file changed, 22 insertions(+), 19 deletions(-)

diff --git a/dlls/msvcrt/environ.c b/dlls/msvcrt/environ.c
index 39a2f6b4e09..729eb0560bd 100644
--- a/dlls/msvcrt/environ.c
+++ b/dlls/msvcrt/environ.c
@@ -56,15 +56,12 @@ char * CDECL getenv(const char *name)
     return getenv_helper(name);
 }

-/*********************************************************************
- *              _wgetenv (MSVCRT.@)
- */
-wchar_t * CDECL _wgetenv(const wchar_t *name)
+static wchar_t * wgetenv_helper(const wchar_t *name)
 {
     wchar_t **env;
     size_t len;

-    if (!MSVCRT_CHECK_PMT(name != NULL)) return NULL;
+    if (!name) return NULL;
     len = wcslen(name);

/* Initialize the _wenviron array if it's not already created. */
@@ -84,6 +81,16 @@ wchar_t * CDECL _wgetenv(const wchar_t *name)
     return NULL;
 }

+/*********************************************************************
+ *              _wgetenv (MSVCRT.@)
+ */
+wchar_t * CDECL _wgetenv(const wchar_t *name)
+{
+    if (!MSVCRT_CHECK_PMT(name != NULL)) return NULL;
+
+    return wgetenv_helper(name);
+}
+
 /*********************************************************************
  *		_putenv (MSVCRT.@)
  */
@@ -306,25 +313,21 @@ int CDECL getenv_s(size_t *ret_len, char* buffer, size_t len, const char *varnam
 /******************************************************************
  *		_wgetenv_s (MSVCRT.@)
  */
-int CDECL _wgetenv_s(size_t *pReturnValue, wchar_t *buffer, size_t numberOfElements,
+int CDECL _wgetenv_s(size_t *ret_len, wchar_t *buffer, size_t len,
                      const wchar_t *varname)
 {
     wchar_t *e;

-    if (!MSVCRT_CHECK_PMT(pReturnValue != NULL)) return EINVAL;
-    if (!MSVCRT_CHECK_PMT(!(buffer == NULL && numberOfElements > 0))) return EINVAL;
-    if (!MSVCRT_CHECK_PMT(varname != NULL)) return EINVAL;
+    if (!MSVCRT_CHECK_PMT(ret_len != NULL)) return EINVAL;
+    *ret_len = 0;
+    if (!MSVCRT_CHECK_PMT((buffer && len > 0) || (!buffer && !len))) return EINVAL;
+    if (buffer) buffer[0] = 0;
+
+    if (!(e = wgetenv_helper(varname))) return 0;
+    *ret_len = wcslen(e) + 1;
+    if (!len) return 0;
+    if (len < *ret_len) return ERANGE;

-    if (!(e = _wgetenv(varname)))
-    {
-        *pReturnValue = 0;
-        return *_errno() = EINVAL;
-    }
-    *pReturnValue = wcslen(e) + 1;
-    if (numberOfElements < *pReturnValue)
-    {
-        return *_errno() = ERANGE;
-    }
     wcscpy(buffer, e);
     return 0;
 }