[PATCH] Memory reads and writes should have size_t length

Siddhesh Poyarekar siddhesh@redhat.com
Thu May 31 08:08:00 GMT 2012


Hi,

Following the patch to bfd to accept size_t when reading objects from
memory:

http://sourceware.org/ml/gdb-patches/2012-05/msg00988.html

here's a patch makes memory reading and writing code consistent with
the above change and hence accept size_t for length. This change is not
comprehensive, i.e. there are a number of other places (to_xfer_partial
and friends) where a similar change should occur, but they're not
included in this change since those places already use LONGEST for
length and hence won't really be *fixing* anything. These changes on
the other hand, are supporting changes for the bitpos and type.length
changes:

http://sourceware.org/ml/gdb-patches/2012-05/msg00128.html

Regards,
Siddhesh

gdb/ChangeLog:

2012-05-31  Siddhesh Poyarekar  <siddhesh@redhat.com>

	* corefile.c (read_memory): Accept LEN argument as size_t.
	(read_stack): Likewise.
	(write_memory): Likewise.
	* gdbcore.h (read_memory): Likewise.
	(read_stack): Likewise.
	(write_memory): Likewise.
	* remote.c (remote_write_bytes_aux): Likewise.
	(remote_write_bytes): Likewise.
	* target.c (target_read_stack): Likewise.
	(target_write_memory): Likewise.
	(target_write_raw_memory): Likewise.
	* target.h (target_read_stack): Likewise.
	(target_write_memory): Likewise.
	(target_write_raw_memory): Likewise.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdb-memory-sizet.patch
Type: text/x-patch
Size: 5185 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20120531/7a9a88e5/attachment.bin>


More information about the Gdb-patches mailing list