From 51297dd6df713b988b5c587e448b27d18ca1bd8a Mon Sep 17 00:00:00 2001 From: cui fliter Date: Fri, 30 Sep 2022 03:41:42 +0000 Subject: syscall: remove redundant type conversion Change-Id: Iae290216687fd1ce8be720600157fb78cc2446d0 GitHub-Last-Rev: 4fba64ecb14a704d39f6ecc33989522bcac6656f GitHub-Pull-Request: golang/go#55959 Reviewed-on: https://go-review.googlesource.com/c/go/+/436881 Auto-Submit: Ian Lance Taylor Run-TryBot: Ian Lance Taylor Reviewed-by: Dmitri Shuralyov Run-TryBot: Tobias Klauser TryBot-Result: Gopher Robot Reviewed-by: Ian Lance Taylor --- src/syscall/exec_libc.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/syscall/exec_libc.go') diff --git a/src/syscall/exec_libc.go b/src/syscall/exec_libc.go index 9e14197dcf..ef0c87e03c 100644 --- a/src/syscall/exec_libc.go +++ b/src/syscall/exec_libc.go @@ -215,7 +215,7 @@ func forkAndExecInChild(argv0 *byte, argv, envv []*byte, chroot, dir *byte, attr nextfd++ } for i = 0; i < len(fd); i++ { - if fd[i] >= 0 && fd[i] < int(i) { + if fd[i] >= 0 && fd[i] < i { if nextfd == pipe { // don't stomp on pipe nextfd++ } @@ -243,7 +243,7 @@ func forkAndExecInChild(argv0 *byte, argv, envv []*byte, chroot, dir *byte, attr closeFD(uintptr(i)) continue } - if fd[i] == int(i) { + if fd[i] == i { // dup2(i, i) won't clear close-on-exec flag on Linux, // probably not elsewhere either. _, err1 = fcntl1(uintptr(fd[i]), F_SETFD, 0) -- cgit v1.3