gdbserver/proc-service.c: Change CORE_ADDR cast to uintptr_t

Fixes on i386:

../../../binutils-gdb/gdb/gdbserver/proc-service.c: In function ps_pdread:
../../../binutils-gdb/gdb/gdbserver/proc-service.c:83:25: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
../../../binutils-gdb/gdb/gdbserver/proc-service.c: In function ps_pdwrite:
../../../binutils-gdb/gdb/gdbserver/proc-service.c:93:30: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]

I could have kept both casts:

  (CORE_ADDR) (uintptr_t) addr

but it's cleaner this way.  The uintptr_t implicitely gets promoted to a
CORE_ADDR, which is at least as long as uintptr_t.

gdb/gdbserver/ChangeLog:

	* proc-service.c (ps_pdread): Change CORE_ADDR cast to uintptr_t.
	(ps_pdwrite): Likewise.
This commit is contained in:
Simon Marchi 2015-10-30 11:50:00 -04:00
parent 26f187cd16
commit 7ea45d72f9
2 changed files with 7 additions and 2 deletions

View File

@ -1,3 +1,8 @@
2015-10-30 Simon Marchi <simon.marchi@ericsson.com>
* proc-service.c (ps_pdread): Change CORE_ADDR cast to uintptr_t.
(ps_pdwrite): Likewise.
2015-10-29 Henrik Wallin <henrik.wallin@windriver.com>
* linux-arm-low.c (arm_new_thread): Move pointer dereference

View File

@ -80,7 +80,7 @@ ps_err_e
ps_pdread (gdb_ps_prochandle_t ph, psaddr_t addr,
gdb_ps_read_buf_t buf, gdb_ps_size_t size)
{
read_inferior_memory ((CORE_ADDR) addr, (gdb_byte *) buf, size);
read_inferior_memory ((uintptr_t) addr, (gdb_byte *) buf, size);
return PS_OK;
}
@ -90,7 +90,7 @@ ps_err_e
ps_pdwrite (gdb_ps_prochandle_t ph, psaddr_t addr,
gdb_ps_write_buf_t buf, gdb_ps_size_t size)
{
if (write_inferior_memory ((CORE_ADDR) addr, (const gdb_byte *) buf, size)
if (write_inferior_memory ((uintptr_t) addr, (const gdb_byte *) buf, size)
!= 0)
return PS_ERR;
return PS_OK;