Merge pull request #401 from SiaFoundation/matt/validate-fund-accounts

validate RPCFundAccountsRequest

Author: n8mgr

.changeset/validate_rpcfundaccountsrequest.md

@@ -0,0 +1,5 @@
+---
+default: patch
+---
+
+# Validate RPCFundAccountsRequest

rhp/v4/rpc.go

@@ -689,6 +689,10 @@ func RPCFundAccounts(ctx context.Context, t TransportClient, cs consensus.State,
 		RenterSignature: revision.RenterSignature,
 	}
 
+	if err := req.Validate(); err != nil {
+		return RPCFundAccountResult{}, fmt.Errorf("invalid request: %w", err)
+	}
+
 	var resp rhp4.RPCFundAccountsResponse
 	if err := callSingleRoundtripRPC(ctx, t, rhp4.RPCFundAccountsID, &req, &resp); err != nil {
 		return RPCFundAccountResult{}, err

rhp/v4/server.go

@@ -397,6 +397,8 @@ func (s *Server) handleRPCFundAccounts(stream net.Conn) error {
 	var req rhp4.RPCFundAccountsRequest
 	if err := rhp4.ReadRequest(stream, &req); err != nil {
 		return errorDecodingError("failed to read request: %v", err)
+	} else if err := req.Validate(); err != nil {
+		return err
 	}
 
 	state, unlock, err := s.lockContractForRevision(req.ContractID)