Skip to content

Instantly share code, notes, and snippets.

@mdeboer
Last active May 5, 2017 20:12
Show Gist options
  • Save mdeboer/5427af1d587d7cea50f5962fa7d0f290 to your computer and use it in GitHub Desktop.
Save mdeboer/5427af1d587d7cea50f5962fa7d0f290 to your computer and use it in GitHub Desktop.
WordPress CVE-2017-8295 patch (proposal, needs testing)
--- a/wp-includes/pluggable.php 2017-03-10 16:06:34.000000000 +0100
+++ b/wp-includes/pluggable.php 2017-05-05 21:44:15.000000000 +0200
@@ -323,11 +323,8 @@
if ( !isset( $from_email ) ) {
// Get the site domain and get rid of www.
- $sitename = strtolower( $_SERVER['SERVER_NAME'] );
- if ( substr( $sitename, 0, 4 ) == 'www.' ) {
- $sitename = substr( $sitename, 4 );
- }
-
+ $sitename = parse_url( network_home_url(), PHP_URL_HOST );
+
$from_email = 'wordpress@' . $sitename;
}
@@ -1491,7 +1488,7 @@
$notify_message .= sprintf( __( 'Spam it: %s' ), admin_url( "comment.php?action=spam&c={$comment->comment_ID}#wpbody-content" ) ) . "\r\n";
}
- $wp_email = 'wordpress@' . preg_replace('#^www\.#', '', strtolower($_SERVER['SERVER_NAME']));
+ $wp_email = 'wordpress@' . parse_url(network_home_url(), PHP_URL_HOST);
if ( '' == $comment->comment_author ) {
$from = "From: \"$blogname\" <$wp_email>";
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment