Make WordPress Core

Opened 20 months ago

Last modified 20 months ago

#38311 new defect (bug)

Remove undefined variable `$port` in src/wp-admin/includes/class-ftp-sockets.php:141

Reported by: miyauchi Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: External Libraries Keywords:
Focuses: Cc:


$port is undefined in this function.

Attachments (1)

38311.patch (661 bytes) - added by miyauchi 20 months ago.

Download all attachments as: .zip

Change History (3)

20 months ago

#1 @dd32
20 months ago

  • Component changed from General to External Libraries

Hi @miyauchi!

Does this cause a PHP Notice or similar for you, or was it discovered through a static analysis application or similar?

The reason I ask, is that although the variable is unused, it doesn't appear to cause any problems as it's a return by-ref variable, and part of an external library.

#2 @miyauchi
20 months ago

OK, thanks! :)

Note: See TracTickets for help on using tickets.