Skip to content

JS: track functions with methods #7409

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.

Sign up for GitHub

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

Jump to bottom
Merged
codeql-ci merged 2 commits into from
Dec 16, 2021
Merged

JS: track functions with methods #7409

codeql-ci merged 2 commits into from
Dec 16, 2021

Conversation

@asgerf
Copy link
Contributor

@asgerf asgerf commented Dec 15, 2021

Fixes #7106 by tracking functions with methods.

Evaluations:

  • Evaluation on default slugs looks ok (unfortunately the meta-queries don't currently works on default slugs)
  • Evaluation on smoke test shows 1 new call edge

@asgerf asgerf added the JS label Dec 15, 2021
@asgerf asgerf requested a review from a team as a code owner December 15, 2021 10:44
@asgerf asgerf mentioned this pull request Dec 15, 2021
Closed
erik-krogh
erik-krogh approved these changes Dec 15, 2021
View reviewed changes
Copy link
Contributor

@erik-krogh erik-krogh left a comment

Choose a reason for hiding this comment

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

👍

@codeql-ci codeql-ci merged commit f274f06 into github:main Dec 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erik-krogh erik-krogh erik-krogh approved these changes

Assignees

No one assigned

Labels

JS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[JavaScript] Another limited case for tainting objects with methods

3 participants

@asgerf @erik-krogh @codeql-ci