diff options
| author | Sven Eckelmann <sven@narfation.org> | 2011-05-14 23:14:49 +0200 |
|---|---|---|
| committer | Sven Eckelmann <sven@narfation.org> | 2011-05-30 07:39:30 +0200 |
| commit | 38e3c5f0dae7a3bbb32c3b2bb28c3f2557d40fe9 (patch) | |
| tree | f0a64eaf3709c5ca1455a1c66ce0a9c22c5c4961 /net/batman-adv | |
| parent | 16f14b45c48f4f1be5857103ee3054114604d98c (diff) | |
| download | op-kernel-dev-38e3c5f0dae7a3bbb32c3b2bb28c3f2557d40fe9.zip op-kernel-dev-38e3c5f0dae7a3bbb32c3b2bb28c3f2557d40fe9.tar.gz | |
batman-adv: Don't do pointer arithmetic with void*
The size of void is currently set by gcc to 1, but is not well defined
in general. Therefore it is more advisable to cast it to char* before
doing pointer arithmetic.
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Diffstat (limited to 'net/batman-adv')
| -rw-r--r-- | net/batman-adv/originator.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/batman-adv/originator.c b/net/batman-adv/originator.c index 40a30bb..ae54f77 100644 --- a/net/batman-adv/originator.c +++ b/net/batman-adv/originator.c @@ -559,7 +559,7 @@ static int orig_node_del_if(struct orig_node *orig_node, memcpy(data_ptr, orig_node->bcast_own, del_if_num * chunk_size); /* copy second part */ - memcpy(data_ptr + del_if_num * chunk_size, + memcpy((char *)data_ptr + del_if_num * chunk_size, orig_node->bcast_own + ((del_if_num + 1) * chunk_size), (max_if_num - del_if_num) * chunk_size); @@ -579,7 +579,7 @@ free_bcast_own: memcpy(data_ptr, orig_node->bcast_own_sum, del_if_num * sizeof(uint8_t)); - memcpy(data_ptr + del_if_num * sizeof(uint8_t), + memcpy((char *)data_ptr + del_if_num * sizeof(uint8_t), orig_node->bcast_own_sum + ((del_if_num + 1) * sizeof(uint8_t)), (max_if_num - del_if_num) * sizeof(uint8_t)); |
