Go执行外部命令函数偶现“文件已关闭”错误排查求助

错误原因分析 #

你的代码里存在几个问题，共同导致了这个偶发的读取错误：

1. 参数传递错误
   exec.CommandContext的第三个参数是可变参数列表，你直接传入单个字符串cmdArgs会被当作命令的一个完整参数，而不是拆分后的多个参数。如果命令需要多个参数（比如ls -l），这种传递方式会导致命令执行失败，提前退出并关闭stdout管道，进而引发scanner读取错误。

2. err变量被遮蔽
   代码里连续两次用:=声明err变量：

stdout, err := cmd.StdoutPipe()
// ...
err := cmd.Start()

第二次声明的err会遮蔽外层的err，导致StdoutPipe的错误被完全忽略。如果StdoutPipe调用失败（比如权限问题），后续scanner操作会基于无效的管道，触发读取错误。

3. 竞态条件导致管道提前关闭
   虽然你用了wg.Wait()等待扫描协程，但cmd.Wait()在独立协程中执行。当外部命令快速退出时，stdout管道可能在扫描协程完成最后一次Scan()前被关闭。此时scanner.Scan()返回false，scanner.Err()会捕获到管道已关闭的错误。

4. 忽略命令执行错误
   代码完全没处理done通道里的cmd.Wait()结果，即使命令执行失败（比如找不到命令、参数非法），函数也会返回nil，同时异常关闭的管道会触发scanner的读取错误。

修复后的代码示例 #

import (
    "bufio"
    "context"
    "log"
    "os/exec"
    "sync"
    "time"
    "strings"
)

func RunCommand(cmdPath string, cmdArgs string) error {
    ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second)
    defer cancel()

    // 拆分参数，处理带空格的参数可以用shellquote.Split（需引入github.com/kballard/go-shellquote）
    args := strings.Fields(cmdArgs)
    cmd := exec.CommandContext(ctx, cmdPath, args...)

    stdout, err := cmd.StdoutPipe()
    if err != nil {
        return err
    }
    scanner := bufio.NewScanner(stdout)

    // 修复err变量遮蔽问题，检查Start错误
    if err := cmd.Start(); err != nil {
        return err
    }

    var wg sync.WaitGroup
    wg.Add(1)

    go func() {
        defer wg.Done()
        for scanner.Scan() {
            doSomething(scanner.Text())
        }
        if err := scanner.Err(); err != nil {
            log.Printf("Error reading stdout: %v", err)
        }
    }()

    // 先等待命令完成，再等待扫描协程读完所有输出
    cmdWaitErr := cmd.Wait()
    wg.Wait()

    // 返回命令执行的错误
    return cmdWaitErr
}

关键修复点说明 #

• 正确拆分参数：用strings.Fields()拆分参数，如果需要处理包含空格或特殊字符的参数，建议使用shellquote.Split来安全解析。
• 检查所有错误：修复err变量遮蔽问题，确保StdoutPipe和cmd.Start()的错误都被捕获并返回。
• 调整等待顺序：先调用cmd.Wait()等待命令执行完毕，再等待扫描协程读完所有stdout输出，避免管道在读取过程中被意外关闭。
• 返回命令执行错误：将cmd.Wait()的结果作为函数返回值，让调用者知道命令是否执行成功。

内容的提问来源于stack exchange，提问作者Dinho