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

Swift: Improvements related to the swift/cleartext-logging query. #13980

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

geoffw0
Copy link
Contributor

@geoffw0 geoffw0 commented Aug 16, 2023

Improvements related to the swift/cleartext-logging query:

  • model flow through getVaList (so that we can pass the NSLogv test cases).
  • add data flow through variadic arguments (this will have effects more widely than the swift/cleartext-logging query and is separately tested).
  • add an allowImplicitRead for array content sinks in the swift/cleartext-logging query (so that, together with the previous bullet point, we can pass the NSLog and more of the print test cases).
  • add more sinks for the swift/cleartext-logging query.

@geoffw0 geoffw0 added the Swift label Aug 16, 2023
@geoffw0 geoffw0 requested a review from a team as a code owner August 16, 2023 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant