oEmbed: Add extra hardening around allowed HTML for improved sandboxing.
Merges [41448] to 4.5 branch. Built from https://develop.svn.wordpress.org/branches/4.5@41454 git-svn-id: http://core.svn.wordpress.org/branches/4.5@41287 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
parent
c3844666a2
commit
f1c76c5532
|
@ -782,24 +782,30 @@ function wp_filter_oembed_result( $result, $data, $url ) {
|
|||
}
|
||||
$html = $content[1] . $content[2];
|
||||
|
||||
preg_match( '/ src=([\'"])(.*?)\1/', $html, $results );
|
||||
|
||||
if ( ! empty( $results ) ) {
|
||||
$secret = wp_generate_password( 10, false );
|
||||
|
||||
$url = esc_url( "{$results[2]}#?secret=$secret" );
|
||||
$q = $results[1];
|
||||
|
||||
$html = str_replace( $results[0], ' src=' . $q . $url . $q . ' data-secret=' . $q . $secret . $q, $html );
|
||||
$html = str_replace( '<blockquote', "<blockquote data-secret=\"$secret\"", $html );
|
||||
}
|
||||
|
||||
$allowed_html['blockquote']['data-secret'] = true;
|
||||
$allowed_html['iframe']['data-secret'] = true;
|
||||
|
||||
$html = wp_kses( $html, $allowed_html );
|
||||
|
||||
if ( ! empty( $content[1] ) ) {
|
||||
// We have a blockquote to fall back on. Hide the iframe by default.
|
||||
$html = str_replace( '<iframe', '<iframe style="position: absolute; clip: rect(1px, 1px, 1px, 1px);"', $html );
|
||||
$html = str_replace( '<blockquote', '<blockquote class="wp-embedded-content"', $html );
|
||||
}
|
||||
|
||||
$html = str_replace( '<iframe', '<iframe class="wp-embedded-content" sandbox="allow-scripts" security="restricted"', $html );
|
||||
|
||||
preg_match( '/ src=[\'"]([^\'"]*)[\'"]/', $html, $results );
|
||||
|
||||
if ( ! empty( $results ) ) {
|
||||
$secret = wp_generate_password( 10, false );
|
||||
|
||||
$url = esc_url( "{$results[1]}#?secret=$secret" );
|
||||
|
||||
$html = str_replace( $results[0], " src=\"$url\" data-secret=\"$secret\"", $html );
|
||||
$html = str_replace( '<blockquote', "<blockquote data-secret=\"$secret\"", $html );
|
||||
}
|
||||
$html = str_ireplace( '<iframe', '<iframe class="wp-embedded-content" sandbox="allow-scripts" security="restricted"', $html );
|
||||
|
||||
return $html;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue