mirror of
https://github.com/golang/go
synced 2024-11-24 16:10:10 -07:00
b2dfacf35e
Instead of pushing the denominator argument on the stack, the denominator is now passed in m. This fixes a variety of bugs related to trying to take stack traces backwards from the middle of the software div/mod routines. Some of those bugs have been kludged around in the past, but others have not. Instead of trying to patch up after breaking the stack, this CL stops breaking the stack. This is an update of https://golang.org/cl/19810043, which was rolled back in https://golang.org/cl/20350043. The problem in the original CL was that there were divisions at bad times, when m was not available. These were divisions by constant denominators, either in C code or in assembly. The Go compiler knows how to generate division by multiplication for constant denominators, but the C compiler did not. There is no longer any C code, so that's taken care of. There was one problematic DIV in runtime.usleep (assembly) but https://golang.org/cl/12898 took care of that one. So now this approach is safe. Reject DIV/MOD in NOSPLIT functions to keep them from coming back. Fixes #6681. Fixes #6699. Fixes #10486. Change-Id: I09a13c76ad08ba75b3bd5d46a3eb78e66a84ab38 Reviewed-on: https://go-review.googlesource.com/12899 Reviewed-by: Ian Lance Taylor <iant@golang.org>
32 lines
519 B
Go
32 lines
519 B
Go
// run
|
|
|
|
// Copyright 2015 The Go Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
// Issue 10486.
|
|
// Check stack walk during div by zero fault,
|
|
// especially on software divide systems.
|
|
|
|
package main
|
|
|
|
import "runtime"
|
|
|
|
var A, B int
|
|
|
|
func divZero() int {
|
|
defer func() {
|
|
if p := recover(); p != nil {
|
|
var pcs [512]uintptr
|
|
runtime.Callers(2, pcs[:])
|
|
runtime.GC()
|
|
}
|
|
}()
|
|
return A / B
|
|
}
|
|
|
|
func main() {
|
|
A = 1
|
|
divZero()
|
|
}
|