diff --git a/index.php b/index.php index 5600588..b5c7339 100644 --- a/index.php +++ b/index.php @@ -17,19 +17,38 @@ if ($conn->connect_error) { // 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); - - if ($result->num_rows > 0) { - while($row = $result->fetch_assoc()) { - echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; - } + // Validate that id is an integer (whitelist) before using it in a query + if (filter_var($id, FILTER_VALIDATE_INT) === false) { + echo "Entrada 'id' inválida"; } else { - echo "0 resultados"; + $id = (int)$id; + // Use a prepared statement to avoid SQL injection + $stmt = $conn->prepare("SELECT * FROM usuarios WHERE id = ?"); + if ($stmt) { + $stmt->bind_param("i", $id); + $stmt->execute(); + $result = $stmt->get_result(); + + if ($result === false) { + echo "Error en la consulta"; + } else { + if ($result->num_rows > 0) { + while($row = $result->fetch_assoc()) { + echo "id: " . $row["id"]. " - Nombre: " . $row["nombre"]. "
"; + } + } else { + echo "0 resultados"; + } + } + + $stmt->close(); + } else { + echo "Error al preparar 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