Upstream/mc-router
1
0
Fork 0
mirror of https://github.com/itzg/mc-router.git synced 2024-11-21 11:25:41 +01:00
Code Issues Packages Projects Releases Wiki Activity

Prevent crash due to large memory allocation (#291)

Browse Source
This commit is contained in:
Ambrose Chua 2024-05-06 10:10:23 +08:00 committed by GitHub
parent 09ed21d467
commit f34b6e85b0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mcproto/read.go
View File

@ -160,6 +160,12 @@ func ReadFrame(reader io.Reader, addr net.Addr) (*Frame, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
// Limit frame length to 2^21 - 1
if frame.Length > 2097151 {
return nil, errors.Errorf("frame length %d too large", frame.Length)
}
logrus. logrus.
WithField("client", addr). WithField("client", addr).
WithField("length", frame.Length). WithField("length", frame.Length).