test_lsp.py's LSan check misses leaks hidden by a stale stack pointer
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- testing-qa
Línea de trabajo
Comienza en test_lsp.py, en la comprobación de LSan «no sanitizer reports», y revisa cómo inicia eigenlsp. Ejecuta la comprobación con LSAN_OPTIONS=use_stacks=0:use_registers=0 y, a continuación, verifica que eliminar free_scopes de src/eigenlsp.c hace que falle la prueba de fuga incorporada intencionadamente. Cualquier falso positivo que quede expuesto debe corregirse o documentarse con una explicación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Found by the blind critic on #1364. With free_scopes deleted from src/eigenlsp.c (a real leak of the rename scope table), the harness's "no sanitizer reports" check passed. Under normal LSAN options: rc 0, no report. With LSAN_OPTIONS=use_stacks=0:use_registers=0: Direct leak of 512 byte(s). A stale pointer left on the stack keeps the block "reachable", so the leak class the harness claims to watch is invisible to it.
Done when
- The LSP leak check runs eigenlsp with
use_stacks=0:use_registers=0(or an equivalent that sees this class), and the plantedfree_scopesdeletion turns it red. - Any false positives that setting surfaces are fixed or listed with a reason.
- Lenguaje dominante
- C
- Estrellas
- 3
- Forks
- 7
- Merge medio
- 4 h 7 min
- PR fusionados (30 d)
- 112
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de InauguralSystems/EigenScript
-
area:embed kind:silent-wrong
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
InauguralSystems/EigenScript#1387 ·
Los mantenedores suelen responder en 1 día
-
area:stdlib kind:silent-wrong
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
InauguralSystems/EigenScript#1378 ·
Los mantenedores suelen responder en 1 día
-
Error carets pad multi-byte UTF-8 byte-for-byte, so the ^ lands right of the token on a terminalAbiertoarea:lint-tooling kind:silent-wrong
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
InauguralSystems/EigenScript#1373 ·
Los mantenedores suelen responder en 1 día
-
area:gates kind:docs-drift
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
InauguralSystems/EigenScript#1372 ·
Los mantenedores suelen responder en 1 día
-
area:lint-tooling enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
InauguralSystems/EigenScript#1371 ·
Los mantenedores suelen responder en 1 día
Todos los issues de InauguralSystems/EigenScript
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
[Issue]: Headers - vx_ext_amd.h does not compile as C (enum types used without the enum keyword)Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Qiskit/qiskit#17079 · 1 comentario ·
Los mantenedores suelen responder en 1 día