Forced the full definition of locations to fix the search of symbols.

git-svn-id: svn://svn.gna.org/svn/chrysalide/trunk@594 abbe820e-26c8-41b2-8c08-b7b2b41f8b0a

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;