diff options
| author | Alan Wu <XrXr@users.noreply.github.com> | 2023-02-24 17:22:48 -0500 |
|---|---|---|
| committer | Maxime Chevalier-Boisvert <maximechevalierb@gmail.com> | 2023-02-27 11:12:22 -0500 |
| commit | 0eb634ae73cb327ede833b72492f912792a4a9d5 (patch) | |
| tree | 93219514a0e566fa3040d5893558d5ac7e60ada4 /internal/file.h | |
| parent | 55a24f9b08ac2217accb720e29232d56aed2e5a4 (diff) | |
| download | ruby-0eb634ae73cb327ede833b72492f912792a4a9d5.tar.gz | |
YJIT: Detect and reject `send(:alias_for_send, :foo)`
Previously, YJIT failed to put the stack into the correct shape when
`BasicObject#send` calls an alias method for the send method itself.
This can manifest as strange `NoMethodError`s in the final non-send
receiver, as [seen][1] with the kt-paperclip gem. I also found a case
where it makes YJIT fail the stack size assertion while compiling
`leave`.
YJIT's `BasicObject#__send__` implementation already rejects sends to
`send`, but didn't detect sends to aliases of `send`. Adjust the
detection and reject these cases.
Fixes [Bug #19464]
[1]: https://github.com/Shopify/yjit/issues/306
Diffstat (limited to 'internal/file.h')
0 files changed, 0 insertions, 0 deletions