diff options
| author | Ian Gudger <igudger@google.com> | 2016-10-28 20:09:02 -0700 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2016-10-30 19:04:09 +0000 |
| commit | 154860ff17dacbf309ed247277d245ec887af6d4 (patch) | |
| tree | c10b3cdfad4db92e6e2aa220bbcec34e688edfbc /api | |
| parent | ce6aa2ebdab0c6616ef17acc9282b0e168f5f21a (diff) | |
| download | go-154860ff17dacbf309ed247277d245ec887af6d4.tar.gz go-154860ff17dacbf309ed247277d245ec887af6d4.zip | |
syscall: remove X__cmsg_data from Cmsghdr
This field is a zero length array and has little use. Since Go 1.5, trailing
zero-length arrays take up space. Both syscall.UnixRights() and
syscall.ParseSocketControlMessage() depend on being able to do an unsafe cast
of socket control message data to Cmsghdr this is only safe if the socket
control message data is greater than or equal to the size of Cmsghdr. Since
control message data that is equal in size to Cmsghdr without X__cmsg_data is
a valid socket control message, we must remove X__cmsg_data or not perform the
unsafe cast.
Removing X__cmsg_data will prevent Go code that uses X__cmsg_data from
compiling, but removing the unsafe cast will cause Go code that uses
X__cmsg_data to fail or exhibit undefined behavior at runtime. It was
therefore decided that removing X__cmsg_data was the better option.
Fixes #17649
Change-Id: I39f323f978eca09d62da5785c5c5c9c7cbdf8c31
Reviewed-on: https://go-review.googlesource.com/32319
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'api')
| -rw-r--r-- | api/except.txt | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/api/except.txt b/api/except.txt index 93302ec819..cd5d8e0df5 100644 --- a/api/except.txt +++ b/api/except.txt @@ -19,6 +19,12 @@ pkg syscall (darwin-386), const ImplementsGetwd = false pkg syscall (darwin-386-cgo), const ImplementsGetwd = false pkg syscall (darwin-amd64), const ImplementsGetwd = false pkg syscall (darwin-amd64-cgo), const ImplementsGetwd = false +pkg syscall (linux-386), type Cmsghdr struct, X__cmsg_data [0]uint8 +pkg syscall (linux-386-cgo), type Cmsghdr struct, X__cmsg_data [0]uint8 +pkg syscall (linux-amd64), type Cmsghdr struct, X__cmsg_data [0]uint8 +pkg syscall (linux-amd64-cgo), type Cmsghdr struct, X__cmsg_data [0]uint8 +pkg syscall (linux-arm), type Cmsghdr struct, X__cmsg_data [0]uint8 +pkg syscall (linux-arm-cgo), type Cmsghdr struct, X__cmsg_data [0]uint8 pkg syscall (openbsd-386), const BIOCGRTIMEOUT = 1074283118 pkg syscall (openbsd-386), const BIOCSRTIMEOUT = 2148024941 pkg syscall (openbsd-386), const RTF_FMASK = 63496 |