Go语言使用Goroutines实现ICMP Ping出现重复结果求助

问题根源 #

你遇到的是Go语言循环变量的经典陷阱：循环遍历IP切片时，所有goroutine捕获的是同一个循环变量的引用，而非每次迭代的IP值。当goroutine真正开始执行时，循环已经跑完，变量指向了切片最后一个元素，所以所有输出都是最后一个IP的结果。另外，如果ICMP请求的ID/序列号没有和每个IP绑定，也会导致响应匹配混乱。

具体修复方案 #

1. 捕获循环变量的副本 #

在循环内部创建局部变量，保存当前迭代的IP值，让每个goroutine持有独立的副本：

for _, ip := range ipList {
    // 把当前IP赋值给局部变量currentIP，每个goroutine拿到独立的值
    currentIP := ip
    go func() {
        // 这里使用currentIP而不是循环变量ip
        if err := doPing(currentIP); err != nil {
            log.Printf("Ping %s failed: %v", currentIP, err)
        }
    }()
}

2. 为每个请求生成唯一的ICMP标识 #

每个Ping请求需要独立的ID和序列号，确保响应能正确匹配到对应的IP：

// 用进程ID作为基础ID，保证同一进程内唯一
baseID := os.Getpid() & 0xffff
// 用原子计数器生成唯一序列号，避免并发冲突
var seqCounter uint16

// 在每个Ping请求里生成独立的seq
seq := atomic.AddUint16(&seqCounter, 1)

// 构造ICMP包时使用独立的ID和seq
msg := icmp.Message{
    Type: ipv4.ICMPTypeEcho, Code: 0,
    Body: &icmp.Echo{
        ID:   baseID,
        Seq:  int(seq),
        Data: []byte("ping test"),
    },
}

3. 等待所有Goroutine完成（可选） #

如果需要程序等待所有Ping任务结束再退出，用sync.WaitGroup做同步：

var wg sync.WaitGroup

for _, ip := range ipList {
    currentIP := ip
    wg.Add(1)
    go func() {
        defer wg.Done() // 任务完成时通知WaitGroup
        doPing(currentIP)
    }()
}

wg.Wait() // 阻塞直到所有任务完成

关键提醒 #

• Go的循环变量在整个循环周期内复用同一个内存地址，goroutine如果直接引用它，拿到的是变量最终的值，不是迭代时的快照。
• ICMP响应匹配必须依赖ID+序列号，否则不同IP的响应会被混在一起，导致结果异常。

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