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

RtlpHeapFailureInfo thwarts leak reporting #292

Closed
derekbruening opened this issue Nov 28, 2014 · 2 comments
Closed

RtlpHeapFailureInfo thwarts leak reporting #292

derekbruening opened this issue Nov 28, 2014 · 2 comments

Comments

@derekbruening
Copy link
Contributor

From bruen...@google.com on February 11, 2011 16:12:48

tests/suppress.c deliberately calls HeapFree 2 words into an allocation to
test invalid heap argument reporting. that allocation is thus leaked.
however, on Vista and Win7, the leak is not reported (it is on xp).

I tracked it down to some RtlpHeapFailureInfo data structure:

Note: next higher malloc: 0x01319a78-0x01319a98
...
Error #15: INVALID HEAP ARGUMENT: RtlFreeHeap 0x01319a80
...
defined range 0x775d4000-0x775d7000
0x775d4278 points to chunk 0x01319a78-0x01319a98

0:001> ln 0x775d4278
(775d4268) ntdll!RtlpHeapFailureInfo+0x10 | (775d4320) ntdll!g_SxsTrackReleaseStacks

This complication would go away if we replaced instead of wrapped

Original issue: http://code.google.com/p/drmemory/issues/detail?id=292

@derekbruening
Copy link
Contributor Author

From bruen...@google.com on March 16, 2011 07:55:46

Hopefully any such leak false negative will always be preceded by
unaddressable errors or heap routine failure warnings, and that once fixed
the leak will be reported

@derekbruening
Copy link
Contributor Author

From derek.br...@gmail.com on March 16, 2011 09:09:52

This issue was closed by revision r229 .

Status: Fixed

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