Opened 4 years ago

Closed 4 years ago

#8969 closed enhancement (wontfix)

Replace tmpfile() function with tempnam()

Reported by: madmuffin Owned by: anonymous
Priority: lowest Milestone:
Component: General Version: 2.7
Severity: minor Keywords: dev-feedback
Cc:

Description

I noticed that Wordpress is using the tmpfile() function which does not regard the upload_tmp_dir value. It would be better to use tempnam() instead, to which the tmp-path can be passed.

these 3 files on my system make use of the tmpfile function:

info.php
wp-admin/includes/class-wp-filesystem-ftpext.php
wp-includes/js/tinymce/plugins/spellchecker/classes/PSpellShell.php

I am a shared host and normally customers have their own tmp folder but tmpfile always writes to /tmp/ on *nix systems, so I have to give wordpress users more permissions than usual.

Change History (2)

comment:1   Denis-de-Bernardy — 4 years ago

  • Keywords dev-feedback added; ftp,update,tmp,tmpfile,tmpnam,permissions removed
  • Milestone changed from 2.8 to Future Release
  • Priority changed from normal to lowest

Shouldn't this be reported as a php bug instead?

comment:2   Denis-de-Bernardy — 4 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Please report this as a php bug

Note: See TracTickets for help on using tickets.