"open_ipc" is only a boolean value containing "olsr_cnf->debug_level > 1".
authorBernd Petrovitsch <bernd@firmix.at>
Thu, 29 Nov 2007 22:21:26 +0000 (22:21 +0000)
committerBernd Petrovitsch <bernd@firmix.at>
Thu, 29 Nov 2007 22:21:26 +0000 (22:21 +0000)
So I see no reason to keep that variable.

Apart from the the attached patch makes the "#if" condition for the
"rts" field identical to all other references to it.

src/cfgparser/olsrd_conf.c
src/cfgparser/oparse.y
src/ipc_frontend.c
src/linux/kernel_routes.c
src/main.c
src/olsr_cfg.h
src/win32/kernel_routes.c

index 6711a93..0e2ac31 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_conf.c,v 1.63 2007/11/29 00:49:40 bernd67 Exp $
+ * $Id: olsrd_conf.c,v 1.64 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 
@@ -436,7 +436,6 @@ set_default_cnf(struct olsrd_config *cnf)
     cnf->rttable = 254;
     cnf->willingness_auto = DEF_WILL_AUTO;
     cnf->ipc_connections = DEF_IPC_CONNECTIONS;
-    cnf->open_ipc = cnf->ipc_connections ? OLSR_TRUE : OLSR_FALSE;
 
     cnf->use_hysteresis = DEF_USE_HYST;
     cnf->hysteresis_param.scaling = HYST_SCALING;
index 92096b6..4b3db26 100644 (file)
@@ -38,7 +38,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: oparse.y,v 1.40 2007/11/29 00:49:40 bernd67 Exp $
+ * $Id: oparse.y,v 1.41 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 
@@ -323,7 +323,6 @@ plstmt:     plparam
 imaxipc: TOK_MAXIPC TOK_INTEGER
 {
   olsr_cnf->ipc_connections = $2->integer;
-  olsr_cnf->open_ipc = olsr_cnf->ipc_connections ? OLSR_TRUE : OLSR_FALSE;
   free($2);
 }
 ;
index 2a9873b..1bf33bf 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: ipc_frontend.c,v 1.42 2007/11/29 18:10:12 bernd67 Exp $
+ * $Id: ipc_frontend.c,v 1.43 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 /*
@@ -270,7 +270,7 @@ ipc_route_send_rtentry(const union olsr_ip_addr *dst,
   struct ipcmsg packet;
   char *tmp;
 
-  if (!olsr_cnf->open_ipc) {
+  if (olsr_cnf->ipc_connections <= 0) {
     return -1;
   }
 
index 763efa2..387423c 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: kernel_routes.c,v 1.32 2007/11/29 18:10:17 bernd67 Exp $
+ * $Id: kernel_routes.c,v 1.33 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 #include "kernel_routes.h"
@@ -151,7 +151,7 @@ static int olsr_netlink_route(const struct rt_entry *rt, olsr_u8_t family, olsr_
                                h = NLMSG_NEXT(h, ret);
                        }
                }
-               if (0 <= ret && olsr_cnf->open_ipc)
+               if (0 <= ret && olsr_cnf->ipc_connections > 0)
                {
                        ipc_route_send_rtentry(
                                &rt->rt_dst.prefix,
index b634d75..f935f72 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: main.c,v 1.106 2007/11/29 00:49:38 bernd67 Exp $
+ * $Id: main.c,v 1.107 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 #include <unistd.h>
@@ -265,9 +265,9 @@ main(int argc, char *argv[])
   /*
    * Print configuration 
    */
-  if(olsr_cnf->debug_level > 1)
+  if(olsr_cnf->debug_level > 1) {
     olsrd_print_cnf(olsr_cnf);
-
+  }
 #ifndef WIN32
   /* Disable redirects globally */
   disable_redirects_global(olsr_cnf->ip_version);
@@ -276,28 +276,27 @@ main(int argc, char *argv[])
   /*
    *socket for icotl calls
    */
-  if ((olsr_cnf->ioctl_s = socket(olsr_cnf->ip_version, SOCK_DGRAM, 0)) < 0) 
-
-    {
-      olsr_syslog(OLSR_LOG_ERR, "ioctl socket: %m");
-      olsr_exit(__func__, 0);
-    }
+  olsr_cnf->ioctl_s = socket(olsr_cnf->ip_version, SOCK_DGRAM, 0);
+  if (olsr_cnf->ioctl_s < 0) {
+    olsr_syslog(OLSR_LOG_ERR, "ioctl socket: %m");
+    olsr_exit(__func__, 0);
+  }
 
 #if LINUX_POLICY_ROUTING
