]> Cypherpunks repositories - gostls13.git/commit
os: remove incorrect detection of O_CLOEXEC flag on darwin
authorMartin Möhrmann <moehrmann@google.com>
Sat, 18 Feb 2017 06:46:41 +0000 (07:46 +0100)
committerBrad Fitzpatrick <bradfitz@golang.org>
Sat, 18 Feb 2017 16:14:15 +0000 (16:14 +0000)
commit6ef92b6e3bce369feeb114dd3267a3f18038fc8c
treeb925a8dbc78d5bb66d714d60574cf94544fdb1ff
parent497b608faba2a9c450d03e2bca2ef005ea32b933
os: remove incorrect detection of O_CLOEXEC flag on darwin

The below range loop will not stop when encountering
the first '.' character in a Darwin version string like "15.6.0".

for i = range osver {
   if osver[i] != '.' {
         continue
      }
   }
}

Therefore, the condition i > 2 was always satisfied and
supportsCloseOnExec was always set to true.

Since the minimum supported version of OSX for go is currently 10.8
and O_CLOEXEC is implemented from OSX 10.7 on the detection code
can be removed and support for O_CLOEXEC is always assumed to exist.

Change-Id: Idd10094d8385dd4adebc8d7a6d9e9a8f29455867
Reviewed-on: https://go-review.googlesource.com/37193
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/os/sys_darwin.go