Merge pull request #1313 from Bytom/fix_prod

p2p/discv5: fix idx can be negative after uint convert to int(can cau…
BytomDAO · Sep 3, 2018 · 6568627 · 6568627
2 parents 2bc2396 + a79fbcb
commit 6568627
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/p2p/discover/net.go b/p2p/discover/net.go
@@ -1221,7 +1221,7 @@ func (net *Network) checkTopicRegister(data *topicRegister) (*pong, error) {
 	if hash != pongpkt.data.(*pong).TopicHash {
 		return nil, errors.New("topic hash mismatch")
 	}
-	if data.Idx < 0 || int(data.Idx) >= len(data.Topics) {
+	if int(data.Idx) < 0 || int(data.Idx) >= len(data.Topics) {
 		return nil, errors.New("topic index out of range")
 	}
 	return pongpkt.data.(*pong), nil