osutil/chattr.go: avoid possible garbage collection of unsafe pointer (…

…#15008) Documentation for `unsafe.Pointer` explicitly mentions that uintptr cannot be stored in variable before implicit conversion back to Pointer during system call. Signed-off-by: Zeyad Gouda <[email protected]>
canonical · Feb 4, 2025 · 4fd3221 · 4fd3221
1 parent 7fe9be2
commit 4fd3221
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/osutil/chattr.go b/osutil/chattr.go
@@ -54,8 +54,7 @@ const (
 )
 
 func ioctl(f *os.File, request uintptr, attrp *int32) error {
-	argp := uintptr(unsafe.Pointer(attrp))
-	_, _, errno := syscall.Syscall(syscall.SYS_IOCTL, f.Fd(), request, argp)
+	_, _, errno := syscall.Syscall(syscall.SYS_IOCTL, f.Fd(), request, uintptr(unsafe.Pointer(attrp)))
 	if errno != 0 {
 		return os.NewSyscallError("ioctl", errno)
 	}