Win32: use low-level memory allocation during initialization
commit9a5df1692483a81e852fbfe2b0dd7d389b85ab0c
authorKarsten Blees <blees@dcon.de>
Thu, 15 Mar 2012 19:29:04 +0000 (15 20:29 +0100)
committerPat Thoyts <patthoyts@users.sourceforge.net>
Mon, 22 Oct 2012 12:30:48 +0000 (22 13:30 +0100)
tree1de5dc51c5d64de0ad3cf04da7bb7906c5b35fe8
parent5140042b213c641a47473f08d6ffe4f11fbc7030
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