8000 Added confidential checker by iywang2016 · Pull Request #6981 · typetools/checker-framework · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added confidential checker #6981

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

Open
wants to merge 223 commits into
base: master
Choose a base branch
from
Open

Conversation

iywang2016
Copy link
Contributor
@iywang2016 iywang2016 commented Mar 3, 2025

Associated with JDK pull request #224

gitfundo and others added 30 commits June 18, 2024 14:42
SQL tainting qualifiers and basic tests
…bined SqlQuoteless and SqlEvenQuotes; revised documentations
iywang2016 and others added 29 commits April 8, 2025 10:53
Copy link
Member
@mernst mernst left a comment

Choose a reason for hiding this comment

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

I have just a couple last comments.

* Enforces Confidential String concatenation rules:
*
* <ul>
* <li>(Confidential + NonConfidential) returns Confidential (commutatively);
Copy link
Member

Choose a reason for hiding this comment

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

This is the same comment text as in the TreeAnnotator. Can you please document the relationship between the two? What does each one do, and why are both necessary?

* @param leftOperand the left operand to be concatenated
* @param rightOperand the right operand to be concatenated
* @param p the input abstract values
* @return the resulting AnnotationMirror of the string concatenation operation
Copy link
Member

Choose a reason for hiding this comment

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

This method may also return null. Please document the conditions under which that happens.

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.

3 participants
0