Skip to content
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

Update plots.py #1968

Merged
merged 1 commit into from
Jan 18, 2021
Merged

Update plots.py #1968

merged 1 commit into from
Jan 18, 2021

Conversation

Anon-Artist
Copy link
Contributor

@Anon-Artist Anon-Artist commented Jan 18, 2021

Fix for Arbitary Code Execution

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved security in hyperparameter loading within YOLOv5 plotting utility.

πŸ“Š Key Changes

  • Changed YAML loader from FullLoader to SafeLoader in the plot_evolution function.

🎯 Purpose & Impact

  • πŸ›‘οΈ Security: The update to SafeLoader prevents potential security vulnerabilities associated with loading YAML files, like executing arbitrary code.
  • 🧬 Reliability: Ensures hyperparameter loading is done safely, promoting more robust and secure code within the YOLOv5 repository.
  • πŸ” User Trust: Users can have increased confidence in the safety of the code they are running.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ‘‹ Hello @Anon-Artist, thank you for submitting a πŸš€ PR! To allow your work to be integrated as seamlessly as possible, we advise you to:

  • βœ… Verify your PR is up-to-date with origin/master. If your PR is behind origin/master update by running the following, replacing 'feature' with the name of your local branch:
git remote add upstream https://github.com/ultralytics/yolov5.git
git fetch upstream
git checkout feature  # <----- replace 'feature' with local branch name
git rebase upstream/master
git push -u origin -f
  • βœ… Verify all Continuous Integration (CI) checks are passing.
  • βœ… Reduce changes to the absolute minimum required for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." -Bruce Lee

@glenn-jocher glenn-jocher merged commit 35400dc into ultralytics:master Jan 18, 2021
KMint1819 pushed a commit to KMint1819/yolov5 that referenced this pull request May 12, 2021
taicaile pushed a commit to taicaile/yolov5 that referenced this pull request Oct 12, 2021
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
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