This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[review] Add static_asserts for the sizes of space-critical structs
- From: "Christian Biesinger (Code Review)" <gerrit at gnutoolchain-gerrit dot osci dot io>
- To: gdb-patches at sourceware dot org
- Cc: Christian Biesinger <cbiesinger at google dot com>
- Date: Fri, 25 Oct 2019 14:42:48 -0400
- Subject: [review] Add static_asserts for the sizes of space-critical structs
- Auto-submitted: auto-generated
- References: <gerrit.1572028967000.Idd68320aa3e79ee7cc749019724636a58ce4b9c6@gnutoolchain-gerrit.osci.io>
- Reply-to: cbiesinger at google dot com, cbiesinger at google dot com, gdb-patches at sourceware dot org
Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/306
......................................................................
Add static_asserts for the sizes of space-critical structs
Specifically the three structs mentioned in symtab.h:
- general_symbol_info
- symbol
- partial_symbol
This ensures that those structs won't accidentally get bigger.
gdb/ChangeLog:
2019-10-25 Christian Biesinger <cbiesinger@google.com>
* psympriv.h: Add static_asserts for sizeof (general_symbol_info)
and sizeof (symbol).
* symtab.h: Add a static_assert for sizeof (partial_symbol).
Change-Id: Idd68320aa3e79ee7cc749019724636a58ce4b9c6
---
M gdb/psympriv.h
M gdb/symtab.h
2 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/gdb/psympriv.h b/gdb/psympriv.h
index 3e89742..b32311c 100644
--- a/gdb/psympriv.h
+++ b/gdb/psympriv.h
@@ -81,6 +81,10 @@
ENUM_BITFIELD(address_class) aclass : SYMBOL_ACLASS_BITS;
};
+gdb_static_assert ((sizeof (void *) == 8 && sizeof (partial_symbol) == 40)
+ || (sizeof (void *) == 4 &&
+ sizeof (partial_symbol) == 24));
+
/* A convenience enum to give names to some constants used when
searching psymtabs. This is internal to psymtab and should not be
used elsewhere. */
diff --git a/gdb/symtab.h b/gdb/symtab.h
index dc65409..e02e49b 100644
--- a/gdb/symtab.h
+++ b/gdb/symtab.h
@@ -445,6 +445,10 @@
short section;
};
+gdb_static_assert ((sizeof (void *) == 8 && sizeof (general_symbol_info) == 32)
+ || (sizeof (void *) == 4 &&
+ sizeof (general_symbol_info) == 20));
+
extern void symbol_set_demangled_name (struct general_symbol_info *,
const char *,
struct obstack *);
@@ -1182,6 +1186,9 @@
struct symbol *hash_next;
};
+gdb_static_assert ((sizeof (void *) == 8 && sizeof (symbol) == 72) ||
+ (sizeof (void *) == 4 && sizeof (symbol) == 40));
+
/* Several lookup functions return both a symbol and the block in which the
symbol is found. This structure is used in these cases. */
--
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: Idd68320aa3e79ee7cc749019724636a58ce4b9c6
Gerrit-Change-Number: 306
Gerrit-PatchSet: 1
Gerrit-Owner: Christian Biesinger <cbiesinger@google.com>
Gerrit-MessageType: newchange