summaryrefslogtreecommitdiff
path: root/src/format
diff options
context:
space:
mode:
Diffstat (limited to 'src/format')
-rw-r--r--src/format/format.c24
-rw-r--r--src/format/symbol.c15
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;