Skip to content

ftp: reject illegal IPport in PASV 227 response - #1997

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/ftp-illegal-PASV-host
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/ftp-illegal-PASV-host

Conversation

@bagder

@bagder bagder commented Oct 19, 2017

Copy link
Copy Markdown
Member

... by using range checks. Among other things, this avoids an undefined
behavior for a left shift that could happen on negative or very large
values.

Detected by OSS-fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=3694

... by using range checks. Among other things, this avoids an undefined
behavior for a left shift that could happen on negative or very large
values.

Detected by OSS-fuzz: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=3694
@bagder bagder added the FTP label Oct 19, 2017
@bagder bagder closed this in 769647e Oct 20, 2017
@bagder
bagder deleted the bagder/ftp-illegal-PASV-host branch October 21, 2017 19:49
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

1 participant