Page MenuHomePhabricator

importImages uploads succeed when source file is unreadable
Open, MediumPublic

Description

I'm not sure if this should be in maintenance scripts or whatever

Due to file permission issues, the importImages script reports that the upload worked, but it really didn't - the file isn't there but the page text is.

If the file isn't readable, the "upload" for that file should be aborted, not creating page text

The specific error being "failed to open stream: Permission denied"

reedy@tin:/tmp/uploads$ sudo -u apache mwscript importImages.php
--wiki=commonswiki --user="Odder" /tmp/uploads
Import Images

Importing 2014 State Of The Union Address Enhanced.webm...PHP Warning:
fopen(/tmp/uploads/2014 State Of The Union Address Enhanced.webm): failed to
open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278

Warning: fopen(/tmp/uploads/2014 State Of The Union Address Enhanced.webm):
failed to open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278
done.
Importing 2013-10-05 President Obama's Weekly Address.webm...PHP Warning:
fopen(/tmp/uploads/2013-10-05 President Obama's Weekly Address.webm): failed to
open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278

Warning: fopen(/tmp/uploads/2013-10-05 President Obama's Weekly Address.webm):
failed to open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278
done.
Importing 2013-10-12 President Obama's Weekly Address.webm...PHP Warning:
fopen(/tmp/uploads/2013-10-12 President Obama's Weekly Address.webm): failed to
open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278

Warning: fopen(/tmp/uploads/2013-10-12 President Obama's Weekly Address.webm):
failed to open stream: Permission denied in
/a/common/php-1.23wmf11/includes/filebackend/SwiftFileBackend.php on line 278
done.

Found: 3
Added: 3
reedy@tin:/tmp/uploads$ ls -al
total rOMWC24047921fcbc
drwxrwxrwx 2 reedy wikidev 4096 Feb 3 18:47 .
drwxrwxrwt 46 root root 12288 Feb 3 18:46 ..
-rw-rw-r-- 1 reedy wikidev 1255 Feb 1 14:50 2013-10-05 President Obama's
Weekly Address.txt
-rw-rw-r-- 1 reedy wikidev 163504321 Jan 27 22:13 2013-10-05 President Obama's
Weekly Address.webm
-rw-rw-r-- 1 reedy wikidev 1197 Feb 1 14:52 2013-10-12 President Obama's
Weekly Address.txt
-rw-rw-r-- 1 reedy wikidev 158637780 Feb 1 14:34 2013-10-12 President Obama's
Weekly Address.webm
-rw-rw-r-- 1 reedy wikidev 884 Jan 31 16:26 2014 State Of The Union
Address Enhanced.txt
-rw-rw-r-- 1 reedy wikidev 909068945 Jan 30 19:37 2014 State Of The Union
Address Enhanced.webm
-rw-r--r-- 1 reedy wikidev 909060459 Feb 3 18:45 SOTU.tar.gz
-rw-r--r-- 1 reedy wikidev 322185685 Feb 3 18:46 WeeklyAddresses.tar.gz
reedy@tin:/tmp/uploads$
reedy@tin:/tmp/uploads$
reedy@tin:/tmp/uploads$
reedy@tin:/tmp/uploads$
reedy@tin:/tmp/uploads$
reedy@tin:/tmp/uploads$ chmod 777 *
reedy@tin:/tmp/uploads$ rm *.gz
reedy@tin:/tmp/uploads$ ls -al
total 1202392
drwxrwxrwx 2 reedy wikidev 4096 Feb 3 18:52 .
drwxrwxrwt 46 root root 12288 Feb 3 18:50 ..
-rwxrwxrwx 1 reedy wikidev 1255 Feb 1 14:50 2013-10-05 President Obama's
Weekly Address.txt
-rwxrwxrwx 1 reedy wikidev 163504321 Jan 27 22:13 2013-10-05 President Obama's
Weekly Address.webm
-rwxrwxrwx 1 reedy wikidev 1197 Feb 1 14:52 2013-10-12 President Obama's
Weekly Address.txt
-rwxrwxrwx 1 reedy wikidev 158637780 Feb 1 14:34 2013-10-12 President Obama's
Weekly Address.webm
-rwxrwxrwx 1 reedy wikidev 884 Jan 31 16:26 2014 State Of The Union
Address Enhanced.txt
-rwxrwxrwx 1 reedy wikidev 909068945 Jan 30 19:37 2014 State Of The Union
Address Enhanced.webm
reedy@tin:/tmp/uploads$ sudo -u apache mwscript importImages.php
--wiki=commonswiki --user="Odder" /tmp/uploads
Import Images

Importing 2014 State Of The Union Address Enhanced.webm...done.
Importing 2013-10-05 President Obama's Weekly Address.webm...done.
Importing 2013-10-12 President Obama's Weekly Address.webm...done.

Found: 3
Added: 3
reedy@tin:/tmp/uploads$


Version: 1.23.0
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=56987
https://bugzilla.wikimedia.org/show_bug.cgi?id=60318

Details

Reference
bz60777