Win32: use low-level memory allocation during initialization
commitcf4b5cb5fff31c3a48605358aaab603fb934effa
authorKarsten Blees <blees@dcon.de>
Thu, 15 Mar 2012 19:29:04 +0000 (15 20:29 +0100)
committerJohannes Schindelin <johannes.schindelin@gmx.de>
Thu, 3 May 2012 16:13:33 +0000 (3 11:13 -0500)
tree2e2aa29ee65ef4543433b21a210519b054dc75e9
parent28158f8d18ef0add1356db1152660a6387658d7f
Win32: use low-level memory allocation during initialization

As of d41489a6 "Add more large blob test cases", git's high-level memory
allocation functions (xmalloc, xmemdupz etc.) access the environment to
simulate limited memory in tests (see 'getenv("GIT_ALLOC_LIMIT")' in
memory_limit_check()). These functions should not be used before the
environment is fully initialized (particularly not to initialize the
environment itself).

The current solution ('environ = NULL; ALLOC_GROW(environ...)') only works
because MSVCRT's getenv() reinitializes environ when it is NULL (i.e. it
leaves us with two sets of unusabe (non-UTF-8) and unfreeable (CRT-
allocated) environments).

Add our own set of malloc-or-die functions to be used in startup code.

Also check the result of __wgetmainargs, which may fail if there's not
enough memory for wide-char arguments and environment.

This patch is in preparation of the sorted environment feature, which
completely replaces MSVCRT's getenv() implementation.

Signed-off-by: Karsten Blees <blees@dcon.de>
compat/mingw.c