diff options
Diffstat (limited to 'src/format')
| -rw-r--r-- | src/format/format.c | 24 | ||||
| -rw-r--r-- | src/format/symbol.c | 15 |
2 files changed, 23 insertions, 16 deletions
diff --git a/src/format/format.c b/src/format/format.c index 9f03924..9c42508 100644 --- a/src/format/format.c +++ b/src/format/format.c @@ -221,7 +221,7 @@ void g_binary_format_setup_disassembling_context(const GBinFormat *format, GProc * * * Description : Ajoute un symbole à la collection du format binaire. * * * -* Retour : - * +* Retour : true si le symbole était bien localisé et a été inséré. * * * * Remarques : - * * * @@ -229,6 +229,26 @@ void g_binary_format_setup_disassembling_context(const GBinFormat *format, GProc void _g_binary_format_add_symbol(GBinFormat *format, GBinSymbol *symbol, bool sort) { + const mrange_t *range; /* Couverture du symbole */ + const vmpa2t *addr; /* Emplacement du symbole */ + + /** + * Lorsque les fonctions de recherche type g_binary_format_find_symbol_at() + * sont appelées avec une localisation, cette dernière peut reposer soit sur + * une position physique soit une adresse virtuelle uniquement. + * + * Pour que les comparaisons de positions puissent se réaliser, il faut donc + * pouvoir satisfaire les deux aspects : physiques et virtuels. + * + * On corrige donc le tir si besoin est ici. + */ + +#ifndef NDEBUG + range = g_binary_symbol_get_range(symbol); + addr = get_mrange_addr(range); + + assert(get_phy_addr(addr) != VMPA_NO_PHYSICAL && get_virt_addr(addr) != VMPA_NO_VIRTUAL); +#endif g_rw_lock_writer_lock(&format->syms_lock); @@ -628,7 +648,7 @@ bool g_binary_format_find_symbol_at(const GBinFormat *format, const vmpa2t *addr range = g_binary_symbol_get_range(*sym); - return cmp_vmpa_by_virt(addr, get_mrange_addr(range)); + return cmp_vmpa(addr, get_mrange_addr(range)); } diff --git a/src/format/symbol.c b/src/format/symbol.c index 70a1d24..2346c63 100644 --- a/src/format/symbol.c +++ b/src/format/symbol.c @@ -202,8 +202,6 @@ int g_binary_symbol_cmp(const GBinSymbol **a, const GBinSymbol **b) int result; /* Bilan à retourner */ const mrange_t *ra; /* Emplacement du symbole A */ const mrange_t *rb; /* Emplacement du symbole B */ - const vmpa2t *aa; /* Adresse du symbole A */ - const vmpa2t *ab; /* Adresse du symbole B */ ra = g_binary_symbol_get_range(*a); rb = g_binary_symbol_get_range(*b); @@ -218,18 +216,7 @@ int g_binary_symbol_cmp(const GBinSymbol **a, const GBinSymbol **b) result = -1; else - { - aa = get_mrange_addr(ra); - ab = get_mrange_addr(rb); - - result = cmp_vmpa_by_virt(aa, ab); - - //result = aa->virtual < ab->virtual ? -1 : (aa->virtual > ab->virtual ? 1 : 0); - ///result = cmp_mrange(ra, rb); - - //printf(" ?? 0x%08lx vs 0x%08lx -> %d\n", aa->virtual, ab->virtual, result); - - } + result = cmp_mrange(ra, rb); return result; |