Skip to content

fix: remove spurious 'install' package name from pip command#89

Merged
wenyhu-google merged 1 commit intoGoogleCloudPlatform:mainfrom
x4v13r64:fix/remove-spurious-install-package
Mar 25, 2026
Merged

fix: remove spurious 'install' package name from pip command#89
wenyhu-google merged 1 commit intoGoogleCloudPlatform:mainfrom
x4v13r64:fix/remove-spurious-install-package

Conversation

@x4v13r64
Copy link
Copy Markdown
Contributor

The GPU training Dockerfile contains a duplicated install keyword that pip interprets as a package name:

RUN pip3 install install torchvision pillow

install is not registered on PyPI. An attacker who registers the name at any version will have their package fetched and executed during container build — as root, inside an NVIDIA CUDA environment with access to build credentials and any service account tokens available to the Docker daemon.

This PR removes the extraneous install argument, correcting the command to:

RUN pip3 install torchvision pillow

@x4v13r64
Copy link
Copy Markdown
Contributor Author

@wenyhu-google can you review please?

@x4v13r64
Copy link
Copy Markdown
Contributor Author

@wenyhu-google looks like a repo owner needs to merge this too

@wenyhu-google wenyhu-google merged commit ecfc90b into GoogleCloudPlatform:main Mar 25, 2026
2 checks passed
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