Changeset 43245 for trunk/src/wp-includes/comment.php

Timestamp:

05/12/2018 03:55:36 PM (6 years ago)

Author:

iandunn

Message:

Privacy: Escape comment URLs in personal export file to prevent XSS.

There doesn't appear to be any way for an attacker to introduce malicious input into the URL, unless a plugin is filtering the URL to add it, but it's better to be safe than sorry.

Props birgire.
Fixes #44054.

File:

• trunk/src/wp-includes/comment.php (modified) (1 diff)

trunk/src/wp-includes/comment.php

@@ -3354 +3354 @@
                 case 'comment_link':
                     $value = get_comment_link( $comment->comment_ID );
-                    $value = '<a href="' . $value . '" target="_blank" rel="noreferrer noopener">' . $value . '</a>';
+                    $value = sprintf(
+                        '<a href="%s" target="_blank" rel="noreferrer noopener">%s</a>',
+                        esc_url( $value ),
+                        esc_html( $value )
+                    );
                     break;
             }