diff --git a/mathjax-latex-admin.php b/mathjax-latex-admin.php
index 21ad63f..375bbaf 100644
--- a/mathjax-latex-admin.php
+++ b/mathjax-latex-admin.php
@@ -53,13 +53,11 @@ function plugin_options_menu() {
wp_die( __( 'You do not have sufficient permissions to access this page.' ) );
}
- $nonce = wp_nonce_field( 'kblog_mathjax_latex_save_action', 'kblog_mathjax_latex_save_field', true, false );
-
- $this->table_head( $nonce );
+ $this->table_head();
// save options if this is a valid post
- if ( isset( $_POST['kblog_mathjax_latex_save_field'] ) &&
- wp_verify_nonce( $_POST['kblog_mathjax_latex_save_field'], 'kblog_mathjax_latex_save_action' )
+ if ( isset( $_POST['kblog_mathjax_latex_save_field'] ) && // input var okay
+ wp_verify_nonce( sanitize_text_field( $_POST['kblog_mathjax_latex_save_field'] ), 'kblog_mathjax_latex_save_action' ) // input var okay
) {
echo "
\n";
$this->admin_save();
@@ -155,65 +153,64 @@ function config_options() {
}
function admin_save() {
- update_option( 'kblog_mathjax_force_load', array_key_exists( 'kblog_mathjax_force_load', $_POST ) );
+ update_option( 'kblog_mathjax_force_load', array_key_exists( 'kblog_mathjax_force_load', $_POST ) ); // input var okay
- if ( array_key_exists( 'kblog_mathjax_latex_inline', $_POST ) &&
- in_array( $_POST['kblog_mathjax_latex_inline'], array( 'inline', 'display' ) )
+ if ( array_key_exists( 'kblog_mathjax_latex_inline', $_POST ) && isset( $_POST['kblog_mathjax_latex_inline'] ) && // input var okay
+ in_array( sanitize_text_field( $_POST['kblog_mathjax_latex_inline'] ), array( 'inline', 'display' ) ) // input var okay
) {
- update_option( 'kblog_mathjax_latex_inline', $_POST['kblog_mathjax_latex_inline'] );
+ update_option( 'kblog_mathjax_latex_inline', sanitize_text_field( $_POST['kblog_mathjax_latex_inline'] ) ); // input var okay
}
- update_option( 'kblog_mathjax_use_wplatex_syntax', array_key_exists( 'kblog_mathjax_use_wplatex_syntax', $_POST ) );
+ update_option( 'kblog_mathjax_use_wplatex_syntax', array_key_exists( 'kblog_mathjax_use_wplatex_syntax', $_POST ) ); // input var okay
- update_option( 'kblog_mathjax_use_cdn', array_key_exists( 'kblog_mathjax_use_cdn', $_POST ) );
+ update_option( 'kblog_mathjax_use_cdn', array_key_exists( 'kblog_mathjax_use_cdn', $_POST ) ); // input var okay
- if ( array_key_exists( 'kblog_mathjax_custom_location', $_POST ) ) {
- update_option( 'kblog_mathjax_custom_location', esc_url_raw( $_POST['kblog_mathjax_custom_location'] ) );
+ if ( array_key_exists( 'kblog_mathjax_custom_location', $_POST ) && isset( $_POST['kblog_mathjax_custom_location'] ) ) { // input var okay
+ update_option( 'kblog_mathjax_custom_location', esc_url_raw( $_POST['kblog_mathjax_custom_location'] ) ); // input var okay
}
- if ( array_key_exists( 'kblog_mathjax_config', $_POST ) &&
- in_array( $_POST['kblog_mathjax_config'], $this->config_options() )
+ if ( array_key_exists( 'kblog_mathjax_config', $_POST ) && isset( $_POST['kblog_mathjax_config'] ) && // input var okay
+ in_array( sanitize_text_field( $_POST['kblog_mathjax_config'] ), $this->config_options() ) // input var okay
) {
- update_option( 'kblog_mathjax_config', $_POST['kblog_mathjax_config'] );
+ update_option( 'kblog_mathjax_config', sanitize_text_field( $_POST['kblog_mathjax_config'] ) ); // input var okay
}
}
- function table_head( $nonce ) {
- echo <<
-Mathjax-Latex by Kblog
-