Skip to content

Commit 89a8479

Browse files
author
Chris Gårdenberg
committed
security: Fix potential LFI vulnerability
1 parent 75de724 commit 89a8479

2 files changed

Lines changed: 22 additions & 2 deletions

File tree

PLUGIN-CHECKSUM

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
0c566c97e2c900136a04c1addc96bc14
1+
b2e347591de3bd448ec824909315da47

includes/edu-shortcodes.php

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ function eduadmin_get_detail_view( $attributes ) {
107107
);
108108
wp_enqueue_style( 'eduadmin_frontend_detail' );
109109

110-
$attributes = shortcode_atts(
110+
$attributes = shortcode_atts(
111111
array(
112112
'template' => $selected_template,
113113
'courseid' => null,
@@ -118,6 +118,16 @@ function eduadmin_get_detail_view( $attributes ) {
118118
normalize_empty_atts( $attributes ),
119119
'eduadmin-detailview'
120120
);
121+
122+
switch ( $attributes['template'] ) {
123+
case "template_A":
124+
case "template_B":
125+
break;
126+
default:
127+
$attributes['template'] = "template_A";
128+
break;
129+
}
130+
121131
EDU()->session['checkEmail'] = null;
122132
EDU()->session['needsLogin'] = null;
123133
unset( EDU()->session['checkEmail'] );
@@ -197,6 +207,16 @@ function eduadmin_get_booking_view( $attributes ) {
197207
normalize_empty_atts( $attributes ),
198208
'eduadmin-bookingview'
199209
);
210+
211+
switch ( $attributes['template'] ) {
212+
case "template_A":
213+
case "template_B":
214+
break;
215+
default:
216+
$attributes['template'] = "template_A";
217+
break;
218+
}
219+
200220
if ( ! EDU()->is_checked( 'eduadmin-useLogin', false ) || ( isset( EDU()->session['eduadmin-loginUser'] ) && ( ( isset( EDU()->session['eduadmin-loginUser']->Contact->PersonId ) && 0 !== EDU()->session['eduadmin-loginUser']->Contact->PersonId ) || isset( EDU()->session['eduadmin-loginUser']->NewCustomer ) ) ) ) {
201221
$str = include_once EDUADMIN_PLUGIN_PATH . '/content/template/bookingTemplate/' . $attributes['template'] . '.php';
202222
} else {

0 commit comments

Comments
 (0)