Skip to content

fix: Path Traversal Bug in SageMaker Server - #8659

Merged
whoisj merged 6 commits into
mainfrom
jwyman/tri-667
Feb 25, 2026
Merged

fix: Path Traversal Bug in SageMaker Server#8659
whoisj merged 6 commits into
mainfrom
jwyman/tri-667

Conversation

@whoisj

@whoisj whoisj commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

This change prevents the /dev/, /proc/, or /sys/ folders being used to load models from in SagemakerAPIServer to avoid issues with interacting with pseudo-filesystem paths.

This change prevents the /dev/, /proc/, or /sys/ folders being used to load models from in
`SagemakerAPIServer` to avoid issues with interacting with pseudo-filesystem paths.
@yinggeh

yinggeh commented Feb 13, 2026

Copy link
Copy Markdown
Contributor

Please keep linear ticket private. It's not supposed to be exposed to the public.

Comment thread src/sagemaker_server.cc Outdated
Comment thread src/sagemaker_server.cc Outdated
@whoisj
whoisj force-pushed the jwyman/tri-667 branch 3 times, most recently from ae0487a to 3f371b5 Compare February 18, 2026 22:23
@whoisj
whoisj requested a review from yinggeh February 18, 2026 22:47
@whoisj
whoisj force-pushed the jwyman/tri-667 branch 2 times, most recently from c7c0de0 to 39ecd25 Compare February 19, 2026 20:32
Comment thread src/sagemaker_server.cc Outdated
Comment thread src/sagemaker_server.cc Outdated
whoisj and others added 2 commits February 23, 2026 13:29
Co-authored-by: Yingge He <157551214+yinggeh@users.noreply.github.com>
Co-authored-by: Yingge He <157551214+yinggeh@users.noreply.github.com>
@whoisj
whoisj merged commit a202ae5 into main Feb 25, 2026
3 checks passed
@whoisj
whoisj deleted the jwyman/tri-667 branch February 25, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants