batman-adv: Don't do pointer arithmetic with void*
authorSven Eckelmann <sven@narfation.org>
Sat, 14 May 2011 21:14:49 +0000 (23:14 +0200)
committerSven Eckelmann <sven@narfation.org>
Mon, 30 May 2011 05:39:30 +0000 (07:39 +0200)
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>
net/batman-adv/originator.c

index 40a30bbcd14789649c4dee0228ece7c63e34eb9e..ae54f77f2c1bced9902487e498fd817b91c41cee 100644 (file)
@@ -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));