Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/gc: runtime failure with -race #5446

Closed
dvyukov opened this issue May 12, 2013 · 7 comments
Closed

cmd/gc: runtime failure with -race #5446

dvyukov opened this issue May 12, 2013 · 7 comments

Comments

@dvyukov
Copy link
Member

dvyukov commented May 12, 2013

16773:30c566874b83 tip, darwin/amd64

The following program runs fine w/o -race, but with -race it crashes with:

panic: runtime error: index out of range
goroutine 1 [running]:
main.main()
    /tmp/race1.go:7 +0x1fe

// tmp/race1.go 
package main

func main() {
        x := []int{10, 20}
        y := []int{99}
        i := 1
        for i, x[i] = range y {
                break
        }
}
@dvyukov
Copy link
Member Author

dvyukov commented May 19, 2013

Comment 1:

Labels changed: added go1.2maybe, removed priority-triage.

@dvyukov
Copy link
Member Author

dvyukov commented May 19, 2013

Comment 2:

Labels changed: added go1.1.1, removed go1.2maybe.

@DanielMorsing
Copy link
Contributor

Comment 3:

Remy has a fix at https://golang.org/cl/9560044/

Owner changed to @remyoudompheng.

Status changed to Started.

@remyoudompheng
Copy link
Contributor

Comment 4:

This issue was closed by revision fc3bec3.

Status changed to Fixed.

@ianlancetaylor
Copy link
Member

Comment 5:

Labels changed: removed go1.1.1.

@dvyukov
Copy link
Member Author

dvyukov commented May 30, 2013

Comment 6:

Why is this not Go1.1.1?

@ianlancetaylor
Copy link
Member

Comment 7:

For 1.1.1 we're only going to include critical bugs with no workaround.  The race
detector is always optional, so we'll leave race detector bugs for 1.2.

@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants