diff options
author | Wei-Ning Huang <w@byzantine-lab.io> | 2019-08-26 10:45:43 +0800 |
---|---|---|
committer | Wei-Ning Huang <w@byzantine-lab.io> | 2019-09-17 16:57:31 +0800 |
commit | 5c6389147f7e0c0ffd76635b27d8891127e471e3 (patch) | |
tree | 5007936bf3fe88bd3162bfc51f9d0557a34af97b | |
parent | f3a8bd79153cdce70a9d189c45aa3880a5548d70 (diff) | |
download | go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar.gz go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar.bz2 go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar.lz go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar.xz go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.tar.zst go-tangerine-5c6389147f7e0c0ffd76635b27d8891127e471e3.zip |
dex: potential flooding protection
-rw-r--r-- | dex/handler.go | 28 |
1 files changed, 18 insertions, 10 deletions
diff --git a/dex/handler.go b/dex/handler.go index 4c1ed9a36..94d7d79e5 100644 --- a/dex/handler.go +++ b/dex/handler.go @@ -347,6 +347,14 @@ func (pm *ProtocolManager) ReceiveChan() <-chan coreTypes.Msg { return pm.receiveCh } +func (pm *ProtocolManager) sendCoreMsg(msg *coreTypes.Msg) { + select { + case pm.receiveCh <- *msg: + default: + log.Warn("ReceiveChan full, dropping", "message", msg) + } +} + func (pm *ProtocolManager) ReportBadPeerChan() chan<- interface{} { return pm.reportBadPeerChan } @@ -895,10 +903,10 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { } pm.cache.addBlocks(blocks) for _, block := range blocks { - pm.receiveCh <- coreTypes.Msg{ + pm.sendCoreMsg(&coreTypes.Msg{ PeerID: p.ID().String(), Payload: block, - } + }) } case msg.Code == VoteMsg: if atomic.LoadInt32(&pm.receiveCoreMessage) == 0 { @@ -912,10 +920,10 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { if vote.Type >= coreTypes.VotePreCom { pm.cache.addVote(vote) } - pm.receiveCh <- coreTypes.Msg{ + pm.sendCoreMsg(&coreTypes.Msg{ PeerID: p.ID().String(), Payload: vote, - } + }) } case msg.Code == AgreementMsg: if atomic.LoadInt32(&pm.receiveCoreMessage) == 0 { @@ -933,10 +941,10 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { block[0].Randomness = agreement.Randomness pm.cache.addFinalizedBlock(block[0]) } - pm.receiveCh <- coreTypes.Msg{ + pm.sendCoreMsg(&coreTypes.Msg{ PeerID: p.ID().String(), Payload: &agreement, - } + }) case msg.Code == DKGPrivateShareMsg: if atomic.LoadInt32(&pm.receiveCoreMessage) == 0 { break @@ -947,10 +955,10 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { return errResp(ErrDecode, "msg %v: %v", msg, err) } p.MarkDKGPrivateShares(rlpHash(ps)) - pm.receiveCh <- coreTypes.Msg{ + pm.sendCoreMsg(&coreTypes.Msg{ PeerID: p.ID().String(), Payload: &ps, - } + }) case msg.Code == DKGPartialSignatureMsg: if atomic.LoadInt32(&pm.receiveCoreMessage) == 0 { break @@ -960,10 +968,10 @@ func (pm *ProtocolManager) handleMsg(p *peer) error { if err := msg.Decode(&psig); err != nil { return errResp(ErrDecode, "msg %v: %v", msg, err) } - pm.receiveCh <- coreTypes.Msg{ + pm.sendCoreMsg(&coreTypes.Msg{ PeerID: p.ID().String(), Payload: &psig, - } + }) case msg.Code == PullBlocksMsg: if atomic.LoadInt32(&pm.receiveCoreMessage) == 0 { break |