Skip to content

Commit

Permalink
Security patch
Browse files Browse the repository at this point in the history
  • Loading branch information
sc0ttkclark committed May 7, 2024
1 parent aa08f03 commit 776893e
Show file tree
Hide file tree
Showing 6 changed files with 46 additions and 5 deletions.
9 changes: 9 additions & 0 deletions classes/Pods.php
Original file line number Diff line number Diff line change
Expand Up @@ -4112,6 +4112,15 @@ public function form( $params = null, $label = null, $thank_you = null ) {
$thank_you = $params['thank_you'];
$fields_only = $params['fields_only'];

// Sanitize thank_you for security.
if ( ! empty( $thank_you ) ) {
// Additional sanitization.
$thank_you = sanitize_text_field( $thank_you );

// Fallback to '' so that the logic below can kick in if the thank you URL was not safe.
$thank_you = pods_enforce_safe_url( $thank_you, '' );
}

PodsForm::$form_counter ++;

ob_start();
Expand Down
20 changes: 20 additions & 0 deletions includes/data.php
Original file line number Diff line number Diff line change
Expand Up @@ -2452,3 +2452,23 @@ function pods_list_filter( $list, $args = array(), $operator = 'AND' ) {

return $filtered;
}

/**
* Enforce a URL as safe and fallback to another URL if it is not safe.
*
* @since 2.7.31.3
*
* @param string $url The URL to enforce as safe.
* @param string|null $fallback_url The fallback URL to use if the URL is not valid.
*
* @return string The safe URL or the fallback URL if that was not valid.
*/
function pods_enforce_safe_url( $url, $fallback_url = null ) {
$url = wp_sanitize_redirect( $url );

if ( null === $fallback_url ) {
$fallback_url = pods_current_url();
}

return wp_validate_redirect( $url, $fallback_url );
}
4 changes: 2 additions & 2 deletions init.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Plugin Name: Pods - Custom Content Types and Fields
Plugin URI: https://pods.io/
Description: Pods is a framework for creating, managing, and deploying customized content types and fields
Version: 2.7.31.1
Version: 2.7.31.3
Author: Pods Framework Team
Author URI: https://pods.io/about/
Text Domain: pods
Expand Down Expand Up @@ -37,7 +37,7 @@
add_action( 'init', 'pods_deactivate_pods_ui' );
} else {
// Current version
define( 'PODS_VERSION', '2.7.31.1' );
define( 'PODS_VERSION', '2.7.31.3' );

// Version tracking between DB updates themselves
define( 'PODS_DB_VERSION', '2.3.5' );
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "pods",
"version": "2.7.31.1",
"version": "2.7.31.3",
"description": "Pods is a development framework for creating, extending, managing, and deploying customized content types in WordPress.",
"author": "Pods Foundation, Inc",
"homepage": "https://pods.io/",
Expand Down
14 changes: 13 additions & 1 deletion readme.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ Tags: pods, custom post types, custom taxonomies, content types, custom fields,
Requires at least: 4.5
Tested up to: 5.8
Requires PHP: 5.3
Stable tag: 2.7.31.1
Stable tag: 2.7.31.3
License: GPLv2 or later
License URI: http://www.gnu.org/licenses/gpl-2.0.html

Expand Down Expand Up @@ -190,6 +190,18 @@ We are also available through our [Live Slack Chat](https://pods.io/chat/) to he

== Changelog ==

= 2.7.31.3 - May 8th, 2024 =

*Security Release*

* Security hardening: Enforce safe URLs for Pods form submission confirmation page URLs. Props to the wesley (wcraft) / Wordfence for responsibly reporting this. (@sc0ttkclark)

= 2.7.31.2 - February 21st, 2024 =

*Security Release*

Resolved issue with release deployment, see previous version for more details about the release.

= 2.7.31.1 - February 21st, 2024 =

*Security Release*
Expand Down
2 changes: 1 addition & 1 deletion ui/front/form.php
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@
if ( !$fields_only ) {
$field_prefix = 'pods_field_';
?>
<form action="" method="post" class="pods-submittable pods-form pods-form-front pods-form-pod-<?php echo esc_attr( $pod->pod ); ?> pods-submittable-ajax" data-location="<?php echo esc_attr( $thank_you ); ?>">
<form action="" method="post" class="pods-submittable pods-form pods-form-front pods-form-pod-<?php echo esc_attr( $pod->pod ); ?> pods-submittable-ajax" data-location="<?php echo esc_attr( pods_enforce_safe_url( $thank_you ) ); ?>">
<div class="pods-submittable-fields">
<?php echo PodsForm::field( 'action', 'pods_admin', 'hidden' ); ?>
<?php echo PodsForm::field( 'method', 'process_form', 'hidden' ); ?>
Expand Down

0 comments on commit 776893e

Please sign in to comment.