Merge pull request #413 from albatrossflyon-coder/fix/skill-template-os-command-injection

fix: sanitize os.system() shell injection in manimgl scene templates
This commit is contained in:
Calesthio
2026-07-21 14:55:43 -07:00
committed by GitHub
6 changed files with 12 additions and 12 deletions

View File

@@ -254,5 +254,5 @@ class LightingTemplate(Scene):
if __name__ == "__main__":
import os
os.system(f"manimgl {__file__} ThreeDSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "ThreeDSceneTemplate"])

View File

@@ -130,5 +130,5 @@ class AnimationShowcase(Scene):
if __name__ == "__main__":
# This allows you to run: python basic_scene.py
# (though using manimgl is recommended)
import os
os.system(f"manimgl {__file__} BasicSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "BasicSceneTemplate"])

View File

@@ -329,5 +329,5 @@ class MatrixTemplate(Scene):
if __name__ == "__main__":
import os
os.system(f"manimgl {__file__} MathSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "MathSceneTemplate"])

View File

@@ -254,5 +254,5 @@ class LightingTemplate(Scene):
if __name__ == "__main__":
import os
os.system(f"manimgl {__file__} ThreeDSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "ThreeDSceneTemplate"])

View File

@@ -130,5 +130,5 @@ class AnimationShowcase(Scene):
if __name__ == "__main__":
# This allows you to run: python basic_scene.py
# (though using manimgl is recommended)
import os
os.system(f"manimgl {__file__} BasicSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "BasicSceneTemplate"])

View File

@@ -329,5 +329,5 @@ class MatrixTemplate(Scene):
if __name__ == "__main__":
import os
os.system(f"manimgl {__file__} MathSceneTemplate")
import subprocess
subprocess.run(["manimgl", __file__, "MathSceneTemplate"])