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

Possible race #32

Closed
lowenna opened this issue Sep 21, 2016 · 2 comments
Closed

Possible race #32

lowenna opened this issue Sep 21, 2016 · 2 comments

Comments

@lowenna
Copy link
Contributor

lowenna commented Sep 21, 2016

Found running CI on a daemon instrumented with go race detector. @jstarks

WARNING: DATA RACE
Write at 0x00c042296dd8 by goroutine 14:
github.com/Microsoft/go-winio.(_win32File).closeHandle()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/file.go:92 +0x7b
github.com/Microsoft/go-winio.(_win32File).Close()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/file.go:103 +0x55
github.com/Microsoft/go-winio.(*win32PipeListener).listenerRoutine()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/pipe.go:275 +0x47c

Previous read at 0x00c042296dd8 by goroutine 94:
github.com/Microsoft/go-winio.(_win32File).asyncIo()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/file.go:145 +0x167
github.com/Microsoft/go-winio.connectPipe()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/pipe.go:360 +0x1b0
github.com/Microsoft/go-winio.(_win32PipeListener).listenerRoutine.func1()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/pipe.go:265 +0x56

Goroutine 14 (running) created at:
github.com/Microsoft/go-winio.ListenPipe()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/pipe.go:350 +0x6ce
github.com/docker/docker/pkg/listeners.Init()
e:/go/src/github.com/docker/docker/pkg/listeners/listeners_windows.go:43 +0x8e1
main.(_DaemonCli).start()
e:/go/src/github.com/docker/docker/cmd/dockerd/daemon.go:215 +0x1f80
main.runDaemon()
e:/go/src/github.com/docker/docker/cmd/dockerd/docker.go:75 +0x25a
main.newDaemonCommand.func1()
e:/go/src/github.com/docker/docker/cmd/dockerd/docker.go:41 +0xe9
github.com/spf13/cobra.(_Command).execute()
e:/go/src/github.com/docker/docker/vendor/src/github.com/spf13/cobra/command.go:643 +0xa03
github.com/spf13/cobra.(_Command).ExecuteC()
e:/go/src/github.com/docker/docker/vendor/src/github.com/spf13/cobra/command.go:739 +0x7ba
github.com/spf13/cobra.(_Command).Execute()
e:/go/src/github.com/docker/docker/vendor/src/github.com/spf13/cobra/command.go:692 +0x51
main.main()
e:/go/src/github.com/docker/docker/cmd/dockerd/docker.go:99 +0x152

Goroutine 94 (running) created at:
github.com/Microsoft/go-winio.(*win32PipeListener).listenerRoutine()
e:/go/src/github.com/docker/docker/vendor/src/github.com/Microsoft/go-winio/pipe.go:266 +0x305

@samuelkarp
Copy link

@jhowardmsft @jstarks I opened #31 to fix this.

@darstahl
Copy link
Contributor

darstahl commented Aug 2, 2017

@darstahl darstahl closed this as completed Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants