T09 · Insecure Skill Coding Practices
- Location
scripts/search.sh:18- Finding
Arbitrary Python Code Execution Through Search Query Injection
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This memory skill has useful, purpose-related features, but its privacy claims and several scripts are too broad or unsafe for automatic trust.
Review this skill before installing. Do not rely on the zero-upload or offline claims if Baidu Embedding is enabled, avoid storing secrets or regulated data until data flows are documented, and do not run the permission repair, cleanup, verification, or external-script refresh paths in a shared /root/clawd workspace without auditing and scoping them first.
scripts/search.sh:18Arbitrary Python Code Execution Through Search Query Injection
scripts/fix.sh:26Recursive Permission Widening Across the Entire Workspace
memory_maintenance.sh:12Unscoped Cleanup Deletes Files Belonging to Unrelated Components
scripts/secure-memory.sh:33Execution of Unverified Scripts Located Outside the Skill Package
SKILL.md:15Local-Only Privacy Claims Conflict with External Baidu API Processing
package.json:23Core Memory Dependencies Use Mutable Versions Without Integrity Locking
scripts/memory_skill_full_verification.sh:95Verification Hook Permanently Modifies the Memory Database
The documentation makes strong privacy and security claims such as '完全本地化', '零上传', and '离线可用', but elsewhere instructs users to configure Baidu Embedding API credentials for semantic search. If semantic search sends memory content or queries to Baidu, users may disclose sensitive data under a false assumption of offline-only operation. In a memory system intended to store personal or security-relevant notes, this mismatch is especially dangerous because users are likely to entrust high-sensitivity data to it.
The document contains contradictory privacy assurances: it promises fully offline, zero-upload operation while also presenting Baidu Embedding API integration as part of the system. Contradictory trust messaging in security-sensitive documentation can cause operators to deploy the skill under incorrect assumptions, exposing private notes, credentials, or internal records to third-party processing. Because this is a 'secure memory' skill, inaccurate privacy claims directly affect user threat modeling and safe use.
The security section states there is '无网络传输' and that the system is '完全离线使用', yet the same document says semantic search uses Baidu Cloud Embedding-V1. This contradiction can mislead users into believing no external transmission occurs, when in practice cloud embedding likely requires sending text-derived data off-device. For a privacy-focused memory product, misleading deployment and data-flow claims materially increase confidentiality risk.
Although the command targets a specific directory, documenting a raw 'rm -rf' deletion command encourages copy-paste execution of a destructive operation. In the context of a local memory/privacy skill, this is more dangerous because users may erase stored memories, configuration, or adjacent data if the path is modified or misunderstood.
# 删除skill目录
rm -rf /root/clawd/skills/secure-memory-stack
# 如果通过ClawdHub安装
clawdhub uninstall secure-memory-stack
Although the command targets a specific directory, documenting a raw 'rm -rf' deletion command encourages copy-paste execution of a destructive operation. In the context of a local memory/privacy skill, this is more dangerous because users may erase stored memories, configuration, or adjacent data if the path is modified or misunderstood.
# 删除skill目录
rm -rf /root/clawd/skills/secure-memory-stack
# 如果通过ClawdHub安装
clawdhub uninstall secure-memory-stack
The guide states that all data is stored locally and that there is no risk of external data leakage, yet earlier sections require Baidu Embedding API access and network connectivity. This creates a misleading security claim that could cause operators to send sensitive memory contents to a third-party service under the false assumption that the system is fully local.
The security section claims there is no external data leakage risk while the same document describes dependence on an external Baidu API. Such contradictory security documentation can lead users to store confidential information in the memory system without understanding that data may leave the local environment for embedding generation or related API interactions.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The skill claims security/privacy protections and Git Notes/Baidu integration that are reportedly not implemented, while still performing scaffold and repository setup. False security assurances are dangerous because users may store sensitive information believing controls exist when they do not.
The documentation promotes semantic search and zero upload without warning that embedding-based search may require sending user content externally. This omission undermines informed consent and is especially severe because the skill's domain is long-term memory, which often contains sensitive personal or operational data.
The documentation states both 'fully local / zero upload' and use of Baidu Embedding API, which inherently requires sending data to an external service. In a privacy-focused memory skill, this contradiction can cause users to expose highly sensitive notes, preferences, or session data under false privacy expectations.
The security section promises local-only storage and no external transmission, but the same document instructs users to configure and troubleshoot an external Baidu API. Misrepresenting data-flow guarantees in a memory system is dangerous because it directly affects consent and handling of potentially confidential information.
The README makes strong privacy and security claims such as '绝不上传', '无需网络连接即可使用全部功能', and '零数据泄露风险', while also advertising Baidu Embedding-based semantic search. If semantic search can rely on a third-party API, user memory content or queries may be transmitted off-device, making the privacy claims materially misleading and creating risk of unintended disclosure of sensitive data.
The document states that all data is stored locally and that there is no risk of external data leakage, yet earlier sections require Baidu Embedding API access for semantic processing. This is a materially misleading security claim because user content or derived text may be transmitted to a third-party service, causing operators to handle sensitive memory data under false privacy assumptions.
The script asserts '网络传输: 无 (完全离线)' and '隐私保护: 最高级别' even though it checks for Baidu API credentials earlier and instantiates the Baidu embedding component, which strongly suggests dependence on an external service. This can mislead users into believing the system never uses networked third-party processing, creating a privacy and trust issue in a skill explicitly marketed as secure and local.
The changelog heading and descriptive text are written in Chinese, and the file provides no indication that language selection is optional or that the skill is intentionally limited to a Chinese-speaking audience. Under the policy, forcing a specific language without user opt-in is a natural-language locale violation.
The changelog simultaneously claims integration with a Baidu Embedding semantic search layer and also asserts 'zero data upload' and 'no network transmission'. For a security/privacy-focused memory system, this contradiction can mislead users into believing data never leaves the device when API-backed embedding generation may require remote transmission, creating privacy and trust risks.
The file documents both 'local storage, no network transmission' and a 'Baidu Embedding API integration wizard', which is an active contradiction in a security-sensitive feature description. Users may configure the system under false assumptions about data residency, potentially exposing sensitive memory contents or metadata to a third-party service.
No suspicious patterns detected.