diff options
| author | Antonio Quartulli <ordex@autistici.org> | 2011-10-05 17:05:25 +0200 |
|---|---|---|
| committer | Sven Eckelmann <sven@narfation.org> | 2011-11-20 13:08:33 +0100 |
| commit | c90681b8505946761b55d4981c9c3b56b3c4171b (patch) | |
| tree | b260388317b5dc1f4a8c00493eea6ff30fb47652 /net/batman-adv/routing.c | |
| parent | eb7e2a1e20488f91c7007caa080b83b8e4222572 (diff) | |
batman-adv: fixed hash functions type to uint32_t instead of int
There are two reasons for this fix:
- the result of choose_orig() and vis_choose() is an index and therefore it can't
be negative. Hence it is correct to make the return type unsigned too.
- sizeof(int) may not be the same on ALL the architectures. Since we plan to use
choose_orig() as DHT hash function, we need to guarantee that, given the same
argument, the result is the same. Then it is correct to explicitly express
the size of the return type (and the second argument). Since the expected
length is currently 4, uint32_t is the most convenient choice.
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Diffstat (limited to 'net/batman-adv/routing.c')
| -rw-r--r-- | net/batman-adv/routing.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/batman-adv/routing.c b/net/batman-adv/routing.c index f961cc5eade..60ce4077348 100644 --- a/net/batman-adv/routing.c +++ b/net/batman-adv/routing.c @@ -39,7 +39,7 @@ void slide_own_bcast_window(struct hard_iface *hard_iface) struct hlist_head *head; struct orig_node *orig_node; unsigned long *word; - int i; + uint32_t i; size_t word_index; for (i = 0; i < hash->size; i++) { |