Skip to content

S
sys
Switch branch/tag
  1. 15 Jul, 2018 1 commit
  3. 09 Jul, 2018 2 commits
  5. 07 Jul, 2018 1 commit
  7. 06 Jul, 2018 1 commit
  9. 04 Jul, 2018 1 commit
  11. 27 Jun, 2018 1 commit
  13. 26 Jun, 2018 1 commit
  15. 25 Jun, 2018 1 commit
    • Tobias Klauser's avatar
      unix: don't check atime in TestUtimesNanoAt · a200a19c
      Tobias Klauser authored 
      If the underlying filesystem doesn't support atime or was mounted with the
noatime mount option, TestUtimesNanoAt fails when comparing atime:

--- FAIL: TestUtimesNanoAt (0.00s)
	syscall_linux_test.go:144: UtimesNanoAt: wrong atime: {1111 0}
	syscall_linux_test.go:147: UtimesNanoAt: wrong mtime: {3333 0}
FAIL
FAIL	golang.org/x/sys/unix	0.376s

Fix it by dropping the atime comparison, it's enough to verify UtimesNanoAt
working correctly by comparing mtime.

Fixes golang/go#26034

Change-Id: Id868f9d4c6a856a99ae930a46bfe91bd64ca8570
Reviewed-on: https://go-review.googlesource.com/120562
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      a200a19c
  17. 22 Jun, 2018 1 commit
    • Tobias Klauser's avatar
      unix: use private copy of ustat_t on Linux · 63fc586f
      Tobias Klauser authored 
      The ustat syscall has been deprecated on Linux for a long time and the
upcoming glibc 2.28 will remove ustat.h and it can no longer be used to
to generate the Ustat_t wrapper type. Since Linux still provides the
syscall, let's not break this functionality and add a private copy of
struct ustat so Ustat_t can still be generated.

Updates golang/go#25990

Change-Id: I74cf405ce6df92da36f1ee900ab6d98997d8f994
Reviewed-on: https://go-review.googlesource.com/120295
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      63fc586f
  19. 20 Jun, 2018 1 commit
  21. 19 Jun, 2018 2 commits
  23. 16 Jun, 2018 1 commit
    • Alex Brainman's avatar
      windows: replace uintptr in crypto structs · 6c888cc5
      Alex Brainman authored 
      This CL is a copy of CL 106275 (see CL 106275 for details).

It introduces CertInfo, CertTrustListInfo and CertRevocationCrlInfo
types. It uses pointers to new types instead of uintptr in CertContext,
CertSimpleChain and CertRevocationInfo.

CertRevocationInfo, CertChainPolicyPara and CertChainPolicyStatus types
have uintptr field that can be pointer to many different things
(according to Windows API). So this CL introduces Pointer type to be
used for those cases.

Fixes golang/go#25797

Change-Id: I7797ddc6daf3e67b7eab69ab9fbf4d51650f8b6a
Reviewed-on: https://go-review.googlesource.com/118797Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
      6c888cc5
  25. 15 Jun, 2018 1 commit
  27. 14 Jun, 2018 2 commits
  29. 13 Jun, 2018 3 commits
  31. 12 Jun, 2018 1 commit
  33. 11 Jun, 2018 2 commits
  35. 10 Jun, 2018 1 commit
  37. 06 Jun, 2018 3 commits
  39. 25 May, 2018 3 commits
  41. 24 May, 2018 4 commits
  43. 23 May, 2018 3 commits
  45. 22 May, 2018 3 commits
    • Tobias Klauser's avatar
      unix: add tests for *xattr functions · 88eb85aa
      Tobias Klauser authored 
      Based on the tests in github.com/pkg/xattr

Change-Id: I3af73ad538877da042602708248063c218260a2e
Reviewed-on: https://go-review.googlesource.com/114135
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      88eb85aa
    • Hana Kim's avatar
      windows: address error reported by vet · dc67e5b8
      Hana Kim authored 
      https://storage.googleapis.com/go-build-log/489a6b5c/misc-vet-vetall_c1e51035.log
cmd/vendor/golang.org/x/sys/windows/asm_windows_amd64.s:12: [amd64] loadlibrary: wrong argument size 8; expected $...-24

Change-Id: I02e7c53216e2c55d0d062e50f72984228c214e23
Reviewed-on: https://go-review.googlesource.com/114082Reviewed-by: 's avatarBrad Fitzpatrick <bradfitz@golang.org>
      dc67e5b8
    • irfan sharif's avatar
      unix: add {get,set,remove,list}xattr on darwin · 0b6bef99
      irfan sharif authored 
      It's only when dest is set to NULL that the OS X implementations of
getxattr() and listxattr() return the current sizes of the named
attributes. An empty byte array is not sufficient. To maintain the same
behaviour as the linux implementation, we wrap around the system calls
and pass in NULL when dest is empty.

The parameters for the OS X implementation of setxattr() vary slightly
compared to the linux system call, specifically the 'position'
parameter:

 linux:
     int setxattr(
         const char *path,
         const char *name,
         const void *value,
         size_t size,
         int flags
     );

 darwin:
     int setxattr(
         const char *path,
         const char *name,
         void *value,
         size_t size,
         u_int32_t position,
         int options
     );

'position' specifies the offset within the extended attribute. In the
current implementation, only the resource fork extended attribute makes
use of this argument. For all others, position is reserved.  We simply
default to setting it to zero.  If that's needed by the package user, a
function with a different name needs to be implemented instead.

Similarly for removexattr(), we wrap around and explicitly zero out the
options provided. We do so for interoperability with the linux variant.
If options are needed by the package user, a function with a different
name needs to be implemented instead.

Fixes golang/go#14456

Change-Id: I2581e1fa8dc9324bced7fda7f8ada10fe2ede3f5
Reviewed-on: https://go-review.googlesource.com/113995Reviewed-by: 's avatarTobias Klauser <tobias.klauser@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
      0b6bef99