T09 · Insecure Skill Coding Practices
- Location
scripts/compare_docx.py:614- Finding
Predictable Temporary Files Permit File Overwrite and Deletion
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill appears intended for DOCX duplicate checking, but its file handling and DOCX parsing have review-worthy risks for local file loss or resource exhaustion.
Install only if you trust the DOCX files you will process and can run it in a controlled working/output directory. Avoid pointing output at directories containing important files, especially files named like __tmp_<document>.docx, and be cautious with large or untrusted DOCX files.
scripts/compare_docx.py:614Predictable Temporary Files Permit File Overwrite and Deletion
scripts/compare_docx.py:166Unbounded DOCX Archive Decompression Enables Memory Exhaustion
The skill documentation declares no explicit tool scope or permission boundaries, yet the described functionality clearly writes output files such as highlighted DOCX documents and text reports. Without an explicit permissions/allowed-tools declaration, an agent may invoke file-writing behavior with broader-than-expected filesystem access, increasing the risk of unintended overwrites or writing to sensitive locations if parameters are user-controlled.
The module docstring, usage text, and all user-facing descriptions are written only in Chinese, and later CLI/help messages are likewise Chinese-only. This imposes a specific language/locale on users without offering opt-in or documenting a justified region-specific limitation, which matches the language-policy violation criteria.
No suspicious patterns detected.