batman-adv: fixed hash functions type to uint32_t instead of int
authorAntonio Quartulli <ordex@autistici.org>
Wed, 5 Oct 2011 15:05:25 +0000 (17:05 +0200)
committerSven Eckelmann <sven@narfation.org>
Sun, 20 Nov 2011 12:08:33 +0000 (13:08 +0100)
commitc90681b8505946761b55d4981c9c3b56b3c4171b
treeb260388317b5dc1f4a8c00493eea6ff30fb47652
parenteb7e2a1e20488f91c7007caa080b83b8e4222572
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>
net/batman-adv/hash.c
net/batman-adv/hash.h
net/batman-adv/originator.c
net/batman-adv/originator.h
net/batman-adv/routing.c
net/batman-adv/translation-table.c
net/batman-adv/vis.c