]> granicus.if.org Git - curl/commit
Reinstate "WIN32 MemoryTracking: track wcsdup() _wcsdup() and _tcsdup() usage".
authorYang Tse <yangsita@gmail.com>
Thu, 18 Jul 2013 18:04:02 +0000 (20:04 +0200)
committerYang Tse <yangsita@gmail.com>
Thu, 18 Jul 2013 21:37:33 +0000 (23:37 +0200)
commit90695fb2c508803a09581b0ca8f55eaa36749a69
tree8ba98611465b9639fada7c24aef8e420f46405b2
parentdd17069c9e307ff135f4eb572671bd351f7e6a42
Reinstate "WIN32 MemoryTracking: track wcsdup() _wcsdup() and _tcsdup() usage".

This reverts commit 7ed25cc, reinstating commit 8ec2cb5.

As of 18-jul-2013 we still do have code in libcurl that makes use of these
memory functions. Commit 8ec2cb5 comment still applies and is yet valid.

These memory functions are solely used in Windows builds, so all related
code is protected with '#ifdef WIN32' preprocessor conditional compilation
directives.

Specifically, wcsdup() _wcsdup() are used when building a Windows target with
UNICODE and USE_WINDOWS_SSPI preprocessor symbols defined. This is the case
when building a Windows UNICODE target with Windows native SSL/TLS support
enabled.

Realizing that wcsdup() _wcsdup() are used is a bit tricky given that usage
of these is hidden behind _tcsdup() which is MS way of dealing with code
that must tolerate UNICODE and non-UNICODE compilation. Additionally, MS
header files and those compatible from other compilers use this preprocessor
conditional compilation directive in order to select at compilation time
whether 'wide' or 'ansi' MS API functions are used.

Without this code, Windows build targets with Windows native SSL/TLS support
enabled and MemoryTracking support enabled misbehave in tracking memory usage,
regardless of being a UNICODE enabled build or not.
lib/curl_memory.h
lib/curl_setup.h
lib/easy.c
lib/memdebug.c
lib/memdebug.h
tests/memanalyze.pl
tests/server/getpart.c