]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit - bfd/mmo.c
add casts to avoid arithmetic on void *
authorTrevor Saunders <tbsaunde+binutils@tbsaunde.org>
Wed, 13 Apr 2016 09:17:31 +0000 (05:17 -0400)
committerTrevor Saunders <tbsaunde+binutils@tbsaunde.org>
Wed, 27 Apr 2016 00:09:57 +0000 (20:09 -0400)
commit28cc9170c3d0bc0c5e82b45a10015e4bbcb01125
tree39cc9b5d7af7877b58da8d4a2e8cf708a7cf64fd
parentffbad59e36154c69cd95d2ed59223e9449206314
add casts to avoid arithmetic on void *

arithmetic on void * is undefined in ISO C, so we should avoid it.  In
GNU C sizeof void * is defined as 1, and that is pretty clearly what
this code wants, so change it to do arithmetic on bfd_byte *.
Unfortunately most of the argument types come from virtual function
interfaces so changing the types to bfd_byte * isn't trivial though it
might make the code clearer.  So for the moment its easiest to leave the
variable types as void * and cast before doing arithmetic.

bfd/ChangeLog:

2016-04-26  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

* elf32-rx.c (rx_set_section_contents): Avoid arithmetic on void *.
* mmo.c (mmo_get_section_contents): Likewise.
(mmo_set_section_contents): Likewise.
bfd/ChangeLog
bfd/elf32-rx.c
bfd/mmo.c