diff options
| author | Ferenc Szabo <frncmx@gmail.com> | 2019-01-17 21:44:29 +0800 |
|---|---|---|
| committer | Anton Evangelatov <anton.evangelatov@gmail.com> | 2019-01-17 21:44:29 +0800 |
| commit | 4f8ec445653aa5131eb18fd6f821ed5fc284fd42 (patch) | |
| tree | e28de46f60c38d42630939511a460b6705a8ef6d /node/service.go | |
| parent | 81e26d5a4837077d5fff17e7b461061b134a4a00 (diff) | |
| download | go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar.gz go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar.bz2 go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar.lz go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar.xz go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.tar.zst go-tangerine-4f8ec445653aa5131eb18fd6f821ed5fc284fd42.zip | |
swarm/network: fix data race in stream.(*Peer).handleOfferedHashesMsg() (#18468)
* swarm/network: fix data race in stream.(*Peer).handleOfferedHashesMsg()
handleOfferedHashesMsg() contained a data race:
- read => in a goroutine, call to c.batchDone()
- write => in the main thread, write to c.sessionAt
c.batchDone() contained a call to c.AddInterval(). Client was a value
receiver for AddInterval. So on c.AddInterval() call the whole client
struct got copied (read) while one of its field was modified in
handleOfferedHashesMsg() (write).
fixes ethersphere/go-ethereum#1086
* swarm/network: simplify some trivial statements
Diffstat (limited to 'node/service.go')
0 files changed, 0 insertions, 0 deletions