vfs: reject statfs for missing paths by trivikr · Pull Request #65693 · nodejs/node · GitHub
Skip to content

vfs: reject statfs for missing paths - #65693

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:vfs-statfs-non-existent-paths
Sep 9, 2026
Merged

nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:vfs-statfs-non-existent-paths

Conversation

@trivikr

@trivikr trivikr commented Sep 1, 2026

Copy link
Copy Markdown
Member

Fixes: #65692

Validate mounted VFS paths before returning synthetic statfs data so fs.statfsSync() and fs.promises.statfs() report ENOENT for missing paths. Forward validation errors asynchronously to fs.statfs() callbacks.


Assisted-by: codex:gpt-5.6-sol

@nodejs-github-bot nodejs-github-bot added fs Issues and PRs related to file-system APIs and the fs module. needs-ci PRs that need a full CI run. vfs Issues and PRs related to the virtual filesystem subsystem. labels Sep 1, 2026
@trivikr trivikr added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Sep 1, 2026
@codecov

codecov Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

@trivikr
trivikr force-pushed the vfs-statfs-non-existent-paths branch from 859f592 to 40bd7ae Compare September 2, 2026 00:07
@trivikr
trivikr requested a review from mcollina September 7, 2026 19:34
Validate mounted VFS paths before returning synthetic statfs data
so fs.statfsSync() and fs.promises.statfs() report ENOENT for missing
paths. Forward validation errors asynchronously to fs.statfs()
callbacks.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: codex:gpt-5.6-sol
@trivikr
trivikr force-pushed the vfs-statfs-non-existent-paths branch from 40bd7ae to a58d860 Compare September 7, 2026 21:03
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Sep 8, 2026
@nodejs-github-bot

This comment was marked as outdated.

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@trivikr trivikr added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. labels Sep 8, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 1dd48de into nodejs:main Sep 9, 2026
72 of 73 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 9, 2026
aduh95 pushed a commit that referenced this pull request Sep 16, 2026
Validate mounted VFS paths before returning synthetic statfs data
so fs.statfsSync() and fs.promises.statfs() report ENOENT for missing
paths. Forward validation errors asynchronously to fs.statfs()
callbacks.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: codex:gpt-5.6-sol
PR-URL: #65693
Fixes: #65692
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
@trivikr
trivikr deleted the vfs-statfs-non-existent-paths branch September 22, 2026 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. fs Issues and PRs related to file-system APIs and the fs module. needs-ci PRs that need a full CI run. vfs Issues and PRs related to the virtual filesystem subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vfs: statfs succeeds for nonexistent paths

4 participants