Skip to content
This repository was archived by the owner on Feb 25, 2021. It is now read-only.

Better check on symlink location #6

Closed
wants to merge 1 commit into from
Closed

Better check on symlink location #6

wants to merge 1 commit into from

Conversation

parkervcp
Copy link
Member

Moved to checking the file is in the correct base folder and not that the path contained the base folder path.

Moves to checking the file is in the correct base folder.
@DaneEveritt
Copy link
Member

Is this still relevant @parkervcp?

@parkervcp
Copy link
Member Author

I am pretty sure it is.

@DaneEveritt
Copy link
Member

Appears I switched it to HasPrefix already.

https://github.com/pterodactyl/sftp-server/blob/master/src/server/handler.go#L383

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.

2 participants