Back to skill

Security audit

Quodd

Security checks for vulnerabilities and agentic risk

Overview

This skill does what it claims by fetching Quodd stock quotes, but users should understand that it sends Quodd credentials to Quodd and caches a short-lived token in a local plaintext file.

Install only if you are comfortable providing Quodd credentials through environment variables and storing a Quodd API token locally for up to 20 hours. On shared or less trusted systems, prefer using --no-cache or tightening permissions on ~/.openclaw/credentials and the token file.

Vulnerability Patterns
  • Insecure Skill Coding PracticesFinds exploitable flaws such as hardcoded secrets or command injection
  • Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
  • Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
  • Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
  • Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally
Findings (1)

T09 · Insecure Skill Coding Practices

Warning
Location
scripts/quote.py:65
Finding

Authentication Token Cached Without Explicit Access Controls or Symlink Protection

Content
View full analysis
Remediation
View remediation
Vulnerability Patterns
  • Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
  • Trigger AbuseOverly Broad Trigger, Shadow Command Trigger, Keyword Baiting Trigger
  • Taint TrackingDirect Taint Flow, Variable-Mediated Taint Flow, Credential Exfiltration Chain
  • MCP Least PrivilegeUnderdeclared Capability, Wildcard Permission, Missing Permission Declaration
  • Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
Findings (5)

Tainted flow: 'req' from os.environ.get (line 183, credential/environment) → urllib.request.urlopen (network output)

Critical
Category
Data Flow
Confidence
90% confidence
Finding

Credentials or environment variables flow to a network sink. This is a high-confidence indicator of credential exfiltration.

Content

Scanner excerpt · scripts/quote.py (reported line 91)May include surrounding context.

python
req = Request(TOKEN_URL, data=b"", headers=headers, method="POST")

    try:
        with urlopen(req, timeout=TIMEOUT_SECONDS) as response:
            data = json.loads(response.read().decode("utf-8"))
            return data.get("token")
    except HTTPError as e:

Tainted flow: 'req' from os.environ.get (line 183, credential/environment) → urllib.request.urlopen (network output)

Critical
Category
Data Flow
Confidence
90% confidence
Finding

Credentials or environment variables flow to a network sink. This is a high-confidence indicator of credential exfiltration.

Content

Scanner excerpt · scripts/quote.py (reported line 186)May include surrounding context.

python
req = Request(TOKEN_URL, data=b"", headers=headers, method="POST")

    try:
        with urlopen(req, timeout=TIMEOUT_SECONDS) as response:
            data = json.loads(response.read().decode("utf-8"))
            return data.get("token")
    except HTTPError as e:

Undeclared Tool Scope

Medium
Category
MCP Least Privilege
Confidence
92% confidence
Finding

The skill declares access to environment variables and its documented behavior implies network access plus credential/token handling, but it does not declare an explicit tool scope such as permissions or allowed-tools. This weakens policy enforcement and reviewability because the runtime capabilities exceed what is transparently constrained in the manifest, increasing the chance of unintended secret access, file writes, or outbound requests.

Content

No source excerpt is available for this finding.

Vague Triggers

Medium
Category
Not specified by scanner
Confidence
89% confidence
Finding

The manifest description says to use the skill when the user asks for 'stock prices, quotes, market data, or ticker information.' Terms like 'market data' and 'ticker information' are broad and lack clear boundaries or exclusion conditions, which could cause the skill to trigger for general finance questions beyond real-time US equity quotes.

Content

No source excerpt is available for this finding.

Missing User Warnings

Medium
Category
Not specified by scanner
Confidence
93% confidence
Finding

The skill persists a Quodd API token to a predictable file under the user's home directory without setting restrictive permissions or warning the user. On multi-user systems or misconfigured environments, another local process or user could read the token and use it to access the Quodd account until expiry.

Content

No source excerpt is available for this finding.

Static analysis

No suspicious patterns detected.