Use ParseInt instead of Atoi when converting process id string to an …

…int (#384)
nginx · Jul 10, 2023 · 171563b · 171563b
1 parent 02f86ca
commit 171563b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/core/metrics/sources/nginx_worker.go b/src/core/metrics/sources/nginx_worker.go
@@ -185,7 +185,7 @@ func (client *NginxWorkerClient) GetWorkerStats(childProcs []*proto.NginxDetails
 		}
 		numWorkers++
 
-		pidAsInt, err := strconv.Atoi(nginxDetails.ProcessId)
+		pidAsInt, err := strconv.ParseInt(nginxDetails.ProcessId, 10, 32)
 		if err != nil {
 			client.logger.Log(fmt.Sprintf("Failed to convert %s to int: %v", nginxDetails.ProcessId, err))
 			continue

diff --git a/test/performance/vendor/github.com/nginx/agent/v2/src/core/metrics/sources/nginx_worker.go b/test/performance/vendor/github.com/nginx/agent/v2/src/core/metrics/sources/nginx_worker.go