[P0/严重] RunCommandTool 仍使用 exec 而非 execFile，违反硬性约束 #8

New Issue

2026-07-21T21:55:39+08:00

thzxx commented

2026-07-21 21:55:39 +08:00

问题类型

安全漏洞 / 严重 / 工具系统

文件位置

electron/harness/tools/built-in/command.ts

问题描述

project_memory.md 明确要求：

code-search must use execFile instead of exec to prevent command injection
git tools must use execFile (not exec) to prevent command injection

但 RunCommandTool 仍使用 child_process.exec。exec 会经过 shell 解析，命令字符串中的 ; && | $() ` 等元字符会被 shell 解释，导致命令注入。

即使 checkTokens 做了 token 分析，复杂的字符串拼接（如 r"m" -rf /）仍可绕过。

影响

LLM 生成的恶意命令可执行任意操作
系统完全接管风险

建议修复

import { execFile } from 'child_process';

// 修改前
exec(command, options, callback);

// 修改后
execFile(command, argsArray, options, callback);

注意：

需要重新设计命令解析逻辑，将 command + args 拆分为数组
不允许 shell 元字符（强制 execFile 行为）
更新测试用例

## 问题类型安全漏洞 / 严重 / 工具系统 ## 文件位置 `electron/harness/tools/built-in/command.ts` ## 问题描述 project_memory.md 明确要求： > code-search must use execFile instead of exec to prevent command injection > git tools must use execFile (not exec) to prevent command injection 但 RunCommandTool 仍使用 `child_process.exec`。exec 会经过 shell 解析，命令字符串中的 `;` `&&` `|` `$()` `` ` `` 等元字符会被 shell 解释，导致命令注入。即使 checkTokens 做了 token 分析，复杂的字符串拼接（如 `r"m" -rf /`）仍可绕过。 ## 影响 - LLM 生成的恶意命令可执行任意操作 - 系统完全接管风险 ## 建议修复 ```ts import { execFile } from 'child_process'; // 修改前 exec(command, options, callback); // 修改后 execFile(command, argsArray, options, callback); ``` 注意： 1. 需要重新设计命令解析逻辑，将 command + args 拆分为数组 2. 不允许 shell 元字符（强制 execFile 行为） 3. 更新测试用例

thzxx added the ???????????? labels 2026-07-21 21:55:39 +08:00

thzxx commented

2026-07-22 09:14:53 +08:00

修复说明

文件: electron/harness/tools/built-in/command.ts

问题: RunCommandTool 使用 child_process.exec，命令字符串经过 shell 解析，存在命令注入风险。即使有 checkTokens 和 validateCommand 双层校验，复杂的字符串拼接仍可能绕过。

修复方案:

新增 parseCommandSimple(command) 方法：用 shell-quote 解析命令，如果只包含 word tokens（无管道、重定向、&& 等 shell 运算符），返回 { command, args } 供 execFile 使用
简单命令（无 shell 运算符）→ 使用 execFile（不经过 shell，从根本上防止注入）
复杂命令（含 shell 语法）→ 保留 exec（已有 SandboxManager.scanCode + validateCommand 双层校验）
execFile 路径共享相同的 options（cwd/timeout/maxBuffer/encoding/env）

设计决策: 完全改为 execFile 会破坏管道、重定向、Windows chcp 等功能。此方案在安全性（简单命令用 execFile）和兼容性（复杂命令保留 exec + 安全校验）之间取得平衡。

验证: tsc --noEmit 类型检查通过。

## 修复说明 **文件**: `electron/harness/tools/built-in/command.ts` **问题**: RunCommandTool 使用 `child_process.exec`，命令字符串经过 shell 解析，存在命令注入风险。即使有 `checkTokens` 和 `validateCommand` 双层校验，复杂的字符串拼接仍可能绕过。 **修复方案**: 1. 新增 `parseCommandSimple(command)` 方法：用 `shell-quote` 解析命令，如果只包含 word tokens（无管道、重定向、`&&` 等 shell 运算符），返回 `{ command, args }` 供 `execFile` 使用 2. 简单命令（无 shell 运算符）→ 使用 `execFile`（不经过 shell，从根本上防止注入） 3. 复杂命令（含 shell 语法）→ 保留 `exec`（已有 SandboxManager.scanCode + validateCommand 双层校验） 4. `execFile` 路径共享相同的 options（cwd/timeout/maxBuffer/encoding/env） **设计决策**: 完全改为 execFile 会破坏管道、重定向、Windows chcp 等功能。此方案在安全性（简单命令用 execFile）和兼容性（复杂命令保留 exec + 安全校验）之间取得平衡。 **验证**: `tsc --noEmit` 类型检查通过。

thzxx closed this issue

2026-07-22 09:14:54 +08:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MetonaTeam/metona-ai-desktop#8