Avoid palloc before CurrentMemoryContext is set up on win32

Instead, write the unconverted output - it will be in the wrong
encoding, but at least we don't crash.

Rushabh Lathia

diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c
index feb0bd51fc9f643475bc1aab31bde7674d6448eb..71aed1e0f1dea480c87e52aa0554c1b74713f62f 100644 (file)
--- a/src/backend/utils/error/elog.c
+++ b/src/backend/utils/error/elog.c
@@ -1658,10 +1658,14 @@ write_console(const char *line, int len)
        /*
         * WriteConsoleW() will fail of stdout is redirected, so just fall through
         * to writing unconverted to the logfile in this case.
+        *
+        * Since we palloc the structure required for conversion, also fall through
+        * to writing unconverted if we have not yet set up CurrentMemoryContext.
         */
        if (GetDatabaseEncoding() != GetPlatformEncoding() &&
                !in_error_recursion_trouble() &&
-               !redirection_done)
+               !redirection_done &&
+               CurrentMemoryContext != NULL)
        {
                WCHAR      *utf16;
                int                     utf16len;