linux: netlink_process_link: simplify NULL comparisons and add some braces
authorFerry Huberts <ferry.huberts@pelagic.nl>
Wed, 10 Feb 2016 09:38:46 +0000 (10:38 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Wed, 10 Feb 2016 13:32:41 +0000 (14:32 +0100)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/linux/kernel_routes_nl.c

index eb33664..9687b3c 100644 (file)
@@ -121,7 +121,7 @@ static void netlink_process_link(struct nlmsghdr *h)
   iface = if_ifwithindex(ifi->ifi_index);
   oif = NULL;
 
-  if (iface == NULL && (ifi->ifi_flags & IFF_UP) == IFF_UP) {
+  if (!iface && ((ifi->ifi_flags & IFF_UP) == IFF_UP)) {
     if (if_indextoname(ifi->ifi_index, namebuffer)) {
       if ((oif = olsrif_ifwithname(namebuffer)) != NULL) {
         /* try to take interface up, will trigger ifchange */
@@ -129,12 +129,12 @@ static void netlink_process_link(struct nlmsghdr *h)
       }
     }
   }
-  else if (iface != NULL && (ifi->ifi_flags & IFF_UP) == 0) {
+  else if (iface && ((ifi->ifi_flags & IFF_UP) == 0)) {
     /* try to take interface down, will trigger ifchange */
     olsr_remove_interface(iface->olsr_if);
   }
 
-  if (iface == NULL && oif == NULL) {
+  if (!iface && !oif) {
     /* this is not an OLSR interface */
     if ((ifi->ifi_flags & IFF_UP) != 0) {
       olsr_trigger_ifchange(ifi->ifi_index, NULL, IFCHG_IF_ADD);