Fix infinite recursion in `getDirectoryHandle` by kkocdko · Pull Request #145972 · microsoft/vscode · GitHub
Skip to content

Fix infinite recursion in getDirectoryHandle - #145972

Merged
Benjamin Pasero (bpasero) merged 2 commits into
microsoft:mainfrom
kkocdko:main
Mar 24, 2022
Merged

Fix infinite recursion in getDirectoryHandle#145972
Benjamin Pasero (bpasero) merged 2 commits into
microsoft:mainfrom
kkocdko:main

Conversation

@kkocdko

Copy link
Copy Markdown
Contributor

While opening the root dir (or disk root on windows), launch debug, getDirectoryHandle will try to find its parent dir. However dirname(root) returns root itself. This caused UI freezed on web target.

Add equal detection to fix this bug.

While opening the root dir (or disk root on windows), launch debug, `getDirectoryHandle` will try to find its parent dir. However `dirname(root)` returns root itself. This caused UI freezed on web target.

Add equal detection to fix this bug.
@kkocdko

kkocdko (kkocdko) commented Mar 24, 2022

Copy link
Copy Markdown
Contributor Author

@bpasero Benjamin Pasero (bpasero) added this to the March 2022 milestone Mar 24, 2022
@bpasero
Benjamin Pasero (bpasero) merged commit 7046d66 into microsoft:main Mar 24, 2022
@bpasero

Copy link
Copy Markdown
Contributor

Gabriela Araujo Britto (gabritto) pushed a commit to gabritto/vscode that referenced this pull request Mar 25, 2022
While opening the root dir (or disk root on windows), launch debug, `getDirectoryHandle` will try to find its parent dir. However `dirname(root)` returns root itself. This caused UI freezed on web target.

Add equal detection to fix this bug.
@github-actions github-actions Bot locked and limited conversation to collaborators May 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants