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

makes the taint-attached observation on taint introductions #1241

Merged
merged 2 commits into from
Nov 17, 2020

Conversation

ivg
Copy link
Member

@ivg ivg commented Nov 16, 2020

The observation was missing when the taint was introduced with
new_direct or new_indirect operators. Although known for a long time,
this behavior was considered more as a feature rather than a bug. We
finally decided to rectify it and post the observation when a new
taint is introduced to make the behavior match with the semantics
described in the low-level terms. It also makes it much easier to
debug taint-related issues.

ivg added 2 commits November 16, 2020 16:37
The observation was missing when the taint was introduced with
new_direct or new_indirect operators. Although known for a long time,
this behavior was considered more as a feature rather than a bug. We
finally decided to rectify it and post the observation when a new
taint is introduced to make the behavior match with the semantics
described in the low-level terms. It also makes it much easier to
debug taint-related issues.
@ivg ivg merged commit 2b911c3 into BinaryAnalysisPlatform:master Nov 17, 2020
@ivg ivg deleted the post-taint-attach-on-intro branch March 9, 2022 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant