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

check for SYSARG_INLINED_BOOLEAN uses compacted argnum instead of real argnum! #1088

Closed
derekbruening opened this issue Nov 28, 2014 · 1 comment

Comments

@derekbruening
Copy link
Contributor

From [email protected] on November 26, 2012 15:05:46

while refactoring the syscall code for issue #822 I noticed this bug. I believe
the main instances of it that I hit back when I added this feature happen
to not have a problem. the consequences are false negatives and false
positives, but I don't recall seeing any false positives that could have
been from this. still, it should be fixed.

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

@derekbruening
Copy link
Contributor Author

From [email protected] on August 30, 2013 15:34:04

AFAICT this was fixed as part of the drsyscall refactoring. The old code used the full ordinal while the new code uses compacted (yes, the title here seems reversed).

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