Windows module list walks: errors drop the whole list silently, no iteration cap

Abierto
#431 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
backend, security

Línea de trabajo

Start with the listed loops in src/os/windows/structs.rs (2866–2876) and src/os/windows/kernel.rs (297–349), then check how src/os/windows/kernel.rs (437) handles their result. Trace how list entry errors and cycles affect the returned modules, and consider the suggested per-entry handling and iteration cap. Done means errors no longer silently discard the full list, cyclic guest-controlled lists terminate, and valid entries are still returned.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Where

  • src/os/windows/structs.rs 2866-2876 with src/os/windows/kernel.rs:437 (unwrap_or_default())
  • src/os/windows/kernel.rs 297-349 (PsLoadedModuleList)

Problem
? inside the list loops; the caller swallows the error. No iteration limit on guest-controlled lists.

Impact
One paged-out entry or non-UTF-16 name yields zero user modules with no log. A cyclic Flink that misses the head grows a Vec forever.

Suggested fix
Warn and skip/break per entry, return the partial list, cap iterations.

Status
Reported by static code review at 79972db; not yet re-verified line by line. Found by reading the code; not reproduced under SIMICS.

Lenguaje dominante
Rust
Estrellas
335
Forks
26
Merge medio
1 d 50 min
PR fusionados (30 d)
12

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de intel/tsffs

Todos los issues de intel/tsffs

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.