Win32: fix environment memory leaks
commit3c5e9718d41a5e742eaf9dcb8d6e46ac7b19411e
authorKarsten Blees <blees@dcon.de>
Fri, 25 Nov 2011 20:29:40 +0000 (25 21:29 +0100)
committerPat Thoyts <patthoyts@users.sourceforge.net>
Mon, 1 Oct 2012 13:34:56 +0000 (1 14:34 +0100)
tree6c51ebb21a4bf06f0c62d3f535c6d778667b9a03
parentcbfaad48ea72ea5f187bbf77005d6b076b2f596e
Win32: fix environment memory leaks

All functions that modify the environment have memory leaks.

Disable gitunsetenv in the Makefile and use env_setenv (via mingw_putenv)
instead (this frees removed environment entries).

Move xstrdup from env_setenv to make_augmented_environ, so that
mingw_putenv no longer copies the environment entries (according to POSIX
[1], "the string [...] shall become part of the environment"). This also
fixes the memory leak in gitsetenv, which expects a POSIX compliant putenv.

[1] http://pubs.opengroup.org/onlinepubs/009695399/functions/putenv.html

Note: This patch depends on taking control of char **environ and having
our own mingw_putenv (both introduced in "Win32: Unicode environment
(incoming)").

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