FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fixed occurs `SIGSEGV` with `mrbgems/mruby-method` by dearblue · Pull Request #5580 · mruby/mruby · GitHub

/ mruby Public

Fixed occurs SIGSEGV with mrbgems/mruby-method - #5580

Merged
matz merged 1 commit into
mruby:masterfrom
dearblue:static-proc
Nov 23, 2021
Merged

Fixed occurs SIGSEGV with mrbgems/mruby-method#5580
matz merged 1 commit into
mruby:masterfrom
dearblue:static-proc

Conversation

Copy link
Copy Markdown
Contributor

Calling the Method#{parameters,source_location} method on a static Proc object resulted in SIGSEGV.
The trigger is #5402.

The original implementation of the Method#{parameters,source_location} method was to temporarily rewrite the object and then call the method of the same name in Proc.
Rewriting of objects placed in the ROM section by #5402 above is prohibited by hardware such as the CPU.
This caused a SIGSEGV.

Calling the `Method#{parameters,source_location}` method on a static `Proc` object resulted in `SIGSEGV`.
The trigger is mruby#5402.

The original implementation of the `Method#{parameters,source_location}` method was to temporarily rewrite the object and then call the method of the same name in `Proc`.
Rewriting of objects placed in the ROM section by mruby#5402 above is prohibited by hardware such as the CPU.
This caused a `SIGSEGV`.
dearblue requested a review from matz as a code owner November 22, 2021 13:06
matz merged commit 4a84fa5 into mruby:master Nov 23, 2021
dearblue deleted the static-proc branch December 5, 2021 04:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL