Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking β€œ#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update test.py #1969

Merged
merged 1 commit into from
Jan 18, 2021
Merged

Update test.py #1969

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 in Test file

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhancing security by using safe YAML loader in testing script.

πŸ“Š Key Changes

  • Switched from yaml.FullLoader to yaml.SafeLoader in the test script.

🎯 Purpose & Impact

  • πŸ›‘οΈ Increased Security: yaml.SafeLoader is used to load a YAML file in a way that prevents the execution of arbitrary code, which could be an exploit vector if untrusted YAML files were loaded.
  • πŸš€ Stability for Users: This change prioritizes user safety without affecting the functionality of the test script, ensuring that users can run tests on their datasets securely.

@glenn-jocher glenn-jocher merged commit 91c30e4 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants