Skip to content

Conversation

pwntester
Copy link
Contributor

Add public_send and __send__ as Code Injection sinks as proposed by @vcsjones

Add `public_send` and `__send__` as Code Injection sinks as proposed by @vcsjones
@pwntester pwntester requested a review from a team as a code owner December 28, 2023 18:09
@github-actions github-actions bot added the Ruby label Dec 28, 2023
@alexrford alexrford added the no-change-note-required This PR does not need a change note label Jan 2, 2024
@sidshank sidshank requested a review from alexrford January 2, 2024 14:19
Copy link
Contributor

@alexrford alexrford left a comment

Choose a reason for hiding this comment

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

LGTM, just an autoformat suggestion to fix the compilation check.

Co-authored-by: Alex Ford <alexrford@users.noreply.github.com>
@pwntester pwntester changed the title Update Kernel.qll to include Object.send aliases Ruby: Update Kernel.qll to include Object.send aliases Jan 8, 2024
Copy link
Contributor

@alexrford alexrford left a comment

Choose a reason for hiding this comment

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

👍

@alexrford alexrford merged commit ef8ca55 into github:main Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-change-note-required This PR does not need a change note Ruby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants