Skip to content

fix(buildkite): download codecov to secure temp dir to prevent bypass#387

Open
jcc-google wants to merge 2 commits into
vllm-project:mainfrom
jcc-google:fix/codecov-binary-bypass-b510376198
Open

fix(buildkite): download codecov to secure temp dir to prevent bypass#387
jcc-google wants to merge 2 commits into
vllm-project:mainfrom
jcc-google:fix/codecov-binary-bypass-b510376198

Conversation

@jcc-google

Copy link
Copy Markdown
Contributor

Always download the official codecov CLI to a secure, temporary directory and execute it from there. This removes the bypass check that allowed executing a local, potentially malicious 'codecov' binary from the untrusted workspace.

BUG=b/510376198
TAG=agy
CONV=f5118c4a-3577-4dc6-a4b6-1f2abb990935

Always download the official codecov CLI to a secure, temporary directory
and execute it from there. This removes the bypass check that allowed
executing a local, potentially malicious 'codecov' binary from the
untrusted workspace.

Signed-off-by: Jincheng Chen <chenjincheng@google.com>
BUG=b/510376198
TAG=agy
CONV=f5118c4a-3577-4dc6-a4b6-1f2abb990935
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.

1 participant