From: Laurent Desnogues <laurent.desnogues@gmail.com>
Date: Thu, 30 Jul 2009 17:23:49 +0000 (+0200)
Subject: Fix symfind.
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=ec822001a2f26eef8701194714f6482b6d852de2;p=qemu.git

Fix symfind.

this patch fixes an issue in symfind.

Assume you have the following symbols:

Address  Size
0045bca0 00000080 T s0
0045bd20 00000112 T s1

You'll notice that s1 is s0 + size.

So the current symfind will find that address 0045bd20 belongs to s0
instead of s1.

Laurent

Signed-off-by: Laurent Desnogues <laurent.desnogues@gmail.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Message-Id:
---

diff --git a/elf_ops.h b/elf_ops.h
index 699651c9e1..15928cbb5b 100644
--- a/elf_ops.h
+++ b/elf_ops.h
@@ -67,7 +67,7 @@ static int glue(symfind, SZ)(const void *s0, const void *s1)
     int result = 0;
     if (key->st_value < sym->st_value) {
         result = -1;
-    } else if (key->st_value > sym->st_value + sym->st_size) {
+    } else if (key->st_value >= sym->st_value + sym->st_size) {
         result = 1;
     }
     return result;
diff --git a/linux-user/elfload.c b/linux-user/elfload.c
index a38167d333..4f04b98287 100644
--- a/linux-user/elfload.c
+++ b/linux-user/elfload.c
@@ -1199,7 +1199,7 @@ static int symfind(const void *s0, const void *s1)
     int result = 0;
     if (key->st_value < sym->st_value) {
         result = -1;
-    } else if (key->st_value > sym->st_value + sym->st_size) {
+    } else if (key->st_value >= sym->st_value + sym->st_size) {
         result = 1;
     }
     return result;