[pull] master from ruby:master#1220
Merged
Merged
Conversation
Use raise(3) instead of kill(2) for sending yourself SEGV.
Bumps the github-actions group with 5 updates in the / directory: | Package | From | To | | --- | --- | --- | | [ruby/setup-ruby](https://github.com/ruby/setup-ruby) | `1.318.0` | `1.319.0` | | [github/codeql-action/init](https://github.com/github/codeql-action) | `4.37.0` | `4.37.1` | | [github/codeql-action/analyze](https://github.com/github/codeql-action) | `4.37.0` | `4.37.1` | | [github/codeql-action/upload-sarif](https://github.com/github/codeql-action) | `4.37.0` | `4.37.1` | | [taiki-e/install-action](https://github.com/taiki-e/install-action) | `2.83.2` | `2.83.3` | Updates `ruby/setup-ruby` from 1.318.0 to 1.319.0 - [Release notes](https://github.com/ruby/setup-ruby/releases) - [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb) - [Commits](ruby/setup-ruby@8e41b36...003a5c4) Updates `github/codeql-action/init` from 4.37.0 to 4.37.1 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@99df26d...7188fc3) Updates `github/codeql-action/analyze` from 4.37.0 to 4.37.1 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@99df26d...7188fc3) Updates `github/codeql-action/upload-sarif` from 4.37.0 to 4.37.1 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@99df26d...7188fc3) Updates `taiki-e/install-action` from 2.83.2 to 2.83.3 - [Release notes](https://github.com/taiki-e/install-action/releases) - [Changelog](https://github.com/taiki-e/install-action/blob/main/CHANGELOG.md) - [Commits](taiki-e/install-action@43aecc8...ed67fa3) --- updated-dependencies: - dependency-name: ruby/setup-ruby dependency-version: 1.319.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions - dependency-name: github/codeql-action/init dependency-version: 4.37.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions - dependency-name: github/codeql-action/analyze dependency-version: 4.37.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions - dependency-name: github/codeql-action/upload-sarif dependency-version: 4.37.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions - dependency-name: taiki-e/install-action dependency-version: 2.83.3 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com>
We can reuse some of this work for gen_new_array fastpath for embedded arrays.
The HIR printer passed references to cfunc fields into PtrPrintMap, so snapshots depended on addresses of cloned instruction storage. Normalize the stored function pointers themselves and cover all C call instruction variants.
triggered_bits is modified between EC_EXEC_TAG and a possible longjmp (a postponed job can raise) and is read after EC_POP_TAG to re-post the bits that have not been executed yet, so C99 7.13.2.1p3 makes its value indeterminate exactly on that path. The neighboring saved_mask and saved_errno are already volatile for the same reason. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
`BCryptGenRandom` with `BCRYPT_USE_SYSTEM_PREFERRED_RNG` is available on every supported Windows version and `bcrypt.lib` is linked unconditionally on both mswin and mingw, so the deprecated CryptoAPI fallback is unreachable in practice. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fiber_scheduler_closed and event_thread_end_hooked are set between setjmp and a possible longjmp from the fiber scheduler close or the THREAD_END event hook, and are read after that longjmp to avoid running either step twice, so C99 7.13.2.1p3 makes their values indeterminate exactly when they are needed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
cl.exe's traditional preprocessor passes __VA_ARGS__ to a nested macro
as a single argument, so the multi-argument probe triggers:
gc/default/default.c(4395): warning C4003: not enough arguments for function-like macro invocation 'RUBY_DTRACE_GC_SWEEP_PAGE'
Expand the probe invocation through an identity macro so that the
arguments are re-scanned and split properly.
Interix (Windows Services for UNIX / Subsystem for UNIX-based Applications) was removed from Windows in 8.1 and Server 2012 R2 and its support has long ended, so Ruby can no longer be built or run on it. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
FreeBSD's libdtrace dt_modtext() mishandles is-enabled probes on aarch64: it rewrites the call site to a bare nop without clearing the return register (unlike x86, which writes an eax-zeroing sequence), so RUBY_DTRACE_*_ENABLED() reads whatever was in w0 and fires probes with no consumer attached. rb_dtrace_setup() then allocates a fresh class-name String via rb_class_path() on every cfunc call, which breaks the object allocation count assertions in TestStringMemory, TestHashOnly and TestGc#test_stat on the freebsd15-arm builder since 5b5d1fb re-enabled DTrace on FreeBSD. Scope the override to aarch64 only; amd64 FreeBSD uses the correct x86 path and stays enabled.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )