summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authormidipix <writeonce@midipix.org>2024-01-28 16:17:45 +0000
committermidipix <writeonce@midipix.org>2024-01-28 16:17:45 +0000
commit1f3f88c2a5ae86bfe910ff7760a57d9b8396ccb4 (patch)
tree824968085e67a36a122858a4b30ae4f61f2536a0
parent2f121c3724d807349b360f8f896f62c2f9dbff90 (diff)
downloadslibtool-1f3f88c2a5ae86bfe910ff7760a57d9b8396ccb4.tar.bz2
slibtool-1f3f88c2a5ae86bfe910ff7760a57d9b8396ccb4.tar.xz
slbt_get_archive_meta(): symrefs vector: validate all member offsets.
-rw-r--r--include/slibtool/slibtool.h1
-rw-r--r--src/arbits/slbt_archive_meta.c25
2 files changed, 21 insertions, 5 deletions
diff --git a/include/slibtool/slibtool.h b/include/slibtool/slibtool.h
index 49e8dbf..04917c5 100644
--- a/include/slibtool/slibtool.h
+++ b/include/slibtool/slibtool.h
@@ -114,6 +114,7 @@ enum slbt_custom_error {
SLBT_ERR_AR_INVALID_ARMAP_SIZE_OF_REFS,
SLBT_ERR_AR_INVALID_ARMAP_SIZE_OF_STRS,
SLBT_ERR_AR_INVALID_ARMAP_STRING_TABLE,
+ SLBT_ERR_AR_INVALID_ARMAP_MEMBER_OFFSET,
};
/* execution modes */
diff --git a/src/arbits/slbt_archive_meta.c b/src/arbits/slbt_archive_meta.c
index 0aebbe3..75de64d 100644
--- a/src/arbits/slbt_archive_meta.c
+++ b/src/arbits/slbt_archive_meta.c
@@ -1238,6 +1238,9 @@ int slbt_get_archive_meta(
}
}
+ /* number of archive members, including internal ones */
+ m->nentries = nentries;
+
/* primary armap (first linker member) */
if (slbt_ar_parse_primary_armap(dctx,m) < 0)
return slbt_free_archive_meta_impl(
@@ -1252,15 +1255,30 @@ int slbt_get_archive_meta(
if (m->armaps.armap_common_32.ar_member) {
symrefs_32 = m->armaps.armap_symrefs_32;
- for (idx=0; idx<m->armaps.armap_nsyms; idx++)
+ for (idx=0; idx<m->armaps.armap_nsyms; idx++) {
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(
+ m,SLBT_CUSTOM_ERROR(
+ dctx,
+ SLBT_ERR_AR_INVALID_ARMAP_MEMBER_OFFSET));
+
+ }
}
if (m->armaps.armap_common_64.ar_member) {
symrefs_64 = m->armaps.armap_symrefs_64;
- for (idx=0; idx<m->armaps.armap_nsyms; idx++)
+ for (idx=0; idx<m->armaps.armap_nsyms; idx++) {
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(
+ m,SLBT_CUSTOM_ERROR(
+ dctx,
+ SLBT_ERR_AR_INVALID_ARMAP_MEMBER_OFFSET));
+ }
}
/* number of public archive members */
@@ -1282,9 +1300,6 @@ int slbt_get_archive_meta(
if (m->armaps.armap_common_64.ar_member)
m->armaps.armap_common_64.ar_num_of_members = nmembers;
- /* number of archive members, including internal ones */
- m->nentries = nentries;
-
/* pe/coff armap attributes (second linker member) */
(void)m->armeta.a_armap_pecoff;