deadlock when writeLoop exits with an error

[slingamn]

I'm using 8e7ce4b with some debugging printlns added, so the line numbers are slightly off (I'll provide links to the deadlocking calls as they appear in master):

SIGQUIT: quit
PC=0x46ede1 m=0 sigcode=0

goroutine 45 [semacquire, 24 minutes]:
sync.runtime_Semacquire(0xc0003e6010)
        /usr/local/go/src/runtime/sema.go:56 +0x45
sync.(*WaitGroup).Wait(0xc0003e6008)
        /usr/local/go/src/sync/waitgroup.go:130 +0x65
github.com/thoj/go-ircevent.(*Connection).Loop(0xc0003e6000)
        /redacted/vendor/github.com/thoj/go-ircevent/irc.go:245 +0x268
c2/cmd.(*Consumer).Client(0xc0003820f0, 0xc00009c500)
        /redacted/cmd/consumer.go:315 +0x419
created by c2/cmd.(*Consumer).Run
        /redacted/cmd/consumer.go:109 +0xe5

goroutine 107483 [select, 24 minutes]:
github.com/thoj/go-ircevent.(*Connection).RunCallbacks(0xc0003e6000, 0xc0002c0820)
        /redacted/vendor/github.com/thoj/go-ircevent/irc_callback.go:178 +0x565
github.com/thoj/go-ircevent.(*Connection).readLoop(0xc0003e6000)
        /redacted/vendor/github.com/thoj/go-ircevent/irc.go:88 +0x515
created by github.com/thoj/go-ircevent.(*Connection).Connect
        /redacted/vendor/github.com/thoj/go-ircevent/irc.go:487 +0x45e

goroutine 107785 [chan send, 24 minutes]:
github.com/thoj/go-ircevent.(*Connection).SendRaw(...)
        /redacted/vendor/github.com/thoj/go-ircevent/irc.go:346
c2/cmd.(*Consumer).ParseIRCMessage(0xc0003820f0, 0xc0002c0820)
        /redacted/cmd/consumer.go:665 +0x89e
c2/cmd.(*Consumer).Client.func3(0xc0002c0820)
        /redacted/cmd/consumer.go:291 +0x3b
github.com/thoj/go-ircevent.(*Connection).RunCallbacks.func1(0xc0003e6000, 0x0, 0xc0002c54a0, 0xc0003a6d80, 0xc0002c0820)
        /redacted/vendor/github.com/thoj/go-ircevent/irc_callback.go:164 +0x6d
created by github.com/thoj/go-ircevent.(*Connection).RunCallbacks
        /redacted/vendor/github.com/thoj/go-ircevent/irc_callback.go:162 +0x45e

Here are the deadlocking calls in master:

go-ircevent/irc.go

Line 242 in 8e7ce4b

irc.Wait()

go-ircevent/irc_callback.go

Line 178 in 8e7ce4b

select {

go-ircevent/irc.go

Line 343 in 8e7ce4b

irc.pwrite <- message + "\r\n"

From my debugging prints, it appears that writeLoop is exiting with an error here:

go-ircevent/irc.go

Line 189 in 8e7ce4b

return

and then there's no consumer for attempts to send to (*Connection).pwrite, which blocks our callback from executing, which in turn blocks readLoop from completing.

Thanks for your time!

[slingamn]

Here's a somewhat artificial, but illustrative, test case that reproduces reliably on my system against master:

https://gist.github.com/slingamn/ecafe727b0e2d35f0a04a627436c68ac