diff --git a/index.php b/index.php index 5600588..4b12d05 100644 --- a/index.php +++ b/index.php @@ -13,27 +13,56 @@ if ($conn->connect_error) { die("Conexión fallida: " . $conn->connect_error); } -// Vulnerabilidad de SQL Injection -// El siguiente código es vulnerable a SQL Injection ya que el input del usuario se concatena directamente en la consulta SQL sin validación o sanitización. -if(isset($_GET['id'])) { - $id = $_GET['id']; // Input del usuario tomado directamente desde la URL - $sql = "SELECT * FROM usuarios WHERE id = $id"; // Vulnerable a SQL Injection - $result = $conn->query($sql); +// ejemplo Julian - if ($result->num_rows > 0) { - while($row = $result->fetch_assoc()) { - echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; - } +// Corrección: Uso de sentencias preparadas para evitar SQL Injection +if (isset($_GET['id'])) { + // Validar y sanitizar el parámetro 'id' + $id_raw = $_GET['id']; + $id_valid = filter_var($id_raw, FILTER_VALIDATE_INT); + + if ($id_valid === false) { + // El id no es un entero válido + echo "ID inválido"; } else { - echo "0 resultados"; + $id = (int) $id_valid; + + // Preparar la sentencia para evitar inyección SQL + $stmt = $conn->prepare("SELECT id, nombre FROM usuarios WHERE id = ?"); + if ($stmt === false) { + // Error en la preparación + error_log("Fallo al preparar la consulta: " . $conn->error); + echo "Error en la consulta"; + } else { + $stmt->bind_param("i", $id); + $stmt->execute(); + + // Obtener resultados usando bind_result para mayor compatibilidad + $stmt->bind_result($res_id, $res_nombre); + + $hasResults = false; + while ($stmt->fetch()) { + $hasResults = true; + // Escapar la salida para prevenir XSS + $safe_nombre = htmlspecialchars($res_nombre, ENT_QUOTES, 'UTF-8'); + $safe_id = htmlspecialchars((string)$res_id, ENT_QUOTES, 'UTF-8'); + echo "id: " . $safe_id . " - Nombre: " . $safe_nombre . "
"; + } + + if (! $hasResults) { + echo "0 resultados"; + } + + $stmt->close(); + } } } -// Vulnerabilidad de Cross-Site Scripting (XSS) -// El siguiente código es vulnerable a XSS ya que imprime directamente en el HTML el contenido de una variable que puede ser manipulada por el usuario sin ninguna sanitización. -if(isset($_GET['mensaje'])) { +// Corregir vulnerabilidad XSS al mostrar 'mensaje' +if (isset($_GET['mensaje'])) { $mensaje = $_GET['mensaje']; // Input del usuario susceptible a XSS - echo "
$mensaje
"; // Vulnerable a XSS + $safe_mensaje = htmlspecialchars($mensaje, ENT_QUOTES, 'UTF-8'); + echo "
" . $safe_mensaje . "
"; } // Cerrar conexión