Skip to content

C#: Account for CFG splitting in AssignableDefinition::getAFirstRead() and AssignableRead::getANextRead()#708

Merged
semmle-qlci merged 3 commits into
github:masterfrom
hvitved:csharp/ssa-read-splitting
Jan 3, 2019
Merged

C#: Account for CFG splitting in AssignableDefinition::getAFirstRead() and AssignableRead::getANextRead()#708
semmle-qlci merged 3 commits into
github:masterfrom
hvitved:csharp/ssa-read-splitting

Conversation

@hvitved
Copy link
Copy Markdown
Contributor

@hvitved hvitved commented Dec 20, 2018

No description provided.

@hvitved hvitved added the C# label Dec 20, 2018
@hvitved hvitved requested a review from calumgrant December 20, 2018 08:56
@hvitved hvitved requested a review from a team as a code owner December 20, 2018 08:56
@hvitved hvitved force-pushed the csharp/ssa-read-splitting branch from 8cd855b to 7172e62 Compare December 21, 2018 11:11
@hvitved
Copy link
Copy Markdown
Contributor Author

hvitved commented Dec 21, 2018

Rebased to resolve merge conflict in Nullness.qll.

Copy link
Copy Markdown
Contributor

@calumgrant calumgrant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor change, otherwise LGTM.

cfn = result.getAControlFlowNode() |
exists(Ssa::ExplicitDefinition def |
result = def.getAFirstReadAtNode(cfn) |
this = def.(Ssa::ExplicitDefinition).getADefinition()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This cast is redundant?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed.

@hvitved hvitved force-pushed the csharp/ssa-read-splitting branch from 7172e62 to 2427f0a Compare January 2, 2019 09:09
@hvitved
Copy link
Copy Markdown
Contributor Author

hvitved commented Jan 2, 2019

Rebased to resolve merge conflict in PreSsaConsistency.ql.

@semmle-qlci semmle-qlci merged commit c0868bc into github:master Jan 3, 2019
@hvitved hvitved deleted the csharp/ssa-read-splitting branch January 3, 2019 18:00
hmlendea added a commit to hmlendea/steam-key-activator that referenced this pull request Nov 19, 2021
until it supports .NET 6.0 (github/codeql#708)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants