batman-adv: main, Convert is_my_mac() to bool
It is much clearer to see a bool type as return value than 'int' for functions that are supposed to return true or false. Signed-off-by: Markus Pargmann <mpa@pengutronix.de> Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
This commit is contained in:
parent
a0c77227ff
commit
e8ad3b1acf
@ -209,10 +209,13 @@ void batadv_mesh_free(struct net_device *soft_iface)
|
|||||||
* interfaces in the current mesh
|
* interfaces in the current mesh
|
||||||
* @bat_priv: the bat priv with all the soft interface information
|
* @bat_priv: the bat priv with all the soft interface information
|
||||||
* @addr: the address to check
|
* @addr: the address to check
|
||||||
|
*
|
||||||
|
* Returns 'true' if the mac address was found, false otherwise.
|
||||||
*/
|
*/
|
||||||
int batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr)
|
bool batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr)
|
||||||
{
|
{
|
||||||
const struct batadv_hard_iface *hard_iface;
|
const struct batadv_hard_iface *hard_iface;
|
||||||
|
bool is_my_mac = false;
|
||||||
|
|
||||||
rcu_read_lock();
|
rcu_read_lock();
|
||||||
list_for_each_entry_rcu(hard_iface, &batadv_hardif_list, list) {
|
list_for_each_entry_rcu(hard_iface, &batadv_hardif_list, list) {
|
||||||
@ -223,12 +226,12 @@ int batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr)
|
|||||||
continue;
|
continue;
|
||||||
|
|
||||||
if (batadv_compare_eth(hard_iface->net_dev->dev_addr, addr)) {
|
if (batadv_compare_eth(hard_iface->net_dev->dev_addr, addr)) {
|
||||||
rcu_read_unlock();
|
is_my_mac = true;
|
||||||
return 1;
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
rcu_read_unlock();
|
rcu_read_unlock();
|
||||||
return 0;
|
return is_my_mac;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -195,7 +195,7 @@ extern struct workqueue_struct *batadv_event_workqueue;
|
|||||||
|
|
||||||
int batadv_mesh_init(struct net_device *soft_iface);
|
int batadv_mesh_init(struct net_device *soft_iface);
|
||||||
void batadv_mesh_free(struct net_device *soft_iface);
|
void batadv_mesh_free(struct net_device *soft_iface);
|
||||||
int batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr);
|
bool batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr);
|
||||||
struct batadv_hard_iface *
|
struct batadv_hard_iface *
|
||||||
batadv_seq_print_text_primary_if_get(struct seq_file *seq);
|
batadv_seq_print_text_primary_if_get(struct seq_file *seq);
|
||||||
int batadv_max_header_len(void);
|
int batadv_max_header_len(void);
|
||||||
|
Loading…
Reference in New Issue
Block a user