WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#36943 closed defect (bug) (fixed)

get_contents doesn't unlink tempfile on error

Reported by: ruud@… Owned by: dd32
Priority: normal Milestone: 4.6
Component: Filesystem API Version:
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Similar to #34772 and #36941, #36942, the get_contents method is also lacking the unlinking of the tempfile on error.

Attachments (1)

class-wp-filesystem-ftpsockets.patch​ (1.4 KB ) - added by ruud@… 10 years ago.
added unlink to error situation

Download all attachments as: .zip

Change History (5)

@ruud@…
10 years ago

added unlink to error situation

#1 @ruud@…
10 years ago

  • Keywords has-patch dev-feedback added

#2 @ocean90
10 years ago

  • Version trunk

#3 @dd32
10 years ago

  • Keywords dev-feedback removed
  • Milestone Awaiting Review → 4.6
  • Owner set to dd32
  • Status new → accepted

#4 @ocean90
10 years ago

  • Resolution → fixed
  • Status accepted → closed

In 38094:

Filesystem API: Cleanup temporary file when the temporary file couldn't be opened.

Props ruud@joyo.
See #34772.
Fixes #36942, #36943.

Note: See TracTickets for help on using tickets.