From: Michael Snyder <msnyder@vmware.com>
Date: Tue, 1 Mar 2011 20:24:48 +0000 (+0000)
Subject: 2011-03-01  Michael Snyder  <msnyder@vmware.com>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d684ab85a385dc954fe05f3dc3de69069df31d7b;p=binutils-gdb.git

2011-03-01  Michael Snyder  <msnyder@vmware.com>

	* reverse.c (record_restore): Move null-check to before pointer
	dereference.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 81957f4ce85..e380be2826f 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,8 @@
 2011-03-01  Michael Snyder  <msnyder@vmware.com>
 
+	* reverse.c (record_restore): Move null-check to before pointer
+	dereference.
+
 	* python/py-utils.c (gdbpy_obj_to_string): Delete unused variable.
 
 	* objc-lang.c (selectors_info): Add explanitory comment.
diff --git a/gdb/record.c b/gdb/record.c
index 784f5fadf82..35673826f2e 100644
--- a/gdb/record.c
+++ b/gdb/record.c
@@ -2214,12 +2214,12 @@ record_restore (void)
 
   /* Now need to find our special note section.  */
   osec = bfd_get_section_by_name (core_bfd, "null0");
+  if (osec == NULL)
+    return;
   osec_size = bfd_section_size (core_bfd, osec);
   if (record_debug)
     fprintf_unfiltered (gdb_stdlog, "Find precord section %s.\n",
 			osec ? "succeeded" : "failed");
-  if (osec == NULL)
-    return;
   if (record_debug)
     fprintf_unfiltered (gdb_stdlog, "%s", bfd_section_name (core_bfd, osec));