bpo-43285 Make ftplib not trust the PASV response. (GH-24838)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Mon, 15 Mar 2021 19:02:45 +0000 (12:02 -0700)
committerAdrian Bunk <bunk@debian.org>
Sun, 1 Dec 2024 12:12:57 +0000 (14:12 +0200)
commitc408d358aac5a8d7a142a8b13a851eabc9ba6018
treedf4041660d36f5e8ecd23aa7a4abe07489f0760c
parent99602a7aa925980b4d933f75458c5f3c8cd25a14
bpo-43285 Make ftplib not trust the PASV response. (GH-24838)

bpo-43285: Make ftplib not trust the PASV response.

The IPv4 address value returned from the server in response to the PASV command
should not be trusted.  This prevents a malicious FTP server from using the
response to probe IPv4 address and port combinations on the client network.

Instead of using the returned address, we use the IP address we're
already connected to.  This is the strategy other ftp clients adopted,
and matches the only strategy available for the modern IPv6 EPSV command
where the server response must return a port number and nothing else.

For the rare user who _wants_ this ugly behavior, set a `trust_server_pasv_ipv4_address`
attribute on your `ftplib.FTP` instance to True.
(cherry picked from commit 0ab152c6b5d95caa2dc1a30fa96e10258b5f188e)

Co-authored-by: Gregory P. Smith <greg@krypto.org>
Gbp-Pq: Name 0007-bpo-43285-Make-ftplib-not-trust-the-PASV-response.-G.patch
Lib/ftplib.py
Lib/test/test_ftplib.py