Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

False pos uninits and leaks from HTTPS OCSP checking #841

Open
derekbruening opened this issue Nov 28, 2014 · 0 comments
Open

False pos uninits and leaks from HTTPS OCSP checking #841

derekbruening opened this issue Nov 28, 2014 · 0 comments

Comments

@derekbruening
Copy link
Contributor

From 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 <webio.dll+0x1a3c>)
3668 # 1 webio.dll!? +0x0 (0x713a2b5b <webio.dll+0x2b5b>)
3668 # 2 webio.dll!? +0x0 (0x713a2ab8 <webio.dll+0x2ab8>)
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 <webio.dll+0x1a5e>)
3668 # 1 webio.dll!? +0x0 (0x713a2b5b <webio.dll+0x2b5b>)
3668 # 2 webio.dll!? +0x0 (0x713a2ab8 <webio.dll+0x2ab8>)
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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant