diff options
| author | Tom Tromey <tom@tromey.com> | 2017-10-12 18:20:09 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2017-10-16 16:10:21 -0600 |
| commit | 87028b873938aa3cc756c99e33a328640dbc30bc (patch) | |
| tree | dfcd9d27e1c072452798930bc3b6f1b658f75e2c /gdb/target.c | |
| parent | b7b030adc405017f01e996a90f85e40730ef8397 (diff) | |
| download | gdb-87028b873938aa3cc756c99e33a328640dbc30bc.zip gdb-87028b873938aa3cc756c99e33a328640dbc30bc.tar.gz gdb-87028b873938aa3cc756c99e33a328640dbc30bc.tar.bz2 | |
Return unique_xmalloc_ptr from target_fileio_read_stralloc
Change target_fileio_read_stralloc to return unique_xmalloc_ptr and
fix up the callers. This removes a number of cleanups.
ChangeLog
2017-10-16 Tom Tromey <tom@tromey.com>
* linux-tdep.c (linux_info_proc, linux_find_memory_regions_full)
(linux_fill_prpsinfo, linux_vsyscall_range_raw): Update.
* target.c (target_fileio_read_stralloc): Update.
* sparc64-tdep.c (adi_is_addr_mapped): Update.
* target.h (target_fileio_read_stralloc): Return
unique_xmalloc_ptr.
Diffstat (limited to 'gdb/target.c')
| -rw-r--r-- | gdb/target.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gdb/target.c b/gdb/target.c index 94307cb..6fa9d59 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -3083,7 +3083,7 @@ target_fileio_read_alloc (struct inferior *inf, const char *filename, /* See target.h. */ -char * +gdb::unique_xmalloc_ptr<char> target_fileio_read_stralloc (struct inferior *inf, const char *filename) { gdb_byte *buffer; @@ -3094,10 +3094,10 @@ target_fileio_read_stralloc (struct inferior *inf, const char *filename) bufstr = (char *) buffer; if (transferred < 0) - return NULL; + return gdb::unique_xmalloc_ptr<char> (nullptr); if (transferred == 0) - return xstrdup (""); + return gdb::unique_xmalloc_ptr<char> (xstrdup ("")); bufstr[transferred] = 0; @@ -3111,7 +3111,7 @@ target_fileio_read_stralloc (struct inferior *inf, const char *filename) break; } - return bufstr; + return gdb::unique_xmalloc_ptr<char> (bufstr); } |