]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: remove unnecessary calls to memclr
authorMichael Munday <munday@ca.ibm.com>
Sat, 20 Aug 2016 17:36:27 +0000 (18:36 +0100)
committerMichael Munday <munday@ca.ibm.com>
Sat, 20 Aug 2016 18:00:09 +0000 (18:00 +0000)
Go will have already cleared the structs (the original C wouldn't
have).

Change-Id: I4a5a0cfd73953181affc158d188aae2ce281bb33
Reviewed-on: https://go-review.googlesource.com/27435
Run-TryBot: Michael Munday <munday@ca.ibm.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/runtime/os_linux.go

index 542f214a42140980eb79f22d675ab284a04b1314..0131c6687db988c1c837938ddc0383f2878cf9fd 100644 (file)
@@ -388,7 +388,6 @@ func osyield()
 //go:nowritebarrierrec
 func setsig(i int32, fn uintptr, restart bool) {
        var sa sigactiont
-       memclr(unsafe.Pointer(&sa), unsafe.Sizeof(sa))
        sa.sa_flags = _SA_SIGINFO | _SA_ONSTACK | _SA_RESTORER
        if restart {
                sa.sa_flags |= _SA_RESTART
@@ -431,8 +430,6 @@ func setsigstack(i int32) {
 //go:nowritebarrierrec
 func getsig(i int32) uintptr {
        var sa sigactiont
-
-       memclr(unsafe.Pointer(&sa), unsafe.Sizeof(sa))
        if rt_sigaction(uintptr(i), nil, &sa, unsafe.Sizeof(sa.sa_mask)) != 0 {
                throw("rt_sigaction read failure")
        }