x86/x86_64 cmplog drops every register/memory vs immediate compare
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
- security, testing-qa
Línea de trabajo
Read the cmplog pairing matches in src/arch/x86_64.rs (lines 179–203) and src/arch/x86.rs (lines 206–230) at the linked revision. Check how yaxpeax-x86 represents immediate and register or memory operands, then trace how operand pairs are logged. Confirm the fix with tests or a SIMICS reproduction showing register/memory versus immediate comparisons are logged.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Where
- https://github.com/intel/tsffs/blob/79972db9c63eba3ac8a1ce348264eee2005b2b16/src/arch/x86_64.rs#L179-L203
- https://github.com/intel/tsffs/blob/79972db9c63eba3ac8a1ce348264eee2005b2b16/src/arch/x86.rs#L206-L230
Problem
yaxpeax-x86 yields signed immediates (I8/I16/I32/I64) while registers/memory become U*. The pairing match only accepts same-variant pairs, so (U32, I32) or (U64, I8) falls to _ => None.
Impact
cmp eax, 0x41424344 / cmp rax, 5 are never logged: magic-constant compares, the main purpose of cmplog, are lost.
Suggested fix
Convert the right operand to the left operand's width (sign-extend then truncate) before pairing.
Status
Confirmed by reading the code at 79972db. 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
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
- Sin 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 intel/tsffs
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
Todos los issues de intel/tsffs
Issues similares
-
bug UI
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
wardian-app/Wardian#1746 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
jlcodes99/cockpit-tools#3125 ·
Los mantenedores suelen responder en 2 días
-
bug user-sprint
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Vondor/KassenSturz#967 ·
Los mantenedores suelen responder en 1 día
-
area/ci
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
[BUG] Incremental sync warns "no stored highest_uid" on every round for folders with no new mailAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
rustmailer/bichon#411 ·