-  if ((olsr_cnf->rtnl_s = socket(PF_NETLINK, SOCK_DGRAM, NETLINK_ROUTE)) < 0) 
-    {
-      olsr_syslog(OLSR_LOG_ERR, "rtnetlink socket: %m");
-      olsr_exit(__func__, 0);
-    }
+  olsr_cnf->rtnl_s = socket(PF_NETLINK, SOCK_DGRAM, NETLINK_ROUTE);
+  if (olsr_cnf->rtnl_s < 0) {
+    olsr_syslog(OLSR_LOG_ERR, "rtnetlink socket: %m");
+    olsr_exit(__func__, 0);
+  }
   fcntl(olsr_cnf->rtnl_s, F_SETFL, O_NONBLOCK);
 #endif
 
 #if defined __FreeBSD__ || defined __MacOSX__ || defined __NetBSD__ || defined __OpenBSD__
-  if ((olsr_cnf->rts = socket(PF_ROUTE, SOCK_RAW, 0)) < 0)
-    {
-      olsr_syslog(OLSR_LOG_ERR, "routing socket: %m");
-      olsr_exit(__func__, 0);
-    }
+  olsr_cnf->rts = socket(PF_ROUTE, SOCK_RAW, 0);
+  if (olsr_cnf->rts < 0) {
+    olsr_syslog(OLSR_LOG_ERR, "routing socket: %m");
+    olsr_exit(__func__, 0);
+  }
 #endif
 
   /* Init empty TC timer */
@@ -372,9 +371,9 @@ main(int argc, char *argv[])
 
   /* Initialize the IPC socket */
 
-  if(olsr_cnf->open_ipc)
+  if (olsr_cnf->ipc_connections > 0) {
       ipc_init();
-
+  }
   /* Initialisation of different tables to be used.*/
   olsr_init_tables();
 
@@ -485,8 +484,9 @@ olsr_shutdown(int signal __attribute__((unused)))
   OLSR_PRINTF(1, "Closing sockets...\n");
 
   /* front-end IPC socket */
-  if(olsr_cnf->open_ipc)
+  if (olsr_cnf->ipc_connections > 0) {
     shutdown_ipc();
+  }
 
   /* OLSR sockets */
   for (ifn = ifnet; ifn; ifn = ifn->int_next) 
@@ -796,7 +796,6 @@ olsr_process_arguments(int argc, char *argv[],
       if (strcmp(*argv, "-ipc") == 0) 
        {
          cnf->ipc_connections = 1;
-         cnf->open_ipc = OLSR_TRUE;
          continue;
        }
 
index b6da995..18ef500 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsr_cfg.h,v 1.41 2007/11/29 15:47:26 bernd67 Exp $
+ * $Id: olsr_cfg.h,v 1.42 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 
@@ -192,7 +192,6 @@ struct olsrd_config
   olsr_u8_t                willingness;
   olsr_bool                willingness_auto;
   int                      ipc_connections;
-  olsr_bool                open_ipc;
   olsr_bool                use_hysteresis;
   struct hyst_param        hysteresis_param;
   struct plugin_entry      *plugins;
@@ -224,7 +223,9 @@ struct olsrd_config
   int                      ioctl_s;              /* Socket used for ioctl calls */
 #if LINUX_POLICY_ROUTING
   int                      rtnl_s;               /* Socket used for rtnetlink messages */
-#else
+#endif
+
+#if defined __FreeBSD__ || defined __MacOSX__ || defined __NetBSD__ || defined __OpenBSD__
   int                      rts;                  /* Socket used for route changes on BSDs */
 #endif
 };
index e60307f..69f0e47 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: kernel_routes.c,v 1.25 2007/11/11 23:10:22 bernd67 Exp $
+ * $Id: kernel_routes.c,v 1.26 2007/11/29 22:21:26 bernd67 Exp $
  */
 
 #include <stdio.h>
@@ -116,7 +116,7 @@ int olsr_ioctl_add_route(const struct rt_entry *rt)
   /*
    * Send IPC route update message
    */
-  if(olsr_cnf->open_ipc) {
+  if (olsr_cnf->ipc_connections > 0) {
     ipc_route_send_rtentry(&rt->rt_dst.prefix, &rt->rt_best->rtp_nexthop.gateway,
         rt->rt_best->rtp_metric.hops, 1,
         if_ifwithindex_name(rt->rt_best->rtp_nexthop.iif_index));
@@ -185,7 +185,7 @@ int olsr_ioctl_del_route(const struct rt_entry *rt)
   /*
    * Send IPC route update message
    */
-  if(olsr_cnf->open_ipc) {
+  if (olsr_cnf->ipc_connections > 0) {
     ipc_route_send_rtentry(&rt->rt_dst.prefix, NULL, 0 , 0, NULL);
   }