Skip to content

buffer: adjust validation to account for buffer.kMaxLength - #35134

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:buffer-validation-uint32
Closed

buffer: adjust validation to account for buffer.kMaxLength#35134
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:buffer-validation-uint32

Conversation

@addaleax

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the buffer Issues and PRs related to the buffer subsystem. label Sep 9, 2020
@addaleaxaddaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2020
@nodejs-github-bot

nodejs-github-bot commented Sep 9, 2020

Copy link
Copy Markdown
Collaborator

@addaleaxaddaleax added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 12, 2020
@github-actionsgithub-actionsBot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 12, 2020
@github-actions

Copy link
Copy Markdown
Contributor

Landed in 3c56463

nodejs-github-bot pushed a commit that referenced this pull request Sep 12, 2020
PR-URL: #35134
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
ruyadorno pushed a commit that referenced this pull request Sep 17, 2020
PR-URL: #35134
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
@ruyadornoruyadorno mentioned this pull request Sep 21, 2020
4 tasks
@addaleax
addaleax deleted the buffer-validation-uint32 branch October 12, 2020 10:40
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#35134
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bufferIssues and PRs related to the buffer subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@addaleax@nodejs-github-bot@jasnell@BridgeAR@ZYSzys