From: tromey <tromey>
Date: Fri, 6 Nov 2009 07:19:09 +0000 (+0000)
Subject: 	* symfile.c (add_psymbol_to_bcache): Make 'psymbol' static again.
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=e0aabeb00eef432e9fb3a00bb4eae155fd14ccf5;p=pf3gnuchains%2Fpf3gnuchains3x.git

	* symfile.c (add_psymbol_to_bcache): Make 'psymbol' static again.
	Zero the 'value' field.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 659e9d12a6..bf20ded4d3 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2009-11-05  Tom Tromey  <tromey@redhat.com>
+
+	* symfile.c (add_psymbol_to_bcache): Make 'psymbol' static again.
+	Zero the 'value' field.
+
 2009-11-05  Doug Evans  <dje@google.com>
 
 	* amd64-tdep.c (amd64_init_frame_cache): Enhance comment describing
diff --git a/gdb/symfile.c b/gdb/symfile.c
index 7461ed6288..117f17fb16 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -3105,9 +3105,15 @@ add_psymbol_to_bcache (char *name, int namelength, domain_enum domain,
 		       enum language language, struct objfile *objfile,
 		       int *added)
 {
-  struct partial_symbol psymbol;
-
-  memset (&psymbol, 0, sizeof (struct partial_symbol));
+  /* psymbol is static so that there will be no uninitialized gaps in the
+     structure which might contain random data, causing cache misses in
+     bcache. */
+  static struct partial_symbol psymbol;
+
+  /* However, we must ensure that the entire 'value' field has been
+     zeroed before assigning to it, because an assignment may not
+     write the entire field.  */
+  memset (&psymbol.ginfo.value, 0, sizeof (psymbol.ginfo.value));
   /* val and coreaddr are mutually exclusive, one of them *will* be zero */
   if (val != 0)
     {