fs,win: do not add a second trailing slash in readdir · nodejs/node@85818db · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 750
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 85818db

Browse filesBrowse the repository at this point in the historyBrowse files

Flarna

authored and

aduh95

committed

fs,win: do not add a second trailing slash in readdir

Do not add a trailing slash if it was not removed by resolve in ToNamespacedPath. PR-URL: #59847 Fixes: #58223 Refs: #56110 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>

1 parent 36b68db commit 85818db
Copy full SHA for 85818db

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• src
  • node_file.cc

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎src/node_file.cc‎

Copy file name to clipboardExpand all lines: src/node_file.cc

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -1902,7 +1902,7 @@ static void ReadDir(const FunctionCallbackInfo<Value>& args) {
1902	1902	ToNamespacedPath(env, &path);
1903	1903
1904	1904	#ifdef _WIN32
1905		- if (slashCheck) {
	1905	+ if (slashCheck && !path.ToStringView().ends_with("\\")) {
1906	1906	size_t new_length = path.length() + 1;
1907	1907	path.AllocateSufficientStorage(new_length + 1);
1908	1908	path.SetLengthAndZeroTerminate(new_length);

0 commit comments

Comments

0 (0)