Skip to content

Fix directory traversal when workdir path is not readable#3080

Merged
ljharb merged 1 commit intonvm-sh:masterfrom
zlotnleo:master
Apr 10, 2023
Merged

Fix directory traversal when workdir path is not readable#3080
ljharb merged 1 commit intonvm-sh:masterfrom
zlotnleo:master

Conversation

@zlotnleo
Copy link
Contributor

@zlotnleo zlotnleo commented Apr 8, 2023

This PR fixes an issue where pwd can return "." due to insufficient permissions, e.g. when switching users using the su command, which causes an infinite loop in the nvm_find_project_dir and nvm_find_up functions.

@ljharb
Copy link
Member

ljharb commented Apr 8, 2023

huh, never seen that. I’d prefer to have a test along with this fix.

@ljharb ljharb merged commit 0d9b5c2 into nvm-sh:master Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments