diff --git a/index.php b/index.php index 5600588..f0d575c 100644 --- a/index.php +++ b/index.php @@ -13,25 +13,42 @@ 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); + // 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'])) { + // Tomar y validar el parámetro 'id' como entero + $raw_id = $_GET['id']; // Input del usuario tomado directamente desde la URL + $id = filter_var($raw_id, FILTER_VALIDATE_INT); + if ($id === false) { + // Parámetro no válido: no ejecutar la consulta y devolver un mensaje genérico + echo "Parámetro 'id' inválido"; + } else { + // Uso de prepared statement para evitar SQL Injection + $stmt = $conn->prepare("SELECT id, nombre FROM usuarios WHERE id = ?"); + if ($stmt) { + $stmt->bind_param("i", $id); + $stmt->execute(); + $result = $stmt->get_result(); - if ($result->num_rows > 0) { - while($row = $result->fetch_assoc()) { - echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; - } - } else { - echo "0 resultados"; - } -} + if ($result && $result->num_rows > 0) { + while($row = $result->fetch_assoc()) { + echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; + } + } else { + echo "0 resultados"; + } + + $stmt->close(); + } else { + // No exponer detalles internos en caso de error + 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. -if(isset($_GET['mensaje'])) { + $mensaje = $_GET['mensaje']; // Input del usuario susceptible a XSS echo "
$mensaje
"; // Vulnerable a XSS }