diff options
author | Antonio Quartulli <ordex@autistici.org> | 2011-07-10 00:36:36 +0200 |
---|---|---|
committer | Marek Lindner <lindner_marek@yahoo.de> | 2011-08-22 15:16:19 +0200 |
commit | 1a1f37d9257a4792ca17b28b1c2e4ad15fe95b28 (patch) | |
tree | 5bc0aa0d34b87d1f9b70682f1c54298860ce7d13 /net/batman-adv/originator.c | |
parent | Linux 3.1-rc1 (diff) | |
download | linux-1a1f37d9257a4792ca17b28b1c2e4ad15fe95b28.tar.xz linux-1a1f37d9257a4792ca17b28b1c2e4ad15fe95b28.zip |
batman-adv: hash_add() has to discriminate on the return value
hash_add() returns 0 on success while returns -1 either on error and on
entry already present. The caller could use such information to select
its behaviour. For this reason it is useful that hash_add() returns -1
in case on error and returns 1 in case of entry already present.
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Diffstat (limited to 'net/batman-adv/originator.c')
-rw-r--r-- | net/batman-adv/originator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/batman-adv/originator.c b/net/batman-adv/originator.c index f3c3f620d195..d448018e514f 100644 --- a/net/batman-adv/originator.c +++ b/net/batman-adv/originator.c @@ -252,7 +252,7 @@ struct orig_node *get_orig_node(struct bat_priv *bat_priv, const uint8_t *addr) hash_added = hash_add(bat_priv->orig_hash, compare_orig, choose_orig, orig_node, &orig_node->hash_entry); - if (hash_added < 0) + if (hash_added != 0) goto free_bcast_own_sum; return orig_node; |