T05 · Unauthorized Access and Privilege Escalation
- Location
SKILL.md:278- Finding
Overbroad Secret Loading and Unnecessary Mailbox Access
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill has a clear daily briefing purpose, but it overreaches by loading all OpenClaw environment secrets and reading recent mailbox entries just to verify an email was sent.
Install only if you are comfortable with a scheduled workflow that scrapes WeChat/Sogou, sends mail through your QQ credentials, and currently performs overbroad verification. Before using it, narrow the .env loading to only QQMAIL_USER and QQMAIL_AUTH_CODE, avoid listing unrelated inbox messages, and resolve the conflicting Sogou versus mp.weixin.qq.com link instructions.
SKILL.md:278Overbroad Secret Loading and Unnecessary Mailbox Access
The instruction to export all variables from ~/.openclaw/.env is a credential-access pattern because it loads potentially sensitive secrets into the active environment for a non-minimal operation. If the environment is inspected, logged, inherited by child processes, or reused by other tools, mail credentials and other secrets may be exposed or abused.
发送后必须验证邮件已到达收件箱:
export $(grep -v '^#' ~/.openclaw/.env | xargs)
python3 ~/.openclaw/workspace/skills/qqmail/scripts/qqmail.py inbox --limit 5
This markdown file describes a workflow that sends content to a user's email and relies on QQMAIL_USER and QQMAIL_AUTH_CODE, which affects user data/privacy and involves credential-backed transmission. Under the markdown criteria for SQP-2, the description should explicitly warn users about this behavior and the use of configured mail credentials.
The instructions require writing repository introductions in Chinese, and other sections also prescribe Chinese content, but the skill does not present this as a user-selectable option. This is a natural-language policy issue because it forces a specific language/locale without opt-in or a clearly justified regional constraint.
The skill gives mutually contradictory instructions about which article URLs to include: one section says to use unresolved Sogou redirect links, while later sections require resolved mp.weixin.qq.com links and reject non-WeChat URLs. This inconsistency can cause the agent to bypass its own validation, send broken or misleading links, or leak users into Sogou redirect/CAPTCHA flows instead of the intended destination.
Skill establishes unauthorized persistence across sessions via cron jobs, startup scripts, or state files. Session persistence allows an attacker to maintain access beyond the current interaction.
⚠️ Sogou redirect links CANNOT be resolved to mp.weixin.qq.com URLs. The sogou redirect service also triggers anti-spider. Use the search result summaries as the content source.
For each filtered article, compose a 2-3 sentence summary in Chinese based on the title + summary fields from the JSON output. The search summaries are detailed enough to create informative briefs.
For the article link, use the sogou redirect URL from the search results (users can open it in a browser where the CAPTCHA is manually solvable). Format as:
The post-send verification step instructs exporting all variables from ~/.openclaw/.env, which expands credential exposure far beyond the earlier narrowly scoped recipient requirement. Broadly loading secrets into the shell environment increases the chance of accidental disclosure, misuse by subsequent commands, or unintended dependence on hidden state.
The subject and body template are explicitly fixed in Chinese, and the file does not indicate that language selection is optional or limited to a region-specific use case. This creates a natural-language policy concern because the skill enforces a specific locale without user opt-in.
The instructions explicitly target removal of articles using Chinese-language patterns such as 科普/标题党, which implies the skill is designed around a specific language/locale. Because the file does not state that this is a China-specific or Chinese-content-only workflow, nor offer user opt-in or choice, it creates a natural-language locale policy concern.
The skill explicitly instructs use of a curl-based anti-spider bypass workflow, repeated web scraping with delay-based rate-limit evasion, and writes multiple files under /tmp without any user-facing disclosure or consent. In an agent setting, this creates risk of unauthorized network activity, terms-of-service violations, and unexpected local artifact creation, especially because the instructions are operational rather than merely descriptive.
The merge step writes consolidated JSON files into predictable /tmp paths and silently overwrites or creates files, with broad exception suppression that hides failures. While lower severity than the scraping behavior, this can still cause unintended file clobbering, information exposure through shared temporary directories, and reduced auditability in multi-tenant or agent-executed environments.
No suspicious patterns detected.