fs: close dir before throwing if `options.bufferSize` is invalid by LiviaMedeiros · Pull Request #58856 · nodejs/node · GitHub
Skip to content

fs: close dir before throwing if options.bufferSize is invalid#58856

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
LiviaMedeiros:fs-dir-close-on-invalid-buffer
Jul 6, 2025
Merged

fs: close dir before throwing if options.bufferSize is invalid#58856
nodejs-github-bot merged 1 commit intonodejs:mainfrom
LiviaMedeiros:fs-dir-close-on-invalid-buffer

Conversation

@LiviaMedeiros
Copy link
Copy Markdown
Member

@LiviaMedeiros LiviaMedeiros commented Jun 26, 2025

Fixes: #58854

Test blocked on: #58855

@LiviaMedeiros LiviaMedeiros added the blocked PRs that are blocked by other issues or PRs. label Jun 26, 2025
@nodejs-github-bot nodejs-github-bot added fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. labels Jun 26, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 26, 2025

@LiviaMedeiros LiviaMedeiros force-pushed the fs-dir-close-on-invalid-buffer branch from 2155c48 to 67ea569 Compare June 29, 2025 05:53
@LiviaMedeiros LiviaMedeiros removed the blocked PRs that are blocked by other issues or PRs. label Jun 29, 2025
@LiviaMedeiros LiviaMedeiros force-pushed the fs-dir-close-on-invalid-buffer branch from 67ea569 to eadbae5 Compare June 29, 2025 05:57
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@juanarbol juanarbol added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 6, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 6, 2025
@nodejs-github-bot nodejs-github-bot merged commit 9ab9763 into nodejs:main Jul 6, 2025
64 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

RafaelGSS pushed a commit that referenced this pull request Jul 8, 2025
PR-URL: #58856
Fixes: #58854
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 21, 2025
PR-URL: #58856
Fixes: #58854
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 24, 2025
PR-URL: #58856
Fixes: #58854
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fs: opendirSync() leaks directory handle on error

5 participants