[reactos] 01/01: [MSVCRT] Follow-up of Follow-up of #5032 (f172503)

[Katayama Hirofumi MZ]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=fdedc549d0163168199156...

commit fdedc549d0163168199156df10cc7f4fd816de7d
Author:     Katayama Hirofumi MZ [e-mail]
AuthorDate: Sun Mar 5 23:28:00 2023 +0900
Commit:     Katayama Hirofumi MZ [e-mail]
CommitDate: Sun Mar 5 23:28:00 2023 +0900

[MSVCRT] Follow-up of Follow-up of #5032 (f172503)

Fix for system/_wsystem.
    Use _set_errno(ENOMEM) for malloc failure.
    Rename status variable as exit_code.
---
 sdk/lib/crt/process/_system.c | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/sdk/lib/crt/process/_system.c b/sdk/lib/crt/process/_system.c
index 2b2696e6d45..a7e3b174e56 100644
--- a/sdk/lib/crt/process/_system.c
+++ b/sdk/lib/crt/process/_system.c
@@ -21,11 +21,10 @@ int system(const char *command)
 {
   char *szCmdLine = NULL;
   char *szComSpec = NULL;
-
   PROCESS_INFORMATION ProcessInformation;
   STARTUPINFOA StartupInfo;
   BOOL result;
-  int status;
+  int exit_code;
   char cmd_exe[MAX_PATH];

szComSpec = getenv("COMSPEC");
@@ -46,8 +45,8 @@ int system(const char *command)
   szCmdLine = malloc(1 + strlen(szComSpec) + 5 + strlen(command) + 1);
   if (szCmdLine == NULL)
   {
-     _dosmaperr(GetLastError());
-     return -1;
+    _set_errno(ENOMEM);
+    return -1;
   }

strcpy(szCmdLine, """);
@@ -88,12 +87,12 @@ int system(const char *command)
   CloseHandle(ProcessInformation.hThread);

/* Wait for the process to exit */
-  _cwait(&status, (intptr_t)ProcessInformation.hProcess, 0);
+  _cwait(&exit_code, (intptr_t)ProcessInformation.hProcess, 0);

CloseHandle(ProcessInformation.hProcess);

_set_errno(0);
-  return status;
+  return exit_code;
 }

int CDECL _wsystem(const wchar_t *cmd)
@@ -103,7 +102,7 @@ int CDECL _wsystem(const wchar_t *cmd)
     PROCESS_INFORMATION process_info;
     STARTUPINFOW startup_info;
     BOOL result;
-    int status;
+    int exit_code;
     wchar_t cmd_exe[MAX_PATH];

comspec = _wgetenv(L"COMSPEC");
@@ -124,7 +123,7 @@ int CDECL _wsystem(const wchar_t *cmd)
     cmdline = malloc((1 + wcslen(comspec) + 5 + wcslen(cmd) + 1) * sizeof(wchar_t));
     if (cmdline == NULL)
     {
-        _dosmaperr(GetLastError());
+        _set_errno(ENOMEM);
         return -1;
     }

@@ -167,10 +166,10 @@ int CDECL _wsystem(const wchar_t *cmd)
     CloseHandle(process_info.hThread);

/* Wait for the process to exit */
-    _cwait(&status, (intptr_t)process_info.hProcess, 0);
+    _cwait(&exit_code, (intptr_t)process_info.hProcess, 0);

CloseHandle(process_info.hProcess);

_set_errno(0);
-    return status;
+    return exit_code;
 }