diff options
| author | Renaud Paquay <rpaquay@google.com> | 2017-05-10 17:48:59 -0700 |
|---|---|---|
| committer | Elliott Hughes <enh@google.com> | 2017-05-11 10:30:52 -0700 |
| commit | b7a4f0b9e2779fbddc675641c5a2978168f0df2f (patch) | |
| tree | 30449eeb15a2b8b7387945c1f2eee6e2eca99976 /trusty/coverage/coverage.cpp | |
| parent | 44f6592b0de532afeb3636afb5177fd39b2442ea (diff) | |
Fix uninitialized member variable
The default constructor of FileMap was missing an initializer for the
mFileMapping variables. This results in CloseHandle being called with
a "random" value, which can cause havoc in Win32 process over time (e.g.
in the case of libaapt2_jni.dll, which is loaded in a JVM process).
Also, update the code to use "NULL" for invalid file map handle
and "INVALID_HANDLE_VALUE" for invalid file handle.
Bug: 38197857
Test: Stress testing on (Windows) machine
Change-Id: Ibd769219d601fbafcfcee89e848b31cc5137826c
Diffstat (limited to 'trusty/coverage/coverage.cpp')
0 files changed, 0 insertions, 0 deletions