-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Security Fix for Arbitrary Code Execution - huntr.dev #892
base: master
Are you sure you want to change the base?
Conversation
Fixed arbitary code execution on tensorflow/tpu
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@Asjidkalam - are you able to sign the CLA? Thanks! 🍰 |
@googlebot I signed it! |
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
2 similar comments
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
https://huntr.dev/users/Asjidkalam has fixed the Arbitrary Code Execution vulnerability 🔨. Think you could fix a vulnerability like this?
Get involved at https://huntr.dev/
Q | A
Version Affected | ALL
Bug Fix | YES
Original Pull Request | 418sec#1
Vulnerability README | https://github.com/418sec/huntr/blob/master/bounties/other/tpu/1/README.md
User Comments:
📊 Metadata *
Arbitrary code exec vulnerability
Bounty URL: https://www.huntr.dev/bounties/1-other-tpu
⚙️ Description *
Arbitrary Code Excecution in tensorflow/tpu.This repository is a collection of reference models and tools used with Cloud TPUs
💻 Technical Description *
This package was vulnerable to Arbitrary code execution due to a use of a known vulnerable function load() in pyyaml. Changing that to safe_load or using SafeLoader will fix the issue.
🐛 Proof of Concept (PoC) *
Install the package and run the below code:
🔥 Proof of Fix (PoF) *
After applying the fix, run the PoC again,
calc
wont pop and no code will be executed. Hence code exec is mitigated.👍 User Acceptance Testing (UAT)
Only
safe_load
is used, which is the safer function, no breaking changes introduced.