Skip to content

Clarify ERB escape allocation alert as a false positive - #23

Draft
nobu with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-code-scanning-alert-1482
Draft

nobu with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-code-scanning-alert-1482

Conversation

Copilot AI commented Oct 1, 2026 •

Copy link
Copy Markdown
  • Assessment: The reported alloca-in-loop alert is a false positive. ALLOCV_N runs only when buf is null; subsequent loop iterations reuse the allocated buffer.
  • Code changes: None. The alert does not indicate repeated stack allocation.
if (!buf) {
    buf = ALLOCV_N(char, vbuf, escaped_length(str));
}

Copilot AI assigned Copilot and nobu Oct 1, 2026
Copilot AI changed the title [WIP] Fix code scanning alert #1482 Clarify ERB escape allocation alert as a false positive Oct 1, 2026
Copilot AI requested a review from nobu October 1, 2026 13:11
@nobu
nobu force-pushed the master branch 2 times, most recently from 7cbd70e to 528c704 Compare October 1, 2026 15:14
Copilot AI and others added 2 commits October 2, 2026 13:44
Co-authored-by: nobu <nobu@ruby-lang.org>
@nobu
nobu force-pushed the copilot/fix-code-scanning-alert-1482 branch from 806befb to 37be5ca Compare October 2, 2026 04:45
@github-actions github-actions Bot added the jit label Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants