diff --git a/index.php b/index.php index 5600588..1a17bab 100644 --- a/index.php +++ b/index.php @@ -13,27 +13,41 @@ 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. +// Mitigación: uso de prepared statements y validación de entrada 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); - - if ($result->num_rows > 0) { - while($row = $result->fetch_assoc()) { - echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; - } + $id_raw = $_GET['id']; + // Validar que el id es un entero + if (filter_var($id_raw, FILTER_VALIDATE_INT) === false) { + echo "ID inválido"; } else { - echo "0 resultados"; + $id = (int)$id_raw; + // Preparar consulta con parámetros y seleccionar columnas específicas + if ($stmt = $conn->prepare("SELECT id, nombre FROM usuarios WHERE id = ?")) { + $stmt->bind_param("i", $id); + $stmt->execute(); + $stmt->bind_result($res_id, $res_nombre); + $found = false; + while ($stmt->fetch()) { + $found = true; + // Escapar salida para prevenir XSS + echo "id: " . htmlspecialchars($res_id, ENT_QUOTES, 'UTF-8') . " - Nombre: " . htmlspecialchars($res_nombre, ENT_QUOTES, 'UTF-8') . "
"; + } + if (!$found) { + echo "0 resultados"; + } + $stmt->close(); + } else { + // No revelar detalles internos al usuario + echo "Error en la consulta"; + } } } -// 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. +// Mitigación de Cross-Site Scripting (XSS) +// Escapar la salida del mensaje para evitar ejecución de HTML/JS if(isset($_GET['mensaje'])) { - $mensaje = $_GET['mensaje']; // Input del usuario susceptible a XSS - echo "
$mensaje
"; // Vulnerable a XSS + $mensaje = $_GET['mensaje']; + echo "
" . htmlspecialchars($mensaje, ENT_QUOTES, 'UTF-8') . "
"; } // Cerrar conexión