Merge pull request #14 from ethpandaops/fix/disconnect

fix(execution/mimicry): empty disconnect reason
ethpandaops · Jun 19, 2023 · 81a2acd · 81a2acd
2 parents 69903db + 1e33641
commit 81a2acd
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/pkg/execution/mimicry/message_disconnect.go b/pkg/execution/mimicry/message_disconnect.go
@@ -21,15 +21,18 @@ func (h *Disconnect) Code() int { return DisconnectCode }
 func (h *Disconnect) ReqID() uint64 { return 0 }
 
 func (c *Client) receiveDisconnect(ctx context.Context, data []byte) *Disconnect {
-	reason := data[0:1]
-	// besu sends 2 byte disconnect message
-	if len(data) > 1 {
-		reason = data[1:2]
-	}
-
 	d := new(p2p.DiscReason)
-	if err := rlp.DecodeBytes(reason, &d); err != nil {
-		c.log.WithError(err).Debug("Error decoding disconnect")
+
+	if len(data) > 0 {
+		reason := data[0:1]
+		// besu sends 2 byte disconnect message
+		if len(data) > 1 {
+			reason = data[1:2]
+		}
+
+		if err := rlp.DecodeBytes(reason, &d); err != nil {
+			c.log.WithError(err).Debug("Error decoding disconnect")
+		}
 	}
 
 	return &Disconnect{Reason: *d}