From a9e538c1ca38b5bf736774f11aeab205c6ec2ff9 Mon Sep 17 00:00:00 2001
From: zhangzhongjun <zhangzhongjun@bytedance.com>
Date: Fri, 22 Nov 2024 09:06:42 +0000
Subject: [PATCH] fix: overflow when calculating stdDev

Signed-off-by: Jone Zhang <jone.zhongjun.zhang@gmail.com>
---
 ping.go | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/ping.go b/ping.go
index b6de420..ef534cd 100644
--- a/ping.go
+++ b/ping.go
@@ -160,7 +160,7 @@ type Pinger struct {
 	maxRtt    time.Duration
 	avgRtt    time.Duration
 	stdDevRtt time.Duration
-	stddevm2  time.Duration
+	stddevm2  float64
 	statsMu   sync.RWMutex
 
 	// If true, keep a record of rtts of all received packets.
@@ -340,9 +340,9 @@ func (p *Pinger) updateStatistics(pkt *Packet) {
 	delta := pkt.Rtt - p.avgRtt
 	p.avgRtt += delta / pktCount
 	delta2 := pkt.Rtt - p.avgRtt
-	p.stddevm2 += delta * delta2
+	p.stddevm2 += float64(delta) * float64(delta2)
 
-	p.stdDevRtt = time.Duration(math.Sqrt(float64(p.stddevm2 / pktCount)))
+	p.stdDevRtt = time.Duration(math.Sqrt(p.stddevm2 / float64(pktCount)))
 }
 
 // SetIPAddr sets the ip address of the target host.