Possible bug in MediumLevelILFunction::GetLLILSSAToMLILExprMap

Abierto
#7,478 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
cpp
Área
reverse-engineering

Línea de trabajo

Start in mediumlevelil.cpp at MediumLevelILFunction::GetLLILSSAToMLILExprMap and inspect the translation data fields used in the loop, especially oldExprIndex, oldLLILSSADirect, and oldLLILSSAIndex. Confirm whether the comparison mixes MLIL and LLIL expression indices; done means the index comparison is consistent and the relevant behavior is covered by existing validation.

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

Descripción

Version and Platform (required):

  • Binary Ninja Version: dev/5.2.8353

Bug Description:
In mediumlevelil.cpp, in MediumLevelILFunction::GetLLILSSAToMLILExprMap, I see this code:

for (auto& [oldExprIndex, newExprIndices]: m_translationData->mlilToMlilExprMap)
    [..]
        size_t oldLLILSSADirect = m_translationData->copyingFunction->GetLowLevelILExprIndex(oldExprIndex);
        [..]
                info.higherToLowerDirect = newDirect && oldExprIndex == oldLLILSSADirect;

I haven't tried to test/reproduce this, but it seems wrong that oldExprIndex (an MLIL expression index) is being compared to both oldLLILSSADirect (an LLIL expression index). I think this is probably supposed to be comparing oldLLILSSADirect to oldLLILSSAIndex.

Lenguaje dominante
C++
Estrellas
1.3k
Forks
298
Merge medio
5 d 5 h
PR fusionados (30 d)
19

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

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 Vector35/binaryninja-api

Todos los issues de Vector35/binaryninja-api

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.