DynamoRIO / DynamoRIO/drmemory
False pos uninits and leaks from HTTPS OCSP checking
- Dominant language
- C
- Stars
- 2.7k
- Forks
- 290
- PR merge metrics
- No merged PRs in 30d
Description
_From [rnk@google.com](https://code.google.com/u/rnk@google.com/) on March 21, 2012 11:06:49_
These showed up on the full bot: http://build.chromium.org/p/chromium.fyi/builders/Windows%20Tests%20%28DrMemory%20full%29/builds/1132 ~~3668~~ Error `#1`: UNINITIALIZED READ: reading register ecx
~~3668~~ # 0 webio.dll!? +0x0 (0x713a1a3c )
~~3668~~ # 1 webio.dll!? +0x0 (0x713a2b5b )
~~3668~~ # 2 webio.dll!? +0x0 (0x713a2ab8 )
~~3668~~ # 3 WINHTTP.dll!WinHttpOpen
~~3668~~ # 4 WINHTTP.dll!WinHttpOpen
~~3668~~ # 5 WINHTTP.dll!WinHttpOpen
~~3668~~ # 6 CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ # 7 CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ # 8 CRYPTNET.dll!CertDllVerifyRevocation
~~3668~~ # 9 CRYPTNET.dll!CryptRetrieveObjectByUrlW
~~3668~~ `#10` CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ `#11` KERNEL32.dll!BaseThreadInitThunk
~~3668~~ Note: @0:07:40.229 in thread 3668
~~3668~~ Note: instruction: cmp %ecx 0x08(%edx)
~~3668~~
~~3668~~ Error `#2`: UNINITIALIZED READ: reading register ecx
~~3668~~ # 0 webio.dll!? +0x0 (0x713a1a5e )
~~3668~~ # 1 webio.dll!? +0x0 (0x713a2b5b )
~~3668~~ # 2 webio.dll!? +0x0 (0x713a2ab8 )
~~3668~~ # 3 WINHTTP.dll!WinHttpOpen
~~3668~~ # 4 WINHTTP.dll!WinHttpOpen
~~3668~~ # 5 WINHTTP.dll!WinHttpOpen
~~3668~~ # 6 CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ # 7 CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ # 8 CRYPTNET.dll!CertDllVerifyRevocation
~~3668~~ # 9 CRYPTNET.dll!CryptRetrieveObjectByUrlW
~~3668~~ `#10` CRYPTNET.dll!I_CryptNetGetConnectivity
~~3668~~ `#11` KERNEL32.dll!BaseThreadInitThunk
~~3668~~ Note: @0:07:40.229 in thread 3668
~~3668~~ Note: instruction: cmp %ecx %ebx
etc.
The stacks involve a bunch of dlls: cryptnet.dll, webio.dll, and winhttp.dll. Not sure why we weren't seeing these before. I haven't been able to untangle which Windows API calls trigger this, but the only net/ change in that build was https://chromiumcodereview.appspot.com/9663017/ , which adds some more tests.
_Original issue: http://code.google.com/p/drmemory/issues/detail?id=841_
Contributor guide
Assessment
This issue has not been assessed yet.