diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2018-04-16 16:47:06 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2018-04-16 16:47:06 -0400 |
commit | 07d28c777757148d13e429c4463da52cbb50d297 (patch) | |
tree | 760314f777b27ecaf7a08c419bfd98a5ed43012a /gdb/linux-tdep.c | |
parent | e3a91079b5e5669567424d3b2f31b48b7b89ef72 (diff) | |
download | gdb-07d28c777757148d13e429c4463da52cbb50d297.zip gdb-07d28c777757148d13e429c4463da52cbb50d297.tar.gz gdb-07d28c777757148d13e429c4463da52cbb50d297.tar.bz2 |
linux_spu_make_corefile_notes: return note_data instead of nullptr
Since commit
9018be2 ("Make target_read_alloc & al return vectors")
the test gdb.threads/gcore-stale-thread.exp test results in UNSUPPORTED:
UNSUPPORTED: gdb.threads/gcore-stale-thread.exp: save a corefile
The problem is that the linux_spu_make_corefile_notes started returning
nullptr when reading TARGET_OBJECT_SPU fails. The previous (and proper)
behaviour is to return the note_data received as a parameter, so that
other functions may continue to append to this buffer.
With this patch, the test goes back to PASS.
gdb/ChangeLog:
* linux-tdep.c (linux_spu_make_corefile_notes): Return note_data
instead of nullptr.
Diffstat (limited to 'gdb/linux-tdep.c')
-rw-r--r-- | gdb/linux-tdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/linux-tdep.c b/gdb/linux-tdep.c index 552a2a4..24bfd74 100644 --- a/gdb/linux-tdep.c +++ b/gdb/linux-tdep.c @@ -1426,7 +1426,7 @@ linux_spu_make_corefile_notes (bfd *obfd, char *note_data, int *note_size) spu_ids = target_read_alloc (¤t_target, TARGET_OBJECT_SPU, NULL); if (!spu_ids) - return nullptr; + return note_data; /* Generate corefile notes for each SPU file. */ for (size_t i = 0; i < spu_ids->size (); i += 4) |