diff options
| author |   Sven Eckelmann <sven@narfation.org> | 2016-07-27 12:31:07 +0200 |
|---|---|---|
| committer |   Simon Wunderlich <sw@simonwunderlich.de> | 2016-10-19 08:37:52 +0200 |
| commit | 176e5b772b113f9ab013d574e13d8a433d47bbcb (patch) | |
| tree | cdac0bb46e98ab5c2529499d2170813ea6521504 /net/batman-adv/fragmentation.h | |
| parent | batman-adv: Remove needless init of variables on stack (diff) | |
| download | linux-dev-176e5b772b113f9ab013d574e13d8a433d47bbcb.tar.xz linux-dev-176e5b772b113f9ab013d574e13d8a433d47bbcb.zip | |
batman-adv: Use proper name for fragments list head
The batman-adv codebase is using "list" for the list node (prev/next) and
<list content descriptor>+"_list" for the head of a list. Not using this
naming scheme can up in confusions because list_head is used for both the
head of the list and the list node (prev/next) in each item of the list.
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
Diffstat (limited to 'net/batman-adv/fragmentation.h')
| -rw-r--r-- | net/batman-adv/fragmentation.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/batman-adv/fragmentation.h b/net/batman-adv/fragmentation.h index 3202fe329e63..b95f619606af 100644 --- a/net/batman-adv/fragmentation.h +++ b/net/batman-adv/fragmentation.h @@ -47,7 +47,7 @@ int batadv_frag_send_packet(struct sk_buff *skb, static inline bool batadv_frag_check_entry(struct batadv_frag_table_entry *frags_entry) { - if (!hlist_empty(&frags_entry->head) && + if (!hlist_empty(&frags_entry->fragment_list) && batadv_has_timed_out(frags_entry->timestamp, BATADV_FRAG_TIMEOUT)) return true; return false; |