Ticket #12201 (closed defect (bug): duplicate)

Opened 2 years ago

Last modified 22 months ago

getmyuid() is disabled - workaround

Reported by: edward mindreantre Owned by: dd32
Priority: normal Milestone:
Component: Filesystem Version: 2.9.2
Severity: normal Keywords: has-patch
Cc: edward, mindreantre, imme-emosol

Description (last modified by nacin) (diff)

On some web hosts the getmyuid() function, used in file.php to discern whether direct access to the file system is allowed or not, is disabled. "For security reasons", whatever that might mean. How much damage can you cause with getmyuid?

This prevents people who, even though they've installed wordpress correctly via FTP and set the correct file permissions, from installing themes, plugins and upgrades.

Sad but true.

Here's a workaround.

Attachments

file.php.diff Download (460 bytes) - added by edward mindreantre 2 years ago.
Diff of file.php from v2.9.1
file.php.2.diff Download (460 bytes) - added by edward mindreantre 2 years ago.
Diff of file.php from v2.9.1
file.php.patch.diff Download (448 bytes) - added by imme-emosol 23 months ago.
Another patch for the ftp-issue, not using the at-sign(@).

Change History

Diff of file.php from v2.9.1

Diff of file.php from v2.9.1

  • Cc edward, mindreantre added
  • Version set to 2.9.2
  • Milestone changed from Unassigned to 3.0

comment:2   dd322 years ago

  • Keywords has-patch needs-testing added; file.php direct access getmyuid removed
  • Status changed from new to accepted

See Also #10424

I dont have an environment under which to test this still, But will get to it if the above ticket fails.

Can you test that patch over on that ticket please?

Even setting define('FS_METHOD', 'direct'); won't work without my patch.

(Fresh trunk, fresh install)

comment:4   dd322 years ago

Even setting define('FS_METHOD', 'direct'); won't work without my patch.

It should, Given that the code you patched isn't even touched if thats defined.

  • Description modified (diff)

Removing language.

Another patch for the ftp-issue, not using the at-sign(@).

  • Cc imme-emosol added
  • Keywords needs-testing removed
  • Status changed from accepted to closed
  • Resolution set to duplicate

Had issue with wordpress 2.8.4-1ubuntu1 . Resolved with my patch, quite sure the same goes for the other supplied patches.

community-related , linked :  http://www.chrisabernethy.com/why-wordpress-asks-connection-info/comment-page-2/#comment-14293

Guess it will al be resolved with http://core.trac.wordpress.org/ticket/10424 so closing this one as duplicate .

ghehe, so I should not have added the following to my patch : Another patch for the ftp-issue, not using the at-sign(@). But something like : Diff of file.php from v2.8.4-1ubuntu1 ?

  • Milestone 3.0 deleted
Note: See TracTickets for help on using tickets.