sendRequest reads req.buf after closing req.seq

NewRequest says you can avoid reallocating a new buffer for each request by calling it directly.
This is not true if req.seq is closed before req.buf is read.

1 files changed, 1 insertions, 1 deletions

diff --git a/xgb.go b/xgb.go
index 487ae16..44634fe 100644
--- a/xgb.go
+++ b/xgb.go
@@ -337,9 +337,9 @@ func (c *Conn) sendRequests() {
 			}
 		}
 		req.cookie.Sequence = c.newSequenceId()
-		close(req.seq)
 		c.cookieChan <- req.cookie
 		c.writeBuffer(req.buf)
+		close(req.seq)
 	}
 	response := make(chan struct{})
 	c.closing <- response