Changeset 39766

Timestamp:

01/11/2017 01:48:22 AM (7 years ago)

Author:

aaroncampbell

Message:

Add nonce for widget accessibility mode.

Props vortfu.

See #23328.

Merges [39765] to 4.2 branch.

Location:

branches/4.2

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/screen.php (modified) (1 diff)
• src/wp-admin/widgets.php (modified) (1 diff)

branches/4.2/src/wp-admin/includes/screen.php

@@ -972 +972 @@
         switch ( $this->base ) {
             case 'widgets':
-                $this->_screen_settings = '<p><a id="access-on" href="widgets.php?widgets-access=on">' . __('Enable accessibility mode') . '</a><a id="access-off" href="widgets.php?widgets-access=off">' . __('Disable accessibility mode') . "</a></p>\n";
+                $nonce = wp_create_nonce( 'widgets-access' );
+                $this->_screen_settings = '<p><a id="access-on" href="widgets.php?widgets-access=on&_wpnonce=' . urlencode( $nonce ) . '">' . __('Enable accessibility mode') . '</a><a id="access-off" href="widgets.php?widgets-access=off&_wpnonce=' . urlencode( $nonce ) . '">' . __('Disable accessibility mode') . "</a></p>\n";
                 break;
             case 'post' :

branches/4.2/src/wp-admin/widgets.php

@@ -18 +18 @@
 $widgets_access = get_user_setting( 'widgets_access' );
 if ( isset($_GET['widgets-access']) ) {
+    check_admin_referer( 'widgets-access' );
+
     $widgets_access = 'on' == $_GET['widgets-access'] ? 'on' : 'off';
     set_user_setting( 'widgets_access', $widgets_access );