bridge: return boolean instead of integer in br_multicast_is_router
authorGustavo A. R. Silva <garsilva@embeddedor.com>
Thu, 18 Jan 2018 23:37:45 +0000 (17:37 -0600)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 Jan 2018 21:13:20 +0000 (16:13 -0500)
Return statements in functions returning bool should use
true/false instead of 1/0.

This issue was detected with the help of Coccinelle.

Fixes: 85b352693264 ("bridge: Fix build error when IGMP_SNOOPING is not enabled")
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Reviewed-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_private.h

index 80559fd11b7e49816b1d8300c6ae63553af6cc10..8e13a64d8c99e382f094480dda40c8eb8d43df8d 100644 (file)
@@ -760,7 +760,7 @@ static inline void br_multicast_flood(struct net_bridge_mdb_entry *mdst,
 
 static inline bool br_multicast_is_router(struct net_bridge *br)
 {
-       return 0;
+       return false;
 }
 
 static inline bool br_multicast_querier_exists(struct net_bridge *br,