If high_pc uses DW_FORM_addr, it's not size but address

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@65101 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 23 insertions, 26 deletions

diff --git a/addr2line.c b/addr2line.c
index 5b9a35a089..928a82c264 100644
--- a/addr2line.c
+++ b/addr2line.c
@@ -937,14 +937,12 @@ di_read_cu(DebugInfoReader *reader)
         reader->q0 = reader->obj->debug_abbrev.ptr + hdr->debug_abbrev_offset;
         reader->address_size = hdr->address_size;
         reader->format = 64;
-        if (hdr->version != 4) return -1;
     } else {
         DW_CompilationUnitHeader32 *hdr = hdr32;
         reader->p += 11;
         reader->q0 = reader->obj->debug_abbrev.ptr + hdr->debug_abbrev_offset;
         reader->address_size = hdr->address_size;
         reader->format = 32;
-        if (hdr->version != 4) return -1;
     }
     reader->level = 0;
     di_read_debug_abbrev_cu(reader);
@@ -1318,24 +1316,27 @@ typedef struct {
 } ranges_t;
 
 static void
-ranges_set_low_pc(ranges_t *ptr, uint64_t low_pc)
+ranges_set(ranges_t *ptr, DebugInfoValue *v)
 {
-    ptr->low_pc = low_pc;
-    ptr->low_pc_set = true;
-}
-
-static void
-ranges_set_high_pc(ranges_t *ptr, uint64_t high_pc)
-{
-    ptr->high_pc = high_pc;
-    ptr->high_pc_set = true;
-}
-
-static void
-ranges_set_ranges(ranges_t *ptr, uint64_t ranges)
-{
-    ptr->ranges = ranges;
-    ptr->ranges_set = true;
+    switch (v->at) {
+      case DW_AT_low_pc:
+        ptr->low_pc = v->as.uint64;
+        ptr->low_pc_set = true;
+        break;
+      case DW_AT_high_pc:
+        if (v->form == DW_FORM_addr) {
+            ptr->high_pc = v->as.uint64;
+        }
+        else {
+            ptr->high_pc = ptr->low_pc + v->as.uint64;
+        }
+        ptr->high_pc_set = true;
+        break;
+      case DW_AT_ranges:
+        ptr->ranges = v->as.uint64;
+        ptr->ranges_set = true;
+        break;
+    }
 }
 
 static uintptr_t
@@ -1345,7 +1346,7 @@ ranges_include(DebugInfoReader *reader, ranges_t *ptr, uint64_t addr)
         if (ptr->ranges_set || !ptr->low_pc_set) {
             exit(1);
         }
-        if (ptr->low_pc <= addr && addr <= ptr->low_pc + ptr->high_pc) {
+        if (ptr->low_pc <= addr && addr <= ptr->high_pc) {
             return ptr->low_pc;
         }
     }
@@ -1463,10 +1464,9 @@ debug_info_read(DebugInfoReader *reader, int num_traces, void **traces,
                 line.line = (int)v.as.uint64;
                 break;
               case DW_AT_low_pc:
-                ranges_set_low_pc(&ranges, v.as.uint64);
-                break;
               case DW_AT_high_pc:
-                ranges_set_high_pc(&ranges, v.as.uint64);
+              case DW_AT_ranges:
+                ranges_set(&ranges, &v);
                 break;
               case DW_AT_declaration:
                 goto skip_die;
@@ -1476,9 +1476,6 @@ debug_info_read(DebugInfoReader *reader, int num_traces, void **traces,
               case DW_AT_abstract_origin:
                 read_abstract_origin(reader, v.as.uint64, &line);
                 break; //goto skip_die;
-              case DW_AT_ranges:
-                ranges_set_ranges(&ranges, v.as.uint64);
-                break;
             }
         }
         /* ranges_inspect(reader, &ranges); */