Followup r11120, wfParseUrl() can return false, if it does, the host isn't going...
authorSam Reed <reedy@users.mediawiki.org>
Sat, 11 Feb 2012 15:13:01 +0000 (15:13 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Sat, 11 Feb 2012 15:13:01 +0000 (15:13 +0000)
includes/upload/UploadFromUrl.php

index 13b0e14..0ae8a22 100644 (file)
@@ -47,8 +47,11 @@ class UploadFromUrl extends UploadBase {
                if ( !count( $wgCopyUploadsDomains ) ) {
                        return true;
                }
-               $valid = false;
                $parsedUrl = wfParseUrl( $url );
+               if ( !$parsedUrl ) {
+                       return false;
+               }
+               $valid = false;
                foreach( $wgCopyUploadsDomains as $domain ) {
                        if ( $parsedUrl['host'] === $domain ) {
                                $valid = true;