Skip to content
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.

fixed Arbitrary Code Execution #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

fixed Arbitrary Code Execution #2

wants to merge 5 commits into from

Conversation

B3EF
Copy link

@B3EF B3EF commented Jan 26, 2021

📊 Metadata *

Fixed Arbitrary Code Execution

Bounty URL: https://www.huntr.dev/bounties/1-pip-tensorflow/

⚙️ Description *

TensorFlow is an end-to-end open source platform for machine learning. It has a comprehensive, flexible ecosystem of tools, libraries, and community resources that lets researchers push the state-of-the-art in ML and developers easily build and deploy ML-powered applications. It is vulnerable to Arbitrary Code Execution.

💻 Technical Description *

Exploitable to Arbitrary Code Execution due to unsafe yaml.load in its python keras module.

🐛 Proof of Concept (PoC) *

tensorflow1

🔥 Proof of Fix (PoF) *

poftensorfolw1

👍 User Acceptance Testing (UAT)

just changed to safeload .

🔗 Relates to...

same disclosure as 418sec/huntr#1533

huntr-helper pushed a commit to 418sec/huntr that referenced this pull request Jan 26, 2021
@huntr-helper
Copy link
Member

👋 Hello, @tensorflower-gardener - @B3EF has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.

Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.

If you have any questions or need support, come and join us on our community Discord!

@tensorflower-gardener & @B3EF - thank you for your efforts in securing the world’s open source code! 🎉

@JamieSlome
Copy link

@yongtang - attaching for reference.

@B3EF B3EF changed the title fixed ACE fixed Arbitrary Code Execution Jan 26, 2021
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.

4 participants