Changeset 14116 for trunk/wp-includes/functions.php

Timestamp:

04/16/2010 07:48:30 PM (15 years ago)

Author:

nacin

Message:

Some sanity checks for parse_url in do_enclose. props miqrogroove, fixes #11029

File:

• trunk/wp-includes/functions.php (modified) (2 diffs)

trunk/wp-includes/functions.php

@@ -1198 +1198 @@
     foreach ( (array) $post_links_temp[0] as $link_test ) {
         if ( !in_array( $link_test, $pung ) ) { // If we haven't pung it already
-            $test = parse_url( $link_test );
+            $test = @parse_url( $link_test );
+            if ( false === $test )
+                continue;
             if ( isset( $test['query'] ) )
                 $post_links[] = $link_test;
@@ -1216 +1218 @@
                 // Check to see if we can figure out the mime type from
                 // the extension
-                $url_parts = parse_url( $url );
-                $extension = pathinfo( $url_parts['path'], PATHINFO_EXTENSION );
-                if ( !empty( $extension ) ) {
-                    foreach ( get_allowed_mime_types( ) as $exts => $mime ) {
-                        if ( preg_match( '!^(' . $exts . ')$!i', $extension ) ) {
-                            $type = $mime;
-                            break;
+                $url_parts = @parse_url( $url );
+                if ( false !== $url_parts ) {
+                    $extension = pathinfo( $url_parts['path'], PATHINFO_EXTENSION );
+                    if ( !empty( $extension ) ) {
+                        foreach ( get_allowed_mime_types( ) as $exts => $mime ) {
+                            if ( preg_match( '!^(' . $exts . ')$!i', $extension ) ) {
+                                $type = $mime;
+                                break;
+                            }
                         }
                     }