diff --git a/public_html/entreaulas/_filefetch.php b/public_html/entreaulas/_filefetch.php index db239db..c2e012d 100755 --- a/public_html/entreaulas/_filefetch.php +++ b/public_html/entreaulas/_filefetch.php @@ -7,9 +7,14 @@ header("Access-Control-Allow-Origin: *"); switch ($_GET["type"]) { case "alumno_photo": - $centro = str_replace('..', '_', $_GET["centro"] ?? ''); - $aulario = str_replace('..', '_', $_GET["aulario"] ?? ''); - $alumno = str_replace('..', '_', $_GET["alumno"] ?? ''); + $centro = basename($_GET["centro"] ?? ''); + $aulario = basename($_GET["aulario"] ?? ''); + $alumno = basename($_GET["alumno"] ?? ''); + // Additional validation to prevent empty names + if (empty($centro) || empty($aulario) || empty($alumno)) { + header("HTTP/1.1 400 Bad Request"); + die("Invalid parameters"); + } $relpath = "entreaulas/Centros/$centro/Aularios/$aulario/Alumnos/$alumno/photo.jpg"; break; case "panel_actividades": diff --git a/public_html/entreaulas/paneldiario.php b/public_html/entreaulas/paneldiario.php index aab0f68..e548c18 100755 --- a/public_html/entreaulas/paneldiario.php +++ b/public_html/entreaulas/paneldiario.php @@ -152,8 +152,14 @@ switch ($_GET["action"]) { break; case "quien_soy": // ¿Quién soy? - Identificación del alumno - $aulario_id = $_GET["aulario"] ?? ""; - $centro_id = $_SESSION["auth_data"]["entreaulas"]["centro"] ?? ""; + $aulario_id = basename($_GET["aulario"] ?? ''); + $centro_id = basename($_SESSION["auth_data"]["entreaulas"]["centro"] ?? ''); + + // Validate parameters + if (empty($aulario_id) || empty($centro_id)) { + echo '

Error: Parámetros inválidos.

'; + break; + } $alumnos_path = "/DATA/entreaulas/Centros/$centro_id/Aularios/$aulario_id/Alumnos"; $alumnos = []; @@ -166,7 +172,7 @@ switch ($_GET["action"]) { function seleccionarAlumno(element, nombre) { element.style.backgroundColor = "#9cff9f"; // Verde announceAndMaybeRedirect( - nombre + ", ¡Hola " + nombre + "!", + "¡Hola " + nombre + "!", "/entreaulas/paneldiario.php?aulario=", true ); @@ -193,7 +199,7 @@ switch ($_GET["action"]) { $photo_path = $alumno_path . "/photo.jpg"; $photo_exists = file_exists($photo_path); ?> - +