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

Fixed Arbitrary Code execution #1

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

Fixed Arbitrary Code execution #1

wants to merge 3 commits into from

Conversation

B3EF
Copy link

@B3EF B3EF commented Jan 31, 2021

📊 Metadata *

Fixed Arbitrary Code execution

Bounty URL:https://www.huntr.dev/bounties/1-other-baselines

⚙️ Description *

OpenAI Baselines: high-quality implementations of reinforcement learning algorithms

💻 Technical Description *

Vulnerability description untrusted loading of data by the pickle.load function leading to Arbitrary code execution.

🐛 Proof of Concept (PoC) *

Screenshot from 2021-01-31 13-52-29

🔥 Proof of Fix (PoF) *

Screenshot from 2021-01-31 13-57-31

👍 User Acceptance Testing (UAT)

Forbidden is displayed when a code execution payload is passed

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

👋 Hello, @pzhokhov - @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!

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

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.

3 participants