Skip to content

dgram: add nread assertion to UDPWrap::OnRecv - #40295

Closed
RaisinTen wants to merge 1 commit into
nodejs:masterfrom
RaisinTen:dgram/add-nread-assertion-to-OnRecv
Closed

dgram: add nread assertion to UDPWrap::OnRecv#40295
RaisinTen wants to merge 1 commit into
nodejs:masterfrom
RaisinTen:dgram/add-nread-assertion-to-OnRecv

Conversation

@RaisinTen

Copy link
Copy Markdown
Member

This asserts that the number of bytes received by the socket is less
than or equal to the size allocated for the temporary storage.

Signed-off-by: Darshan Sen darshan.sen@postman.com

This asserts that the number of bytes received by the socket is less
than or equal to the size allocated for the temporary storage.
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. dgram Issues and PRs related to the dgram subsystem / UDP. needs-ci PRs that need a full CI run. labels Oct 3, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@RaisinTenRaisinTen added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed needs-ci PRs that need a full CI run. labels Oct 3, 2021
@RaisinTenRaisinTen added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 9, 2021
@github-actionsgithub-actionsBot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 9, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Landed in 7723148...cca9b95

nodejs-github-bot pushed a commit that referenced this pull request Oct 9, 2021
This asserts that the number of bytes received by the socket is less
than or equal to the size allocated for the temporary storage.
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
PR-URL: #40295
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
@RaisinTen
RaisinTen deleted the dgram/add-nread-assertion-to-OnRecv branch October 9, 2021 12:29
targos pushed a commit that referenced this pull request Oct 13, 2021
This asserts that the number of bytes received by the socket is less
than or equal to the size allocated for the temporary storage.
Signed-off-by: Darshan Sen <darshan.sen@postman.com>
PR-URL: #40295
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Oct 14, 2021
2 tasks
@richardlaurichardlau mentioned this pull request Oct 18, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.c++Issues and PRs that require attention from people who are familiar with C++.dgramIssues and PRs related to the dgram subsystem / UDP.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RaisinTen@nodejs-github-bot@jasnell@addaleax@JungMinu