unistack-org/micro
4
0
Fork 3
Code Issues 17 Pull Requests 1 Actions Packages Projects Releases 258 Wiki Activity

Merge pull request #420 from unistack-org/race_transport

Browse Source 
fix race in http transport
This commit is contained in:
Asim Aslam 2019-02-15 15:58:49 +00:00 committed by GitHub
commit b63213a225
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
transport/http_transport.go
View File

@ -133,12 +133,6 @@ func (h *httpTransportClient) Recv(m *Message) error {
r = rc r = rc
} }
h.RLock()
if h.buff == nil {
return io.EOF
}
h.RUnlock()
// set timeout if its greater than 0 // set timeout if its greater than 0
if h.ht.opts.Timeout > time.Duration(0) { if h.ht.opts.Timeout > time.Duration(0) {
h.conn.SetDeadline(time.Now().Add(h.ht.opts.Timeout)) h.conn.SetDeadline(time.Now().Add(h.ht.opts.Timeout))
@ -181,7 +175,6 @@ func (h *httpTransportClient) Close() error {
h.once.Do(func() { h.once.Do(func() {
h.Lock() h.Lock()
h.buff.Reset(nil) h.buff.Reset(nil)
h.buff = nil
h.Unlock() h.Unlock()
close(h.r) close(h.r)
}) })