forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-06-04 22:18:40 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

rpc: fix method-name matched before maxMethodNameLength (#35038)

Browse source 
Co-authored-by: MariusVanDerWijden <m.vanderwijden@live.de>
This commit is contained in:
cui 2026-05-27 22:32:51 +08:00 committed by GitHub
parent 9d21f6ebd5
commit f4a90d178a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
rpc/handler.go
View file

@ -502,6 +502,10 @@ func (h *handler) handleCallMsg(ctx *callProc, msg *jsonrpcMessage) *jsonrpcMess
// handleCall processes method calls.
func (h *handler) handleCall(cp *callProc, msg *jsonrpcMessage) *jsonrpcMessage {
// Check method name length
if len(msg.Method) > maxMethodNameLength {
return msg.errorResponse(&invalidRequestError{fmt.Sprintf("method name too long: %d > %d", len(msg.Method), maxMethodNameLength)})
}
if msg.isSubscribe() {
return h.handleSubscribe(cp, msg)
}
@ -512,11 +516,6 @@ func (h *handler) handleCall(cp *callProc, msg *jsonrpcMessage) *jsonrpcMessage
}
return h.runMethod(cp.ctx, msg, h.unsubscribeCb, args)
}
// Check method name length
if len(msg.Method) > maxMethodNameLength {
return msg.errorResponse(&invalidRequestError{fmt.Sprintf("method name too long: %d > %d", len(msg.Method), maxMethodNameLength)})
}
callb, service, method := h.reg.callback(msg.Method)
// If the method is not found, return an error.

4
rpc/websocket_test.go
View file

@ -432,10 +432,10 @@ func TestWebsocketMethodNameLengthLimit(t *testing.T) {
isSubscription: true,
},
{
name: "subscription name too long",
name: "method name too long subscribe",
method: string(make([]byte, maxMethodNameLength+1)) + "_subscribe",
params: []interface{}{"newHeads"},
expectedError: "subscription name too long",
expectedError: "method name too long",
isSubscription: true,
},
}