aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPedro Franco de Carvalho <pedromfc@linux.ibm.com>2019-08-09 16:49:46 -0300
committerPedro Franco de Carvalho <pedromfc@linux.ibm.com>2019-08-09 16:49:46 -0300
commitdac36daf78653a8e4688b825cdb15461a88470d2 (patch)
tree79c9da1215c992c5b640fa47f1749d6caeb71c65
parent1022c627dbd9d7f7f67ac68f16de05474de7a75a (diff)
downloadbinutils-dac36daf78653a8e4688b825cdb15461a88470d2.zip
binutils-dac36daf78653a8e4688b825cdb15461a88470d2.tar.gz
binutils-dac36daf78653a8e4688b825cdb15461a88470d2.tar.bz2
Fix access to uninitialized variable in fill_in_stop_func
This patch changes find_pc_partial_function so that *block is set to nullptr when it fails, so that fill_in_stop_func won't access an uninitialized variable. gdb/ChangeLog: 2019-08-09 Pedro Franco de Carvalho <pedromfc@linux.ibm.com> * blockframe.c (find_pc_partial_function): Set *block to nullptr when the function fails.
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/blockframe.c2
2 files changed, 7 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 00e4bac..4008783 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2019-08-09 Pedro Franco de Carvalho <pedromfc@linux.ibm.com>
+
+ * blockframe.c (find_pc_partial_function): Set *block to nullptr
+ when the function fails.
+
2019-08-09 Andreas Arnez <arnez@linux.ibm.com>
* s390-tdep.c (s390_type_align): New function.
diff --git a/gdb/blockframe.c b/gdb/blockframe.c
index fe7807b..4462274 100644
--- a/gdb/blockframe.c
+++ b/gdb/blockframe.c
@@ -331,6 +331,8 @@ find_pc_partial_function (CORE_ADDR pc, const char **name, CORE_ADDR *address,
*address = 0;
if (endaddr != NULL)
*endaddr = 0;
+ if (block != nullptr)
+ *block = nullptr;
return 0;
}