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

@addaleax addaleax commented Sep 9, 2020

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
@nodejs-github-bot nodejs-github-bot added the buffer Issues and PRs related to the buffer subsystem. label Sep 9, 2020
@addaleax addaleax added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2020
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 9, 2020
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

nodejs-github-bot commented Sep 9, 2020

@addaleax addaleax requested a review from jasnell September 11, 2020 08:25
@addaleax addaleax added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 12, 2020
@github-actions github-actions Bot 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

@github-actions github-actions Bot closed this Sep 12, 2020
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>
@ruyadorno ruyadorno 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buffer Issues and PRs related to the buffer subsystem.

5 participants