diff options
author | midipix <writeonce@midipix.org> | 2024-02-06 20:43:10 +0000 |
---|---|---|
committer | midipix <writeonce@midipix.org> | 2024-02-07 03:50:47 +0000 |
commit | 4223c83c4bb6ea3aeee68f1810ca86a8fb785f86 (patch) | |
tree | dcc2c7bcd01a840d6ba75bd3c63f842e2f4c4d58 | |
parent | 8a77f66005f77d9f8173c1e646ff521517d9c452 (diff) | |
download | slibtool-4223c83c4bb6ea3aeee68f1810ca86a8fb785f86.tar.bz2 slibtool-4223c83c4bb6ea3aeee68f1810ca86a8fb785f86.tar.xz |
slbt_get_archive_meta(): bsd symrefs vector: fix ar_name_offset logic.
-rw-r--r-- | src/arbits/slbt_archive_meta.c | 6 | ||||
-rw-r--r-- | src/arbits/slbt_armap_bsd_32.c | 13 | ||||
-rw-r--r-- | src/arbits/slbt_armap_bsd_64.c | 17 |
3 files changed, 30 insertions, 6 deletions
diff --git a/src/arbits/slbt_archive_meta.c b/src/arbits/slbt_archive_meta.c index 1af0ab8..8c8f999 100644 --- a/src/arbits/slbt_archive_meta.c +++ b/src/arbits/slbt_archive_meta.c @@ -762,7 +762,8 @@ int slbt_get_archive_meta( symrefs_32 = m->armaps.armap_symrefs_32; for (idx=0; idx<m->armaps.armap_nsyms; idx++) { - symrefs_32[idx].ar_name_offset = m->symstrv[idx] - m->symstrv[0]; + if (m->armaps.armap_common_32.ar_armap_attr & AR_ARMAP_ATTR_SYSV) + symrefs_32[idx].ar_name_offset = m->symstrv[idx] - m->symstrv[0]; if (!slbt_archive_member_from_offset(m,symrefs_32[idx].ar_member_offset)) return slbt_free_archive_meta_impl( @@ -787,7 +788,8 @@ int slbt_get_archive_meta( symrefs_64 = m->armaps.armap_symrefs_64; for (idx=0; idx<m->armaps.armap_nsyms; idx++) { - symrefs_64[idx].ar_name_offset = m->symstrv[idx] - m->symstrv[0]; + if (m->armaps.armap_common_64.ar_armap_attr & AR_ARMAP_ATTR_SYSV) + symrefs_64[idx].ar_name_offset = m->symstrv[idx] - m->symstrv[0]; if (!slbt_archive_member_from_offset(m,symrefs_64[idx].ar_member_offset)) return slbt_free_archive_meta_impl( diff --git a/src/arbits/slbt_armap_bsd_32.c b/src/arbits/slbt_armap_bsd_32.c index ed236b6..f41c500 100644 --- a/src/arbits/slbt_armap_bsd_32.c +++ b/src/arbits/slbt_armap_bsd_32.c @@ -118,13 +118,22 @@ int slbt_ar_parse_primary_armap_bsd_32( mark = armap->ar_first_name_offset; symrefs = m->armaps.armap_symrefs_32; - for (idx=0,uch=*++mark; idx<nsyms; idx++,uch=*++mark) { + for (idx=0; idx<nsyms; idx++) { + uch = *mark++; + + uref = (attr == AR_ARMAP_ATTR_BE_32) + ? (uch[0] << 24) + (uch[1] << 16) + (uch[2] << 8) + uch[3] + : (uch[3] << 24) + (uch[2] << 16) + (uch[1] << 8) + uch[0]; + + symrefs[idx].ar_name_offset = uref; + + uch = *mark++; + uref = (attr == AR_ARMAP_ATTR_BE_32) ? (uch[0] << 24) + (uch[1] << 16) + (uch[2] << 8) + uch[3] : (uch[3] << 24) + (uch[2] << 16) + (uch[1] << 8) + uch[0]; symrefs[idx].ar_member_offset = uref; - mark++; } armap->ar_string_table = m->symstrv; diff --git a/src/arbits/slbt_armap_bsd_64.c b/src/arbits/slbt_armap_bsd_64.c index b7b7d6d..6dcc737 100644 --- a/src/arbits/slbt_armap_bsd_64.c +++ b/src/arbits/slbt_armap_bsd_64.c @@ -127,7 +127,21 @@ int slbt_ar_parse_primary_armap_bsd_64( mark = armap->ar_first_name_offset; symrefs = m->armaps.armap_symrefs_64; - for (idx=0,uch=*++mark; idx<nsyms; idx++,uch=*++mark) { + for (idx=0; idx<nsyms; idx++) { + uch = *mark++; + + if (attr == AR_ARMAP_ATTR_BE_64) { + uref_hi = (uch[0] << 24) + (uch[1] << 16) + (uch[2] << 8) + uch[3]; + uref_lo = (uch[4] << 24) + (uch[5] << 16) + (uch[6] << 8) + uch[7]; + } else { + uref_lo = (uch[3] << 24) + (uch[2] << 16) + (uch[1] << 8) + uch[0]; + uref_hi = (uch[7] << 24) + (uch[6] << 16) + (uch[5] << 8) + uch[4]; + } + + symrefs[idx].ar_name_offset = (uref_hi << 32) + uref_lo; + + uch = *mark++; + if (attr == AR_ARMAP_ATTR_BE_64) { uref_hi = (uch[0] << 24) + (uch[1] << 16) + (uch[2] << 8) + uch[3]; uref_lo = (uch[4] << 24) + (uch[5] << 16) + (uch[6] << 8) + uch[7]; @@ -137,7 +151,6 @@ int slbt_ar_parse_primary_armap_bsd_64( } symrefs[idx].ar_member_offset = (uref_hi << 32) + uref_lo; - mark++; } armap->ar_string_table = m->symstrv; |