cleanup, 1. step (to get some local changes away):
authorBernd Petrovitsch <bernd@firmix.at>
Fri, 20 Apr 2007 13:46:05 +0000 (13:46 +0000)
committerBernd Petrovitsch <bernd@firmix.at>
Fri, 20 Apr 2007 13:46:05 +0000 (13:46 +0000)
* killed lots of warnings
* added __attribure__((unused)) and __attribure__((format(printf, ..., ...)))
* added "const"
* moved declarations of static functions into .c files
* added "static" declarations
* removed all "inline" declarations. Usually the compiler knows better anyways.
* removed emtpy "return" statements at the end of functions
* localized declarations
* avoided unnecessary data copies
* removed lots of empty lines
There is much more to do.

70 files changed:
lib/mini/src/olsrd_plugin.c
lib/nameservice/src/nameservice.c
lib/nameservice/src/olsrd_plugin.c
lib/pgraph/src/olsrd_pgraph.c
lib/pgraph/src/olsrd_pgraph.h
lib/pgraph/src/olsrd_plugin.c
lib/secure/src/olsrd_plugin.c
lib/secure/src/olsrd_secure.c
lib/tas/src/glua.c
lib/tas/src/lib.h
lib/tas/src/lua/lapi.c
lib/tas/src/lua/lauxlib.c
lib/tas/src/lua/lauxlib.h
lib/tas/src/lua/lbaselib.c
lib/tas/src/lua/ldebug.c
lib/tas/src/lua/ldebug.h
lib/tas/src/lua/ldo.h
lib/tas/src/lua/liolib.c
lib/tas/src/lua/llex.c
lib/tas/src/lua/llex.h
lib/tas/src/lua/lparser.c
lib/tas/src/lua/lua.h
lib/tas/src/lua/lundump.c
lib/tas/src/plugin.c
lib/txtinfo/src/olsrd_plugin.c
lib/txtinfo/src/olsrd_txtinfo.c
src/bsd/net.h
src/build_msg.c
src/cfgparser/Makefile
src/cfgparser/olsrd_conf.c
src/cfgparser/oparse.y
src/cfgparser/oscan.lex
src/defs.h
src/duplicate_set.c
src/generate_msg.c
src/hashing.c
src/hna_set.c
src/hysteresis.h
src/interfaces.c
src/interfaces.h
src/ipc_frontend.c
src/ipc_frontend.h
src/link_set.c
src/linux/apm.c
src/linux/kernel_routes.c
src/linux/net.c
src/local_hna_set.c
src/log.h
src/main.c
src/mid_set.c
src/mpr.c
src/mpr_selector_set.c
src/neighbor_table.c
src/net_olsr.c
src/net_olsr.h
src/olsr.c
src/olsr.h
src/olsr_types.h
src/olsrd_plugin.h
src/parser.c
src/plugin_loader.c
src/process_package.c
src/process_routes.c
src/routing_table.c
src/scheduler.c
src/scheduler.h
src/socket_parser.c
src/tc_set.c
src/two_hop_neighbor_table.c
src/unix/ifnet.c

index 914ee51..591362f 100644 (file)
@@ -29,7 +29,7 @@
  *
  */
 
-/* $Id: olsrd_plugin.c,v 1.2 2005/05/29 12:47:41 br1 Exp $ */
+/* $Id: olsrd_plugin.c,v 1.3 2007/04/20 13:46:02 bernd67 Exp $ */
 
  /*
  * Example plugin for olsrd.org OLSR daemon
@@ -43,6 +43,8 @@
 #include "olsrd_plugin.h"
 #include "olsr.h"
 
+#define PLUGIN_INTERFACE_VERSION 1
+
 
 /****************************************************************************
  *                Functions that the plugin MUST provide                    *
@@ -53,9 +55,9 @@
  * Used by main olsrd to check plugin interface version
  */
 int 
-olsrd_plugin_interface_version()
+olsrd_plugin_interface_version(void)
 {
-       return OLSRD_PLUGIN_INTERFACE_VERSION;
+       return PLUGIN_INTERFACE_VERSION;
 }
 
 
@@ -79,7 +81,7 @@ olsrd_plugin_register_param(char *key, char *value)
  * Called after all parameters are passed
  */
 int
-olsrd_plugin_init()
+olsrd_plugin_init(void)
 {
        printf("*** MINI: plugin_init\n");
        
@@ -107,7 +109,7 @@ my_fini(void);
  * Optional Private Constructor
  */
 static void
-my_init()
+my_init(void)
 {
        printf("*** MINI: constructor\n");
 }
@@ -117,7 +119,7 @@ my_init()
  * Optional Private Destructor
  */
 static void
-my_fini()
+my_fini(void)
 {
        printf("*** MINI: destructor\n");
 }
index ee0af41..06764e0 100644 (file)
@@ -30,7 +30,7 @@
  *
  */
 
-/* $Id: nameservice.c,v 1.20 2007/04/20 10:33:08 bernd67 Exp $ */
+/* $Id: nameservice.c,v 1.21 2007/04/20 13:46:02 bernd67 Exp $ */
 
 /*
  * Dynamic linked library for UniK OLSRd
@@ -104,7 +104,7 @@ static struct rt_entry *host_lookup_routing_table(union olsr_ip_addr *);
  * do initialization
  */
 void
-name_constructor() 
+name_constructor(void
 {
        int i;
        
@@ -257,7 +257,7 @@ add_name_to_list(struct name_entry *my_list, char *value, int type, const union
  *   - register_olsr_data() then then finally calls this function
  */
 int
-name_init()
+name_init(void)
 {
        struct name_entry *name;
        union olsr_ip_addr ipz;
@@ -390,7 +390,7 @@ remove_nonvalid_names_from_list(struct name_entry *my_list, int type)
  * XXX: should I delete the hosts/services/resolv.conf files on exit?
  */
 void
-name_destructor()
+name_destructor(void)
 {
        olsr_printf(2, "NAME PLUGIN: exit. cleaning up...\n");
        
@@ -412,15 +412,13 @@ void
 free_all_list_entries(struct db_entry **this_db_list) 
 {
        int i;
-       struct db_entry **tmp;
-       struct db_entry *to_delete;
        
        for(i = 0; i < HASHSIZE; i++)
        {
-               tmp = &list[i];
+        struct db_entry **tmp = &this_db_list[i];
                while(*tmp != NULL)
                {
-                       to_delete = *tmp;
+            struct db_entry *to_delete = *tmp;
                        *tmp = (*tmp)->next;
                        free_name_entry_list(&to_delete->names);
                        free(to_delete);
@@ -440,7 +438,7 @@ free_all_list_entries(struct db_entry **this_db_list)
  * and write changes to file
  */
 void
-olsr_timeout()
+olsr_timeout(void)
 {
     timeout_old_names(list, &name_table_changed);
     timeout_old_names(forwarder_list, &forwarder_table_changed);
@@ -488,7 +486,7 @@ timeout_old_names(struct db_entry **this_list, olsr_bool *this_table_changed)
  * Scheduled event: generate and send NAME packet
  */
 void
-olsr_event(void *foo)
+olsr_event(void *foo __attribute__((unused)))
 {
        union olsr_message *message = (union olsr_message*)buffer;
        struct interface *ifn;
@@ -833,7 +831,7 @@ insert_new_name_in_list(union olsr_ip_addr *originator, struct db_entry **this_l
  * write names to a file in /etc/hosts compatible format
  */
 void
-write_hosts_file()
+write_hosts_file(void)
 {
        int hash;
        struct name_entry *name;
@@ -914,7 +912,7 @@ write_hosts_file()
  * http://me.olsr:80|tcp|my little homepage
  */
 void
-write_services_file()
+write_services_file(void)
 {
        int hash;
        struct name_entry *name;
@@ -972,7 +970,7 @@ write_services_file()
  * best means the 3 with the best etx value in routing table
  */
 void
-write_resolv_file()
+write_resolv_file(void)
 {
        int hash;
        struct name_entry *name, *tmp_dns, *last_dns, *dnslist = NULL;
index 0c48eed..dedf4bc 100644 (file)
@@ -29,7 +29,7 @@
  *
  */
 
-/* $Id: olsrd_plugin.c,v 1.11 2005/05/29 12:47:42 br1 Exp $ */
+/* $Id: olsrd_plugin.c,v 1.12 2007/04/20 13:46:02 bernd67 Exp $ */
 
 /*
  * Dynamic linked library for olsr.org olsrd
@@ -41,7 +41,6 @@
 #include "olsrd_plugin.h"
 #include "nameservice.h"
 
-
 #define PLUGIN_NAME    "OLSRD nameservice plugin"
 #define PLUGIN_VERSION "0.2"
 #define PLUGIN_AUTHOR   "Bruno Randolf"
@@ -54,22 +53,21 @@ my_init(void);
 static void __attribute__ ((destructor)) 
 my_fini(void);
 
-
 int 
-olsrd_plugin_interface_version()
+olsrd_plugin_interface_version(void)
 {
        return OLSRD_PLUGIN_INTERFACE_VERSION;
 }
 
-
-int olsrd_plugin_init()
+int
+olsrd_plugin_init(void)
 {
        return name_init();
 }
 
 
 static void
-my_init()
+my_init(void)
 {
        /* Print plugin info to stdout */
        printf("%s\n", MOD_DESC);
@@ -81,7 +79,7 @@ my_init()
 
 
 static void
-my_fini()
+my_fini(void)
 {
        name_destructor();
 }
index fc87712..5db8aef 100755 (executable)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_pgraph.c,v 1.3 2006/01/07 08:17:43 kattemat Exp $
+ * $Id: olsrd_pgraph.c,v 1.4 2007/04/20 13:46:02 bernd67 Exp $
  */
 
 /*
@@ -61,23 +61,37 @@ int ipc_open;
 int ipc_connection;
 int ipc_socket_up;
 
-static void inline
+static void
+ipc_print_neigh_link(struct neighbor_entry *neighbor);
+
+static void
+ipc_print_tc_link(struct tc_entry *entry, struct topo_dst *dst_entry);
+
+#if 0
+static void
+ipc_print_net(union olsr_ip_addr *, union olsr_ip_addr *, union hna_netmask *);
+#endif
+
+static int
+ipc_send(const char *, int);
+
+static void
 ipc_print_neigh_link(struct neighbor_entry *);
 
-int
+static int
 plugin_ipc_init(void);
 
 void
 olsr_plugin_exit(void);
 
 
-static void inline
+static void
 ipc_print_neigh_link(struct neighbor_entry *neighbor)
 {
   char buf[256];
   int len;
-  char* main_adr;
-  char* adr;
+  const char* main_adr;
+  const char* adr;
 //  double etx=0.0;
 //  char* style = "solid";
 //  struct link_entry* link;
@@ -120,7 +134,7 @@ ipc_print_neigh_link(struct neighbor_entry *neighbor)
  *function in uolsrd_plugin.c
  */
 int
-olsrd_plugin_init()
+olsrd_plugin_init(void)
 {
 
   /* Initial IPC value */
@@ -134,7 +148,7 @@ olsrd_plugin_init()
 }
 
 int
-plugin_ipc_init()
+plugin_ipc_init(void)
 {
   struct sockaddr_in sin;
   olsr_u32_t yes = 1;
@@ -193,7 +207,7 @@ plugin_ipc_init()
 }
 
 void
-ipc_action(int fd)
+ipc_action(int fd __attribute__((unused)))
 {
   struct sockaddr_in pin;
   socklen_t addrlen;
@@ -235,7 +249,7 @@ ipc_action(int fd)
  * destructor - called at unload
  */
 void
-olsr_plugin_exit()
+olsr_plugin_exit(void)
 {
   if(ipc_open)
     close(ipc_socket);
@@ -249,7 +263,7 @@ olsr_plugin_exit()
 int
 pcf_event(int changes_neighborhood,
          int changes_topology,
-         int changes_hna)
+         int changes_hna __attribute__((unused)))
 {
   int res;
   olsr_u8_t index;
@@ -344,13 +358,13 @@ calc_etx(double loss, double neigh_loss)
 }
 #endif
 
-static void inline
+static void
 ipc_print_tc_link(struct tc_entry *entry, struct topo_dst *dst_entry)
 {
   char buf[256];
   int len;
-  char* main_adr;
-  char* adr;
+  const char* main_adr;
+  const char* adr;
 //  double etx = calc_etx( dst_entry->link_quality, dst_entry->inverse_link_quality );
 
   main_adr = olsr_ip_to_string(&entry->T_last_addr);
@@ -359,10 +373,11 @@ ipc_print_tc_link(struct tc_entry *entry, struct topo_dst *dst_entry)
   ipc_send(buf, len);
 }
 
-static void inline
+#if 0
+static void
 ipc_print_net(union olsr_ip_addr *gw, union olsr_ip_addr *net, union hna_netmask *mask)
 {
-  char *adr;
+  const char *adr;
 
   adr = olsr_ip_to_string(gw);
   ipc_send("\"", 1);
@@ -383,11 +398,11 @@ ipc_print_net(union olsr_ip_addr *gw, union olsr_ip_addr *net, union hna_netmask
   ipc_send("\"", 1);
   ipc_send("[shape=diamond];\n", strlen("[shape=diamond];\n"));
 }
-
+#endif
 
 
 int
-ipc_send(char *data, int size)
+ipc_send(const char *data, int size)
 {
   if(!ipc_open)
     return 0;
index e4dee94..f15f469 100755 (executable)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_pgraph.h,v 1.1 2005/07/13 21:45:08 kattemat Exp $
+ * $Id: olsrd_pgraph.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 /*
@@ -73,7 +73,7 @@
 extern struct in_addr ipc_accept_ip;
 extern int ipc_port;
 
-char netmask[5];
+extern char netmask[5];
 
 /* Event function to register with the sceduler */
 int
@@ -82,18 +82,6 @@ pcf_event(int, int, int);
 void
 ipc_action(int);
 
-static void inline
-ipc_print_neigh_link(struct neighbor_entry *neighbor);
-
-static void inline
-ipc_print_tc_link(struct tc_entry *entry, struct topo_dst *dst_entry);
-
-static void inline
-ipc_print_net(union olsr_ip_addr *, union olsr_ip_addr *, union hna_netmask *);
-
-int
-ipc_send(char *, int);
-
 char *
 olsr_netmask_to_string(union hna_netmask *);
 
index d3344da..b39c2c3 100755 (executable)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_plugin.c,v 1.1 2005/07/13 21:45:08 kattemat Exp $
+ * $Id: olsrd_plugin.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 /*
@@ -78,7 +78,7 @@ my_fini(void);
  * Do not alter unless you know what you are doing!
  */
 int 
-olsrd_plugin_interface_version()
+olsrd_plugin_interface_version(void)
 {
   return PLUGIN_INTERFACE_VERSION;
 }
@@ -88,7 +88,7 @@ olsrd_plugin_interface_version()
  *Constructor
  */
 void
-my_init()
+my_init(void)
 {
   /* Print plugin info to stdout */
   printf("%s\n", MOD_DESC);
@@ -104,7 +104,7 @@ my_init()
  *Destructor
  */
 void
-my_fini()
+my_fini(void)
 {
 
   /* Calls the destruction function
index 8be0276..08f62f3 100644 (file)
@@ -33,7 +33,7 @@
  * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE 
  * POSSIBILITY OF SUCH DAMAGE.
  *
- * $Id: olsrd_plugin.c,v 1.10 2005/05/29 12:47:43 br1 Exp $
+ * $Id: olsrd_plugin.c,v 1.11 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 
@@ -63,18 +63,17 @@ my_fini(void);
  * Do not alter unless you know what you are doing!
  */
 int 
-olsrd_plugin_interface_version()
+olsrd_plugin_interface_version(void)
 {
   return OLSRD_PLUGIN_INTERFACE_VERSION;
 }
 
 
-
 /**
  *Constructor
  */
 static void
-my_init()
+my_init(void)
 {
   /* Print plugin info to stdout */
   /* We cannot use olsr_printf yet! */
@@ -86,7 +85,7 @@ my_init()
  *Destructor
  */
 static void
-my_fini()
+my_fini(void)
 {
 
   /* Calls the destruction function
@@ -112,7 +111,7 @@ olsrd_plugin_register_param(char *key, char *value)
 
 
 int
-olsrd_plugin_init() {
+olsrd_plugin_init(void) {
   /* Calls the initialization function
    * olsr_plugin_init()
    * This function should be present in your
index ce860cf..23f3ee2 100644 (file)
@@ -33,7 +33,7 @@
  * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE 
  * POSSIBILITY OF SUCH DAMAGE.
  *
- * $Id: olsrd_secure.c,v 1.21 2007/01/31 12:36:49 bernd67 Exp $
+ * $Id: olsrd_secure.c,v 1.22 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 
@@ -115,7 +115,7 @@ MD5_checksum(const olsr_u8_t *data, const olsr_u16_t data_len, olsr_u8_t *hashbu
  */
 
 int
-secure_plugin_init()
+secure_plugin_init(void)
 {
   struct interface *ints;
   int i;
@@ -174,7 +174,7 @@ secure_plugin_init()
 
 
 int
-plugin_ipc_init()
+plugin_ipc_init(void)
 {
   return 1;
 }
@@ -184,7 +184,7 @@ plugin_ipc_init()
  * destructor - called at unload
  */
 void
-secure_plugin_exit()
+secure_plugin_exit(void)
 {
 }
 
@@ -193,14 +193,14 @@ secure_plugin_exit()
  *Scheduled event
  */
 void
-olsr_event()
+olsr_event(void)
 {
 
 }
 
 
 int
-ipc_send(char *data, int size)
+ipc_send(char *data __attribute__((unused)), int size __attribute__((unused)))
 {
   return 1;
 }
@@ -358,7 +358,7 @@ packet_parser(int fd)
  *
  */
 int
-check_auth(char *pck, int *size)
+check_auth(char *pck, int *size __attribute__((unused)))
 {
 
   olsr_printf(3, "[ENC]Checking packet for challenge response message...\n");
@@ -426,7 +426,7 @@ add_signature(olsr_u8_t *pck, int *size)
   
   /* Add timestamp */
   msg->sig.timestamp = htonl(now.tv_sec);
-  olsr_printf(3, "[ENC]timestamp: %d\n", now.tv_sec);
+  olsr_printf(3, "[ENC]timestamp: %ld\n", now.tv_sec);
   
   /* Set the new size */
   *size = *size + sizeof(struct s_olsrmsg);
@@ -593,7 +593,7 @@ validate_packet(char *pck, int *size)
       return 0;
     }
 
-  olsr_printf(1, "[ENC]Received timestamp %d diff: %d\n", rec_time, now.tv_sec - rec_time);
+  olsr_printf(1, "[ENC]Received timestamp %ld diff: %ld\n", rec_time, now.tv_sec - rec_time);
 
   /* Remove signature message */
   *size = packetsize;
@@ -684,7 +684,7 @@ send_challenge(union olsr_ip_addr *new_host)
   memcpy(&cmsg.destination, new_host, olsr_cnf->ipsize);
   cmsg.challenge = htonl(challenge);
 
-  olsr_printf(3, "[ENC]Size: %d\n", sizeof(struct challengemsg));
+  olsr_printf(3, "[ENC]Size: %lu\n", (unsigned long)sizeof(struct challengemsg));
 
   {
   olsr_u8_t checksum_cache[512 + KEYLENGTH];
@@ -1056,7 +1056,7 @@ send_cres(union olsr_ip_addr *to, union olsr_ip_addr *from, olsr_u32_t chal_in,
 
   /* set timestamp */
   crmsg.timestamp = now.tv_sec;
-  olsr_printf(3, "[ENC]Timestamp %d\n", crmsg.timestamp);
+  olsr_printf(3, "[ENC]Timestamp %ld\n", crmsg.timestamp);
 
   /* Fill subheader */
   memcpy(&crmsg.destination, to, olsr_cnf->ipsize);
@@ -1135,7 +1135,7 @@ send_rres(union olsr_ip_addr *to, union olsr_ip_addr *from, olsr_u32_t chal_in)
 
   /* set timestamp */
   rrmsg.timestamp = now.tv_sec;
-  olsr_printf(3, "[ENC]Timestamp %d\n", rrmsg.timestamp);
+  olsr_printf(3, "[ENC]Timestamp %ld\n", rrmsg.timestamp);
 
   /* Fill subheader */
   memcpy(&rrmsg.destination, to, olsr_cnf->ipsize);
@@ -1218,7 +1218,7 @@ lookup_timestamp_entry(union olsr_ip_addr *adr)
  *@return nada
  */
 void
-timeout_timestamps(void* foo)
+timeout_timestamps(void* foo __attribute__((unused)))
 {
   struct stamp *tmp_list;
   struct stamp *entry_to_delete;
index 52b7a9d..5cdbb97 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: glua.c,v 1.3 2005/11/15 23:46:19 tlopatic Exp $
+ * $Id: glua.c,v 1.4 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 #include "lua/lua.h"
@@ -229,7 +229,7 @@ int lspToLua(const char *rootDir, const char *lspFileName,
   return 0;
 }
 
-static int luaWriter(lua_State *lua, const void *buff, int len, FILE *file)
+static int luaWriter(lua_State *lua __attribute__((unused)), const void *buff, int len, FILE *file)
 {
   return fwrite(buff, len, 1, file) == 1;
 }
index 314ac00..7333c10 100644 (file)
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: lib.h,v 1.2 2005/04/12 19:57:26 tlopatic Exp $
+ * $Id: lib.h,v 1.3 2007/04/20 13:46:03 bernd67 Exp $
  */
 
+#ifndef _OLSRD_TAS_LIB_H
+#define _OLSRD_TAS_LIB_H
+
 #define DEBUG_MESSAGE 1
 #define DEBUG_SESSION 2
 #define DEBUG_CONNECTION 4
 #define DEBUG_LUA 16
 #define DEBUG_QUANTUM 32
 
-extern void error(char *form, ...);
-extern void debug(int facility, char *form, ...);
+extern void error(char *form, ...) __attribute__((format(printf,1,2)));
+extern void debug(int facility, char *form, ...) __attribute__((format(printf,2,3)));
 extern char *strdupAdd(const char *string, int add);
 extern char *myStrdup(const char *string);
 extern void chomp(char *line, int len);
 extern char *intToString(char *buff, unsigned int val);
 extern int stringToInt(unsigned int *val, const char *buff);
+
+#endif
index 4baf93d..df21c8f 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lapi.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: lapi.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Lua API
 ** See Copyright Notice in lua.h
 */
@@ -805,7 +805,6 @@ LUA_API int lua_error (lua_State *L) {
   api_checknelems(L, 1);
   luaG_errormsg(L);
   lua_unlock(L);
-  return 0;  /* to avoid warnings */
 }
 
 
index 940d519..2046913 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lauxlib.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: lauxlib.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Auxiliary functions for building Lua libraries
 ** See Copyright Notice in lua.h
 */
@@ -50,11 +50,11 @@ LUALIB_API int luaL_argerror (lua_State *L, int narg, const char *extramsg) {
   if (strcmp(ar.namewhat, "method") == 0) {
     narg--;  /* do not count `self' */
     if (narg == 0)  /* error is in the self argument itself? */
-      return luaL_error(L, "calling `%s' on bad self (%s)", ar.name, extramsg);
+      luaL_error(L, "calling `%s' on bad self (%s)", ar.name, extramsg);
   }
   if (ar.name == NULL)
     ar.name = "?";
-  return luaL_error(L, "bad argument #%d to `%s' (%s)",
+  luaL_error(L, "bad argument #%d to `%s' (%s)",
                         narg, ar.name, extramsg);
 }
 
@@ -62,10 +62,11 @@ LUALIB_API int luaL_argerror (lua_State *L, int narg, const char *extramsg) {
 LUALIB_API int luaL_typerror (lua_State *L, int narg, const char *tname) {
   const char *msg = lua_pushfstring(L, "%s expected, got %s",
                                     tname, lua_typename(L, lua_type(L,narg)));
-  return luaL_argerror(L, narg, msg);
+  luaL_argerror(L, narg, msg);
 }
 
 
+static void tag_error (lua_State *L, int narg, int tag) __attribute__((noreturn));
 static void tag_error (lua_State *L, int narg, int tag) {
   luaL_typerror(L, narg, lua_typename(L, tag)); 
 }
@@ -91,7 +92,7 @@ LUALIB_API int luaL_error (lua_State *L, const char *fmt, ...) {
   lua_pushvfstring(L, fmt, argp);
   va_end(argp);
   lua_concat(L, 2);
-  return lua_error(L);
+  lua_error(L);
 }
 
 /* }====================================================== */
index c95cfee..52d77d5 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lauxlib.h,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: lauxlib.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Auxiliary functions for building Lua libraries
 ** See Copyright Notice in lua.h
 */
@@ -31,8 +31,8 @@ LUALIB_API void luaL_openlib (lua_State *L, const char *libname,
                                const luaL_reg *l, int nup);
 LUALIB_API int luaL_getmetafield (lua_State *L, int obj, const char *e);
 LUALIB_API int luaL_callmeta (lua_State *L, int obj, const char *e);
-LUALIB_API int luaL_typerror (lua_State *L, int narg, const char *tname);
-LUALIB_API int luaL_argerror (lua_State *L, int numarg, const char *extramsg);
+LUALIB_API int luaL_typerror (lua_State *L, int narg, const char *tname) __attribute__((noreturn));
+LUALIB_API int luaL_argerror (lua_State *L, int numarg, const char *extramsg) __attribute__((noreturn));
 LUALIB_API const char *luaL_checklstring (lua_State *L, int numArg, size_t *l);
 LUALIB_API const char *luaL_optlstring (lua_State *L, int numArg,
                                            const char *def, size_t *l);
@@ -48,7 +48,7 @@ LUALIB_API void  luaL_getmetatable (lua_State *L, const char *tname);
 LUALIB_API void *luaL_checkudata (lua_State *L, int ud, const char *tname);
 
 LUALIB_API void luaL_where (lua_State *L, int lvl);
-LUALIB_API int luaL_error (lua_State *L, const char *fmt, ...);
+LUALIB_API int luaL_error (lua_State *L, const char *fmt, ...) __attribute__((noreturn));
 
 LUALIB_API int luaL_findstring (const char *st, const char *const lst[]);
 
index f141e0d..ab91a19 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lbaselib.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: lbaselib.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Basic library
 ** See Copyright Notice in lua.h
 */
@@ -75,7 +75,7 @@ static int luaB_tonumber (lua_State *L) {
   return 1;
 }
 
-
+static int luaB_error (lua_State *L)  __attribute__((noreturn));
 static int luaB_error (lua_State *L) {
   int level = luaL_optint(L, 2, 1);
   luaL_checkany(L, 1);
@@ -86,7 +86,7 @@ static int luaB_error (lua_State *L) {
     lua_pushvalue(L, 1);
     lua_concat(L, 2);
   }
-  return lua_error(L);
+  lua_error(L);
 }
 
 
index e5a5631..a2484cd 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: ldebug.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: ldebug.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Debug Interface
 ** See Copyright Notice in lua.h
 */
@@ -547,7 +547,6 @@ int luaG_ordererror (lua_State *L, const TObject *p1, const TObject *p2) {
     luaG_runerror(L, "attempt to compare two %s values", t1);
   else
     luaG_runerror(L, "attempt to compare %s with %s", t1, t2);
-  return 0;
 }
 
 
index 1e7efef..9673e86 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: ldebug.h,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: ldebug.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Auxiliary functions from Debug Interface module
 ** See Copyright Notice in lua.h
 */
 
 
 void luaG_inithooks (lua_State *L);
-void luaG_typeerror (lua_State *L, const TObject *o, const char *opname);
-void luaG_concaterror (lua_State *L, StkId p1, StkId p2);
-void luaG_aritherror (lua_State *L, const TObject *p1, const TObject *p2);
-int luaG_ordererror (lua_State *L, const TObject *p1, const TObject *p2);
-void luaG_runerror (lua_State *L, const char *fmt, ...);
-void luaG_errormsg (lua_State *L);
+void luaG_typeerror (lua_State *L, const TObject *o, const char *opname) __attribute__((noreturn));
+void luaG_concaterror (lua_State *L, StkId p1, StkId p2) __attribute__((noreturn));
+void luaG_aritherror (lua_State *L, const TObject *p1, const TObject *p2) __attribute__((noreturn));
+int luaG_ordererror (lua_State *L, const TObject *p1, const TObject *p2) __attribute__((noreturn));
+void luaG_runerror (lua_State *L, const char *fmt, ...) __attribute__((noreturn));
+void luaG_errormsg (lua_State *L) __attribute__((noreturn));
 int luaG_checkcode (const Proto *pt);
 
 
index 6a92cfe..1d833b2 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: ldo.h,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: ldo.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Stack and Call structure of Lua
 ** See Copyright Notice in lua.h
 */
@@ -53,7 +53,7 @@ void luaD_reallocCI (lua_State *L, int newsize);
 void luaD_reallocstack (lua_State *L, int newsize);
 void luaD_growstack (lua_State *L, int n);
 
-void luaD_throw (lua_State *L, int errcode);
+void luaD_throw (lua_State *L, int errcode) __attribute__((noreturn));
 int luaD_rawrunprotected (lua_State *L, Pfunc f, void *ud);
 
 
index 6375370..c41b122 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: liolib.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: liolib.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Standard I/O (and system) library
 ** See Copyright Notice in lua.h
 */
@@ -50,6 +50,8 @@
 
 
 
+static int io_exit (lua_State *L) __attribute__((noreturn));
+
 
 /*
 ** {======================================================
@@ -555,10 +557,15 @@ static int io_rename (lua_State *L) {
 }
 
 
+#if !USE_TMPNAME
+static int io_tmpname (lua_State *L) __attribute__((noreturn));
+#else
+static int io_tmpname (lua_State *L);
+#endif
+
 static int io_tmpname (lua_State *L) {
 #if !USE_TMPNAME
   luaL_error(L, "`tmpname' not supported");
-  return 0;
 #else
   char buff[L_tmpnam];
   if (tmpnam(buff) != buff)
@@ -714,7 +721,6 @@ static int io_setloc (lua_State *L) {
 
 static int io_exit (lua_State *L) {
   exit(luaL_optint(L, 1, EXIT_SUCCESS));
-  return 0;  /* to avoid warnings */
 }
 
 static const luaL_reg syslib[] = {
index 25e252f..dbe3b9c 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: llex.c,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: llex.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Lexical Analyzer
 ** See Copyright Notice in lua.h
 */
 #include "lzio.h"
 
 
-
 #define next(LS) (LS->current = zgetc(LS->z))
 
+static void luaX_error (LexState *ls, const char *s, const char *token) __attribute__((noreturn));
+static void luaX_lexerror (LexState *ls, const char *s, int token) __attribute__((noreturn));
 
 
 /* ORDER RESERVED */
@@ -58,7 +59,6 @@ void luaX_checklimit (LexState *ls, int val, int limit, const char *msg) {
   }
 }
 
-
 void luaX_errorline (LexState *ls, const char *s, const char *token, int line) {
   lua_State *L = ls->L;
   char buff[MAXSRC];
@@ -67,7 +67,6 @@ void luaX_errorline (LexState *ls, const char *s, const char *token, int line) {
   luaD_throw(L, LUA_ERRSYNTAX);
 }
 
-
 static void luaX_error (LexState *ls, const char *s, const char *token) {
   luaX_errorline(ls, s, token, ls->linenumber);
 }
index 7d58785..9ce0fd8 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: llex.h,v 1.1 2005/04/12 17:17:26 tlopatic Exp $
+** $Id: llex.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Lexical Analyzer
 ** See Copyright Notice in lua.h
 */
@@ -67,8 +67,8 @@ void luaX_init (lua_State *L);
 void luaX_setinput (lua_State *L, LexState *LS, ZIO *z, TString *source);
 int luaX_lex (LexState *LS, SemInfo *seminfo);
 void luaX_checklimit (LexState *ls, int val, int limit, const char *msg);
-void luaX_syntaxerror (LexState *ls, const char *s);
-void luaX_errorline (LexState *ls, const char *s, const char *token, int line);
+void luaX_syntaxerror (LexState *ls, const char *s) __attribute__((noreturn));
+void luaX_errorline (LexState *ls, const char *s, const char *token, int line) __attribute__((noreturn));
 const char *luaX_token2str (LexState *ls, int token);
 
 
index 1ac4b40..b073818 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lparser.c,v 1.1 2005/04/12 17:17:27 tlopatic Exp $
+** $Id: lparser.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Lua Parser
 ** See Copyright Notice in lua.h
 */
@@ -51,6 +51,7 @@ typedef struct BlockCnt {
 */
 static void chunk (LexState *ls);
 static void expr (LexState *ls, expdesc *v);
+static void error_expected (LexState *ls, int token) __attribute__((noreturn));
 
 
 
index f65f7dd..755f223 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lua.h,v 1.1 2005/04/12 17:17:27 tlopatic Exp $
+** $Id: lua.h,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** Lua - An Extensible Extension Language
 ** Tecgraf: Computer Graphics Technology Group, PUC-Rio, Brazil
 ** http://www.lua.org  mailto:info@lua.org
@@ -213,7 +213,7 @@ LUA_API void  lua_setgcthreshold (lua_State *L, int newthreshold);
 
 LUA_API const char *lua_version (void);
 
-LUA_API int   lua_error (lua_State *L);
+LUA_API int   lua_error (lua_State *L) __attribute__((noreturn));
 
 LUA_API int   lua_next (lua_State *L, int idx);
 
index 029838e..46a32b6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-** $Id: lundump.c,v 1.1 2005/04/12 17:17:35 tlopatic Exp $
+** $Id: lundump.c,v 1.2 2007/04/20 13:46:03 bernd67 Exp $
 ** load pre-compiled Lua chunks
 ** See Copyright Notice in lua.h
 */
@@ -26,6 +26,8 @@ typedef struct {
  const char* name;
 } LoadState;
 
+static void unexpectedEOZ (LoadState* S) __attribute__((noreturn));
+
 static void unexpectedEOZ (LoadState* S)
 {
  luaG_runerror(S->L,"unexpected end of file in %s",S->name);
index c584cf0..858a48b 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: plugin.c,v 1.7 2006/01/07 08:17:45 kattemat Exp $
+ * $Id: plugin.c,v 1.8 2007/04/20 13:46:03 bernd67 Exp $
  */
 
 #include <string.h>
@@ -465,7 +465,7 @@ int olsrd_plugin_interface_version(void)
   return 4;
 }
 
-int olsrd_plugin_init()
+int olsrd_plugin_init(void)
 {
   ipAddrLen = olsr_cnf->ipsize;
   mainAddr = &olsr_cnf->main_addr;
index 327e141..5b4405c 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_plugin.c,v 1.1 2007/01/31 12:38:26 bernd67 Exp $
+ * $Id: olsrd_plugin.c,v 1.2 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /*
@@ -78,7 +78,7 @@ my_fini(void);
  *Constructor
  */
 static void
-my_init()
+my_init(void)
 {
   /* Print plugin info to stdout */
   printf("%s\n", MOD_DESC);
@@ -96,7 +96,7 @@ my_init()
  *Destructor
  */
 static void
-my_fini()
+my_fini(void)
 {
   /* Calls the destruction function
    * olsr_plugin_exit()
@@ -109,7 +109,7 @@ my_fini()
 
 
 int 
-olsrd_plugin_interface_version()
+olsrd_plugin_interface_version(void)
 {
   return PLUGIN_INTERFACE_VERSION;
 }
index 56040f6..98540e7 100644 (file)
@@ -40,7 +40,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsrd_txtinfo.c,v 1.2 2007/03/27 03:59:27 tlopatic Exp $
+ * $Id: olsrd_txtinfo.c,v 1.3 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /*
@@ -101,21 +101,21 @@ send_info(int neighonly);
 static void
 ipc_action(int);
 
-static void inline
+static void
 ipc_print_neigh_link(void);
 
-static void inline
+static void
 ipc_print_routes(void);
 
-static void inline
+static void
 ipc_print_topology(void);
 
-static void inline
+static void
 ipc_print_hna(void);
 
 #define TXT_IPC_BUFSIZE 256
 static int 
-ipc_sendf(const char* format, ...);
+ipc_sendf(const char* format, ...) __attribute__((format(printf, 1, 2)));
 
 /**
  *Do initialization here
@@ -124,7 +124,7 @@ ipc_sendf(const char* format, ...);
  *function in uolsrd_plugin.c
  */
 int
-olsrd_plugin_init()
+olsrd_plugin_init(void)
 {
   /* Initial IPC value */
   ipc_open = 0;
@@ -139,7 +139,7 @@ olsrd_plugin_init()
  * destructor - called at unload
  */
 void
-olsr_plugin_exit()
+olsr_plugin_exit(void)
 {
   if(ipc_open)
     close(ipc_socket);
@@ -148,7 +148,7 @@ olsr_plugin_exit()
 
 
 static int
-plugin_ipc_init()
+plugin_ipc_init(void)
 {
   struct sockaddr_in sin;
   olsr_u32_t yes = 1;
@@ -280,7 +280,7 @@ ipc_action(int fd)
   }
 }
 
-static void inline
+static void
 ipc_print_neigh_link(void)
 {
   struct neighbor_entry *neigh;
@@ -339,7 +339,7 @@ ipc_print_neigh_link(void)
 }
 
 
-static void inline
+static void
 ipc_print_routes(void)
 {
   int size = 0, index;
@@ -383,7 +383,7 @@ ipc_print_routes(void)
 
 }
 
-static void inline
+static void
 ipc_print_topology(void)
 {
   olsr_u8_t index;
@@ -420,7 +420,7 @@ ipc_print_topology(void)
   ipc_sendf("\n");
 }
 
-static void inline
+static void
 ipc_print_hna(void)
 {
   int size;
index 26dc4bd..a4b9bec 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net.h,v 1.3 2004/11/21 10:50:52 kattemat Exp $
+ * $Id: net.h,v 1.4 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -55,7 +55,4 @@
 #include <netinet/in.h>
 #include "../olsr_protocol.h"
 
-extern int
-olsr_printf(int, char *, ...);
-
 #endif
index f8cb817..6164388 100644 (file)
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: build_msg.c,v 1.33 2006/12/14 11:29:19 bernd67 Exp $
+ * $Id: build_msg.c,v 1.34 2007/04/20 13:46:04 bernd67 Exp $
  */
 
-
 #include "defs.h"
 #include "olsr.h"
 #include "log.h"
@@ -113,7 +112,7 @@ set_empty_tc_timer(clock_t empty_tc_new)
  * empty TC messages
  */
 clock_t
-get_empty_tc_timer()
+get_empty_tc_timer(void)
 {
   return send_empty_tc;
 }
index 2b14184..0452791 100644 (file)
@@ -36,7 +36,7 @@
 # to the project. For more information see the website or contact
 # the copyright holders.
 #
-# $Id: Makefile,v 1.22 2007/03/27 03:05:21 tlopatic Exp $
+# $Id: Makefile,v 1.23 2007/04/20 13:46:05 bernd67 Exp $
 
 TOPDIR=../..
 include $(TOPDIR)/Makefile.inc
@@ -89,11 +89,20 @@ default_target: $(NAME)
 $(NAME):       $(OBJS)
                $(CC) $(LDFLAGS) -o $(NAME) $(OBJS) $(LIBS)
 
-oscan.c:       oscan.lex
-               $(FLEX) -ooscan.c oscan.lex
+oscan.c:       oscan.lex Makefile
+               $(FLEX) -Cem -ooscan.c oscan.lex
+               mv oscan.c oscan.c-tmp && \
+                       sed     -e '/^static/s/yy_get_next_buffer[\(][\)]/yy_get_next_buffer(void)/' \
+                               -e '/^static/s/yy_get_previous_state[\(][\)]/yy_get_previous_state(void)/' \
+                               -e '/^static/s/input[\(][\)]/input(void)/' \
+                               -e '/^static  *void  *yy_fatal_error/s/^\(.*)\)\();\)$$/\1 __attribute__((noreturn)) \2/' \
+                               < oscan.c-tmp >oscan.c
+               $(RM) oscan.c-tmp
 
-oparse.c:      oparse.y olsrd_conf.h
-               $(BISON) -d -ooparse.c oparse.y
+oparse.c:      oparse.y olsrd_conf.h Makefile
+               $(BISON) -d -o oparse.c oparse.y
+
+#oscan.o oparse.o:: CFLAGS := $(filter-out -Wunreachable-code, $(CFLAGS))
 
 .PHONY: install clean uberclean
 
@@ -105,6 +114,7 @@ clean:
                rm -f *.[od~]
                rm -f $(LIBNAME)
                rm -f $(BINNAME)
+               $(RM) oparse.c-tmp
 
 uberclean:     clean
                rm -f oparse.c oparse.h oscan.c
index 4788f41..17bf312 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.51 2007/03/14 13:59:30 bernd67 Exp $
+ * $Id: olsrd_conf.c,v 1.52 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -419,7 +419,7 @@ olsrd_free_cnf(struct olsrd_config *cnf)
 
 
 struct olsrd_config *
-olsrd_get_default_cnf()
+olsrd_get_default_cnf(void)
 {
   cnf = malloc(sizeof(struct olsrd_config));
   if (cnf == NULL)
@@ -481,7 +481,7 @@ set_default_cnf(struct olsrd_config *cnf)
 
 
 struct if_config_options *
-get_default_if_config()
+get_default_if_config(void)
 {
   struct if_config_options *io = malloc(sizeof(struct if_config_options));
   struct in6_addr in6;
@@ -732,5 +732,5 @@ void win32_stdio_hack(unsigned int handle)
   setbuf(stderr, NULL);
 }
 #else
-void win32_stdio_hack(unsigned int handle) {}
+void win32_stdio_hack(unsigned int handle __attribute__((unused))) {}
 #endif
index be86710..94cb298 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.31 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: oparse.y,v 1.32 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
 
 #define PARSER_DEBUG 0
 
+#if PARSER_DEBUG
+#define PARSER_DEBUG_PRINTF(x, ...)   printf(x, ##args)
+#else
+#define PARSER_DEBUG_PRINTF(x, ...)   do { } while (0)
+#endif
+
 #define YYSTYPE struct conf_token *
 
 void yyerror(char *);
@@ -305,7 +311,7 @@ ipchost: TOK_HOSTLABEL TOK_IP4_ADDR
   struct in_addr in;
   struct ipc_host *ipch;
 
-  if(PARSER_DEBUG) printf("\tIPC host: %s\n", $2->string);
+  PARSER_DEBUG_PRINTF("\tIPC host: %s\n", $2->string);
   
   if(inet_aton($2->string, &in) == 0)
     {
@@ -330,7 +336,7 @@ ipcnet: TOK_NETLABEL TOK_IP4_ADDR TOK_IP4_ADDR
   struct in_addr in1, in2;
   struct ipc_net *ipcn;
 
-  if(PARSER_DEBUG) printf("\tIPC net: %s/%s\n", $2->string, $3->string);
+  PARSER_DEBUG_PRINTF("\tIPC net: %s/%s\n", $2->string, $3->string);
   
   if(inet_aton($2->string, &in1) == 0)
     {
@@ -364,7 +370,7 @@ iifweight:       TOK_IFWEIGHT TOK_INTEGER
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("Fixed willingness: %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("Fixed willingness: %d\n", $2->integer);
 
   while(ifcnt)
     {
@@ -385,7 +391,7 @@ isetip4br: TOK_IP4BROADCAST TOK_IP4_ADDR
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tIPv4 broadcast: %s\n", $2->string);
+  PARSER_DEBUG_PRINTF("\tIPv4 broadcast: %s\n", $2->string);
 
   if(inet_aton($2->string, &in) == 0)
     {
@@ -442,7 +448,7 @@ isetip6mults: TOK_IP6MULTISITE TOK_IP6_ADDR
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tIPv6 site-local multicast: %s\n", $2->string);
+  PARSER_DEBUG_PRINTF("\tIPv6 site-local multicast: %s\n", $2->string);
 
   if(inet_pton(AF_INET6, $2->string, &in6) < 0)
     {
@@ -471,7 +477,7 @@ isetip6multg: TOK_IP6MULTIGLOBAL TOK_IP6_ADDR
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tIPv6 global multicast: %s\n", $2->string);
+  PARSER_DEBUG_PRINTF("\tIPv6 global multicast: %s\n", $2->string);
 
   if(inet_pton(AF_INET6, $2->string, &in6) < 0)
     {
@@ -497,7 +503,7 @@ isethelloint: TOK_HELLOINT TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tHELLO interval: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tHELLO interval: %0.2f\n", $2->floating);
 
   while(ifcnt)
     {
@@ -515,7 +521,7 @@ isethelloval: TOK_HELLOVAL TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tHELLO validity: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tHELLO validity: %0.2f\n", $2->floating);
 
   while(ifcnt)
     {
@@ -533,7 +539,7 @@ isettcint: TOK_TCINT TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tTC interval: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tTC interval: %0.2f\n", $2->floating);
 
   while(ifcnt)
     {
@@ -550,7 +556,7 @@ isettcval: TOK_TCVAL TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
   
-  if(PARSER_DEBUG) printf("\tTC validity: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tTC validity: %0.2f\n", $2->floating);
   while(ifcnt)
     {
       ifs->cnf->tc_params.validity_time = $2->floating;
@@ -568,7 +574,7 @@ isetmidint: TOK_MIDINT TOK_FLOAT
   struct olsr_if *ifs = cnf->interfaces;
 
 
-  if(PARSER_DEBUG) printf("\tMID interval: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tMID interval: %0.2f\n", $2->floating);
   while(ifcnt)
     {
       ifs->cnf->mid_params.emission_interval = $2->floating;
@@ -585,7 +591,7 @@ isetmidval: TOK_MIDVAL TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tMID validity: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tMID validity: %0.2f\n", $2->floating);
   while(ifcnt)
     {
       ifs->cnf->mid_params.validity_time = $2->floating;
@@ -602,7 +608,7 @@ isethnaint: TOK_HNAINT TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
   
-  if(PARSER_DEBUG) printf("\tHNA interval: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tHNA interval: %0.2f\n", $2->floating);
   while(ifcnt)
     {
       ifs->cnf->hna_params.emission_interval = $2->floating;
@@ -619,7 +625,7 @@ isethnaval: TOK_HNAVAL TOK_FLOAT
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tHNA validity: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("\tHNA validity: %0.2f\n", $2->floating);
   while(ifcnt)
     {
       ifs->cnf->hna_params.validity_time = $2->floating;
@@ -636,7 +642,7 @@ isetautodetchg: TOK_AUTODETCHG TOK_BOOLEAN
   int ifcnt = ifs_in_curr_cfg;
   struct olsr_if *ifs = cnf->interfaces;
 
-  if(PARSER_DEBUG) printf("\tAutodetect changes: %s\n", $2->boolean ? "YES" : "NO");
+  PARSER_DEBUG_PRINTF("\tAutodetect changes: %s\n", $2->boolean ? "YES" : "NO");
   while(ifcnt)
     {
       ifs->cnf->autodetect_chg = $2->boolean;
@@ -673,7 +679,7 @@ idebug:       TOK_DEBUGLEVEL TOK_INTEGER
 {
 
   cnf->debug_level = $2->integer;
-  if(PARSER_DEBUG) printf("Debug level: %d\n", cnf->debug_level);
+  PARSER_DEBUG_PRINTF("Debug level: %d\n", cnf->debug_level);
   free($2);
 }
 ;
@@ -691,7 +697,7 @@ iipversion:    TOK_IPVERSION TOK_INTEGER
       YYABORT;
     }
 
-  if(PARSER_DEBUG) printf("IpVersion: %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("IpVersion: %d\n", $2->integer);
   free($2);
 }
 ;
@@ -702,7 +708,7 @@ ihna4entry:     TOK_IP4_ADDR TOK_IP4_ADDR
   struct hna4_entry *h = malloc(sizeof(struct hna4_entry));
   struct in_addr in;
 
-  if(PARSER_DEBUG) printf("HNA IPv4 entry: %s/%s\n", $1->string, $2->string);
+  PARSER_DEBUG_PRINTF("HNA IPv4 entry: %s/%s\n", $1->string, $2->string);
 
   if(h == NULL)
     {
@@ -740,7 +746,7 @@ ihna6entry:     TOK_IP6_ADDR TOK_INTEGER
   struct hna6_entry *h = malloc(sizeof(struct hna6_entry));
   struct in6_addr in6;
 
-  if(PARSER_DEBUG) printf("HNA IPv6 entry: %s/%d\n", $1->string, $2->integer);
+  PARSER_DEBUG_PRINTF("HNA IPv6 entry: %s/%d\n", $1->string, $2->integer);
 
   if(h == NULL)
     {
@@ -755,7 +761,7 @@ ihna6entry:     TOK_IP6_ADDR TOK_INTEGER
     }
   memcpy(&h->net, &in6, sizeof(struct in6_addr));
 
-  if(($2->integer < 0) || ($2->integer > 128))
+  if($2->integer > 128)
     {
       fprintf(stderr, "ihna6entry: Illegal IPv6 prefix length %d\n", $2->integer);
       return -1;
@@ -775,7 +781,7 @@ ihna6entry:     TOK_IP6_ADDR TOK_INTEGER
 
 ifstart: TOK_INTERFACE
 {
-  if(PARSER_DEBUG) printf("setting ifs_in_curr_cfg = 0\n");
+  PARSER_DEBUG_PRINTF("setting ifs_in_curr_cfg = 0\n");
   ifs_in_curr_cfg = 0;
 }
 ;
@@ -812,7 +818,7 @@ ifnick: TOK_STRING
 
 bnoint: TOK_NOINT TOK_BOOLEAN
 {
-  if(PARSER_DEBUG) printf("Noint set to %d\n", $2->boolean);
+  PARSER_DEBUG_PRINTF("Noint set to %d\n", $2->boolean);
 
   cnf->allow_no_interfaces = $2->boolean;
 
@@ -822,7 +828,7 @@ bnoint: TOK_NOINT TOK_BOOLEAN
 
 atos: TOK_TOS TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("TOS: %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("TOS: %d\n", $2->integer);
   cnf->tos = $2->integer;
 
   free($2);
@@ -834,7 +840,7 @@ awillingness: TOK_WILLINGNESS TOK_INTEGER
 {
   cnf->willingness_auto = OLSR_FALSE;
 
-  if(PARSER_DEBUG) printf("Willingness: %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("Willingness: %d\n", $2->integer);
   cnf->willingness = $2->integer;
 
   free($2);
@@ -849,11 +855,11 @@ busehyst: TOK_USEHYST TOK_BOOLEAN
   cnf->use_hysteresis = $2->boolean;
   if(cnf->use_hysteresis)
     {
-      if(PARSER_DEBUG) printf("Hysteresis enabled\n");
+      PARSER_DEBUG_PRINTF("Hysteresis enabled\n");
     }
   else
     {
-      if(PARSER_DEBUG) printf("Hysteresis disabled\n");
+      PARSER_DEBUG_PRINTF("Hysteresis disabled\n");
     }
   free($2);
 
@@ -864,7 +870,7 @@ busehyst: TOK_USEHYST TOK_BOOLEAN
 fhystscale: TOK_HYSTSCALE TOK_FLOAT
 {
   cnf->hysteresis_param.scaling = $2->floating;
-  if(PARSER_DEBUG) printf("Hysteresis Scaling: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("Hysteresis Scaling: %0.2f\n", $2->floating);
   free($2);
 }
 ;
@@ -873,7 +879,7 @@ fhystscale: TOK_HYSTSCALE TOK_FLOAT
 fhystupper: TOK_HYSTUPPER TOK_FLOAT
 {
   cnf->hysteresis_param.thr_high = $2->floating;
-  if(PARSER_DEBUG) printf("Hysteresis UpperThr: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("Hysteresis UpperThr: %0.2f\n", $2->floating);
   free($2);
 }
 ;
@@ -882,14 +888,14 @@ fhystupper: TOK_HYSTUPPER TOK_FLOAT
 fhystlower: TOK_HYSTLOWER TOK_FLOAT
 {
   cnf->hysteresis_param.thr_low = $2->floating;
-  if(PARSER_DEBUG) printf("Hysteresis LowerThr: %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("Hysteresis LowerThr: %0.2f\n", $2->floating);
   free($2);
 }
 ;
 
 fpollrate: TOK_POLLRATE TOK_FLOAT
 {
-  if(PARSER_DEBUG) printf("Pollrate %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("Pollrate %0.2f\n", $2->floating);
   cnf->pollrate = $2->floating;
 
   free($2);
@@ -898,7 +904,7 @@ fpollrate: TOK_POLLRATE TOK_FLOAT
 
 fnicchgspollrt: TOK_NICCHGSPOLLRT TOK_FLOAT
 {
-  if(PARSER_DEBUG) printf("NIC Changes Pollrate %0.2f\n", $2->floating);
+  PARSER_DEBUG_PRINTF("NIC Changes Pollrate %0.2f\n", $2->floating);
   cnf->nic_chgs_pollrate = $2->floating;
 
   free($2);
@@ -907,7 +913,7 @@ fnicchgspollrt: TOK_NICCHGSPOLLRT TOK_FLOAT
 
 atcredundancy: TOK_TCREDUNDANCY TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("TC redundancy %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("TC redundancy %d\n", $2->integer);
   cnf->tc_redundancy = $2->integer;
   free($2);
 }
@@ -915,7 +921,7 @@ atcredundancy: TOK_TCREDUNDANCY TOK_INTEGER
 
 amprcoverage: TOK_MPRCOVERAGE TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("MPR coverage %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("MPR coverage %d\n", $2->integer);
   cnf->mpr_coverage = $2->integer;
   free($2);
 }
@@ -923,7 +929,7 @@ amprcoverage: TOK_MPRCOVERAGE TOK_INTEGER
 
 alq_level: TOK_LQ_LEVEL TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("Link quality level %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("Link quality level %d\n", $2->integer);
   cnf->lq_level = $2->integer;
   free($2);
 }
@@ -931,7 +937,7 @@ alq_level: TOK_LQ_LEVEL TOK_INTEGER
 
 alq_fish: TOK_LQ_FISH TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("Link quality fish eye %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("Link quality fish eye %d\n", $2->integer);
   cnf->lq_fish = $2->integer;
   free($2);
 }
@@ -939,7 +945,7 @@ alq_fish: TOK_LQ_FISH TOK_INTEGER
 
 alq_dlimit: TOK_LQ_DLIMIT TOK_INTEGER TOK_FLOAT
 {
-  if(PARSER_DEBUG) printf("Link quality dijkstra limit %d, %0.2f\n", $2->integer, $3->floating);
+  PARSER_DEBUG_PRINTF("Link quality dijkstra limit %d, %0.2f\n", $2->integer, $3->floating);
   cnf->lq_dlimit = $2->integer;
   cnf->lq_dinter = $3->floating;
   free($2);
@@ -948,7 +954,7 @@ alq_dlimit: TOK_LQ_DLIMIT TOK_INTEGER TOK_FLOAT
 
 alq_wsize: TOK_LQ_WSIZE TOK_INTEGER
 {
-  if(PARSER_DEBUG) printf("Link quality window size %d\n", $2->integer);
+  PARSER_DEBUG_PRINTF("Link quality window size %d\n", $2->integer);
   cnf->lq_wsize = $2->integer;
   free($2);
 }
@@ -958,8 +964,7 @@ bclear_screen: TOK_CLEAR_SCREEN TOK_BOOLEAN
 {
   cnf->clear_screen = $2->boolean;
 
-  if (PARSER_DEBUG)
-    printf("Clear screen %s\n", cnf->clear_screen ? "enabled" : "disabled");
+  PARSER_DEBUG_PRINTF("Clear screen %s\n", cnf->clear_screen ? "enabled" : "disabled");
 
   free($2);
 }
@@ -979,7 +984,7 @@ plblock: TOK_PLUGIN TOK_STRING
 
   pe->params = NULL;
   
-  if(PARSER_DEBUG) printf("Plugin: %s\n", $2->string);
+  PARSER_DEBUG_PRINTF("Plugin: %s\n", $2->string);
 
   /* Queue */
   pe->next = cnf->plugins;
@@ -999,7 +1004,7 @@ plparam: TOK_PLPARAM TOK_STRING TOK_STRING
       YYABORT;
     }
   
-  if(PARSER_DEBUG) printf("Plugin param key:\"%s\" val: \"%s\"\n", $2->string, $3->string);
+  PARSER_DEBUG_PRINTF("Plugin param key:\"%s\" val: \"%s\"\n", $2->string, $3->string);
   
   pp->key = $2->string;
   pp->value = $3->string;
@@ -1015,7 +1020,7 @@ plparam: TOK_PLPARAM TOK_STRING TOK_STRING
 
 vcomment:       TOK_COMMENT
 {
-    //if(PARSER_DEBUG) printf("Comment\n");
+    //PARSER_DEBUG_PRINTF("Comment\n");
 }
 ;
 
index f49d9f8..06aba4d 100644 (file)
@@ -38,7 +38,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: oscan.lex,v 1.21 2006/04/17 18:31:09 kattemat Exp $
+ * $Id: oscan.lex,v 1.22 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -74,7 +74,7 @@ struct conf_token *
 get_conf_token(void);
 
 struct conf_token *
-get_conf_token()
+get_conf_token(void)
 {
   struct conf_token *t = malloc(sizeof(struct conf_token));
 
index d5c372e..677b459 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: defs.h,v 1.56 2007/04/02 09:40:48 tlopatic Exp $
+ * $Id: defs.h,v 1.57 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
 #define MIN_PACKET_SIZE(ver)  (int)(sizeof(olsr_u8_t) * ((ver == AF_INET) ? 4 : 7))
 /* Debug helper macro */
 #ifdef DEBUG
-#define olsr_debug(lvl,format,args...) \
+#define olsr_debug(lvl,format,args...) do {                         \
    OLSR_PRINTF(lvl, "%s (%s:%d): ", __func__, __FILE__, __LINE__); \
-   OLSR_PRINTF(lvl, format, ##args);
+   OLSR_PRINTF(lvl, format, ##args); \
+  } while (0)
 #endif
 
 extern FILE *debug_handle;
@@ -101,6 +102,8 @@ extern FILE *debug_handle;
         ((int)((s1) - now_times) < 0)
 
 
+#define ARRAYSIZE(x) (sizeof(x)/sizeof(*(x)))
+
 /*
  * Queueing macros
  */
@@ -118,6 +121,8 @@ extern FILE *debug_handle;
        elem->next->prev = elem->prev
 
 
+#define CLOSE(fd)  do { close(fd); (fd) = -1;} while (0)
+
 /*
  * Global olsrd configuragtion
  */
@@ -148,9 +153,6 @@ int
 ipc_output(struct olsr *);
 
 int
-ipc_send_net_info(void);
-
-int
 ipc_route_send_rtentry(union olsr_ip_addr*, union olsr_ip_addr *, int, int, char *);
 
 #endif
index 7150a2b..af2aac8 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: duplicate_set.c,v 1.13 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: duplicate_set.c,v 1.14 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -57,7 +57,7 @@ static float dup_hold_time = DUP_HOLD_TIME;
  *@return nada
  */
 void
-olsr_init_duplicate_table()
+olsr_init_duplicate_table(void)
 {
   int i;
 
@@ -225,7 +225,7 @@ olsr_del_dup_entry(struct dup_entry *entry)
 
 
 void
-olsr_time_out_duplicate_table(void *foo)
+olsr_time_out_duplicate_table(void *foo __attribute__((unused)))
 {
   int i;
 
@@ -344,14 +344,8 @@ olsr_set_dup_forward(union olsr_ip_addr *originator,
   return 1;
 }
 
-
-
-
-
-
-
 void
-olsr_print_duplicate_table()
+olsr_print_duplicate_table(void)
 {
   int i;
 
@@ -370,6 +364,6 @@ olsr_print_duplicate_table()
          tmp_dup_table = tmp_dup_table->next;
        }
     }
-printf("\n");
+  printf("\n");
 
 }
index a1da5ab..e19896c 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: generate_msg.c,v 1.24 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: generate_msg.c,v 1.25 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "generate_msg.h"
@@ -121,7 +121,7 @@ generate_hna(void *p)
 
 
 void
-generate_stdout_pulse(void *foo)
+generate_stdout_pulse(void *foo __attribute__((unused)))
 {
   if(olsr_cnf->debug_level == 0)
     return;
index be79d05..f8ae670 100644 (file)
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: hashing.c,v 1.10 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: hashing.c,v 1.11 2007/04/20 13:46:04 bernd67 Exp $
  */
 
-
 #include "olsr_protocol.h"
 #include "hashing.h"
 #include "defs.h"
index 06aa20f..2b40994 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: hna_set.c,v 1.17 2005/05/29 12:47:45 br1 Exp $
+ * $Id: hna_set.c,v 1.18 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -52,7 +52,7 @@ size_t netmask_size;
  *Initialize the HNA set
  */
 int
-olsr_init_hna_set()
+olsr_init_hna_set(void)
 {
 
   int index;
@@ -265,7 +265,7 @@ olsr_update_hna_entry(union olsr_ip_addr *gw, union olsr_ip_addr *net, union hna
  *@return nada
  */
 void
-olsr_time_out_hna_set(void *foo)
+olsr_time_out_hna_set(void *foo __attribute__((unused)))
 {
   int index;
 
@@ -319,7 +319,7 @@ olsr_time_out_hna_set(void *foo)
  *@return nada
  */
 void
-olsr_print_hna_set()
+olsr_print_hna_set(void)
 {
   int index;
 
index 998a02d..7a64ad1 100644 (file)
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: hysteresis.h,v 1.8 2006/12/14 11:29:19 bernd67 Exp $
+ * $Id: hysteresis.h,v 1.9 2007/04/20 13:46:04 bernd67 Exp $
  */
-
-
 #ifndef _OLSR_HYSTERESIS
 #define _OLSR_HYSTERESIS
 
 #include "link_set.h"
 
-
-
-inline float
+float
 olsr_hyst_calc_stability(float);
 
-inline int
+int
 olsr_process_hysteresis(struct link_entry *);
 
 float
index d08185a..b0550ea 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: interfaces.c,v 1.29 2006/04/17 18:31:09 kattemat Exp $
+ * $Id: interfaces.c,v 1.30 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -72,7 +72,7 @@ static struct ifchgf *ifchgf_list;
  *@return the number of interfaces configured
  */
 int
-ifinit()
+ifinit(void)
 {
   struct olsr_if *tmp_if;
 
@@ -123,7 +123,7 @@ ifinit()
 
 
 olsr_u32_t
-get_if_property_id()
+get_if_property_id(void)
 {
   return if_property_id++;
 }
index b69ed70..a761e08 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: interfaces.h,v 1.38 2007/02/10 19:59:51 bernd67 Exp $
+ * $Id: interfaces.h,v 1.39 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -111,7 +111,7 @@ struct vtimes
  */
 struct olsr_netbuf
 {
-  olsr_u8_t *buff;/* Pointer to the allocated buffer */
+  olsr_u8_t *buff;     /* Pointer to the allocated buffer */
   int bufsize;    /* Size of the buffer */
   int maxsize;    /* Max bytes of payload that can be added to the buffer */
   int pending;    /* How much data is currently pending in the buffer */
index 63c393c..1fc404a 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.28 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: ipc_frontend.c,v 1.29 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /*
@@ -65,6 +65,16 @@ WinSockPError(char *);
 #define MSG_NOSIGNAL 0
 #endif
 
+static int ipc_sock = -1;
+static int ipc_conn = -1;
+
+static int
+ipc_send_all_routes(int fd);
+
+static int
+ipc_send_net_info(int fd);
+
+
 /**
  *Create the socket to use for IPC to the
  *GUI front-end
@@ -72,7 +82,7 @@ WinSockPError(char *);
  *@return the socket FD
  */
 int
-ipc_init()
+ipc_init(void)
 {
   //int flags;
   struct   sockaddr_in sin;
@@ -138,7 +148,7 @@ ipc_accept(int fd)
 
   addrlen = sizeof (struct sockaddr_in);
   
-  if ((ipc_connection = accept(ipc_sock, (struct sockaddr *)  &pin, &addrlen)) == -1)
+  if ((ipc_conn = accept(fd, (struct sockaddr *)  &pin, &addrlen)) == -1)
     {
       perror("IPC accept");
       olsr_exit("IPC accept", EXIT_FAILURE);
@@ -150,15 +160,15 @@ ipc_accept(int fd)
       if(ipc_check_allowed_ip((union olsr_ip_addr *)&pin.sin_addr.s_addr))
        {
          ipc_active = OLSR_TRUE;
-         ipc_send_net_info();
-         ipc_send_all_routes();
+         ipc_send_net_info(ipc_conn);
+         ipc_send_all_routes(ipc_conn);
          OLSR_PRINTF(1, "Connection from %s\n",addr)
        }
       else
        {
          OLSR_PRINTF(1, "Front end-connection from foregin host(%s) not allowed!\n", addr)
          olsr_syslog(OLSR_LOG_ERR, "OLSR: Front end-connection from foregin host(%s) not allowed!\n", addr);
-         close(ipc_connection);
+         CLOSE(ipc_conn);
        }
     }
 
@@ -200,7 +210,7 @@ ipc_check_allowed_ip(union olsr_ip_addr *addr)
  *@return 1
  */
 int
-ipc_input(int sock)
+ipc_input(int sock __attribute__((unused)))
 {
   /*
   union 
@@ -228,7 +238,7 @@ ipc_input(int sock)
  *@return negative on error
  */
 void
-frontend_msgparser(union olsr_message *msg, struct interface *in_if, union olsr_ip_addr *from_addr)
+frontend_msgparser(union olsr_message *msg, struct interface *in_if __attribute__((unused)), union olsr_ip_addr *from_addr __attribute__((unused)))
 {
   int size;
 
@@ -240,10 +250,10 @@ frontend_msgparser(union olsr_message *msg, struct interface *in_if, union olsr_
   else
     size = ntohs(msg->v6.olsr_msgsize);
   
-  if (send(ipc_connection, (void *)msg, size, MSG_NOSIGNAL) < 0) 
+  if (send(ipc_conn, (void *)msg, size, MSG_NOSIGNAL) < 0) 
     {
       OLSR_PRINTF(1, "(OUTPUT)IPC connection lost!\n")
-      close(ipc_connection);
+      CLOSE(ipc_conn);
       //olsr_cnf->open_ipc = 0;
       ipc_active = OLSR_FALSE;
       return;
@@ -308,10 +318,10 @@ ipc_route_send_rtentry(union olsr_ip_addr *dst, union olsr_ip_addr *gw, int met,
   printf("\n");
   */
   
-  if (send(ipc_connection, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
+  if (send(ipc_conn, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
     {
       OLSR_PRINTF(1, "(RT_ENTRY)IPC connection lost!\n")
-      close(ipc_connection);
+      CLOSE(ipc_conn);
       //olsr_cnf->open_ipc = 0;
       ipc_active = OLSR_FALSE;
       return -1;
@@ -322,8 +332,8 @@ ipc_route_send_rtentry(union olsr_ip_addr *dst, union olsr_ip_addr *gw, int met,
 
 
 
-int
-ipc_send_all_routes()
+static int
+ipc_send_all_routes(int fd)
 {
   struct rt_entry  *destination;
   struct interface *ifn;
@@ -370,10 +380,10 @@ ipc_send_all_routes()
 
          tmp = (char *) &packet;
   
-         if (send(ipc_connection, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
+         if (send(fd, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
            {
              OLSR_PRINTF(1, "(RT_ENTRY)IPC connection lost!\n")
-             close(ipc_connection);
+             CLOSE(ipc_conn);
              //olsr_cnf->open_ipc = 0;
              ipc_active = OLSR_FALSE;
              return -1;
@@ -415,10 +425,10 @@ ipc_send_all_routes()
 
          tmp = (char *) &packet;
   
-         if (send(ipc_connection, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
+         if (send(ipc_conn, tmp, IPC_PACK_SIZE, MSG_NOSIGNAL) < 0) // MSG_NOSIGNAL to avoid sigpipe
            {
              OLSR_PRINTF(1, "(RT_ENTRY)IPC connection lost!\n")
-             close(ipc_connection);
+             CLOSE(ipc_conn);
              //olsr_cnf->open_ipc = 0;
              ipc_active = OLSR_FALSE;
              return -1;
@@ -440,8 +450,8 @@ ipc_send_all_routes()
  *
  *@return negative on error
  */
-int
-ipc_send_net_info()
+static int
+ipc_send_net_info(int fd)
 {
   struct ipc_net_msg *net_msg;
   //int x, i;
@@ -518,10 +528,10 @@ ipc_send_net_info()
   */
 
 
-  if (send(ipc_connection, (char *)net_msg, sizeof(struct ipc_net_msg), MSG_NOSIGNAL) < 0) 
+  if (send(fd, (char *)net_msg, sizeof(struct ipc_net_msg), MSG_NOSIGNAL) < 0) 
     {
       OLSR_PRINTF(1, "(NETINFO)IPC connection lost!\n")
-      close(ipc_connection);
+      CLOSE(ipc_conn);
       //olsr_cnf->open_ipc = 0;
       return -1;
     }
@@ -533,11 +543,11 @@ ipc_send_net_info()
 
 
 int
-shutdown_ipc()
+shutdown_ipc(void)
 {
   OLSR_PRINTF(1, "Shutting down IPC...\n")
-  close(ipc_sock);
-  close(ipc_connection);
+  CLOSE(ipc_sock);
+  CLOSE(ipc_conn);
   
   return 1;
 }
index b870939..368c9a1 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: ipc_frontend.h,v 1.12 2004/11/21 11:28:56 kattemat Exp $
+ * $Id: ipc_frontend.h,v 1.13 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /*
@@ -98,8 +98,6 @@ struct ipc_net_msg
 };
 
 
-int ipc_connection;
-int ipc_sock;
 olsr_bool ipc_active;
 
 olsr_bool
@@ -108,9 +106,6 @@ ipc_check_allowed_ip(union olsr_ip_addr *);
 void
 ipc_accept(int);
 
-int
-ipc_send_all_routes(void);
-
 void
 frontend_msgparser(union olsr_message *, struct interface *, union olsr_ip_addr *);
 
index eb354f3..06e5888 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: link_set.c,v 1.66 2007/02/10 19:27:32 bernd67 Exp $
+ * $Id: link_set.c,v 1.67 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -79,19 +79,19 @@ get_neighbor_status(union olsr_ip_addr *);
 
 
 clock_t 
-get_hold_time_neighbor()
+get_hold_time_neighbor(void)
 {
   return hold_time_neighbor;
 }
 
 struct link_entry *
-get_link_set()
+get_link_set(void)
 {
   return link_set;
 }
 
 void
-olsr_init_link_set()
+olsr_init_link_set(void)
 {
 
   /* Timers */
@@ -819,7 +819,7 @@ check_link_status(struct hello_message *message, struct interface *in_if)
  *
  */
 static void
-olsr_time_out_link_set()
+olsr_time_out_link_set(void)
 {
 
   struct link_entry *tmp_link_set, *last_link_entry;
@@ -896,7 +896,7 @@ olsr_time_out_link_set()
  *@return nada
  */
 static void
-olsr_time_out_hysteresis()
+olsr_time_out_hysteresis(void)
 {
   struct link_entry *tmp_link_set;
 
@@ -1138,7 +1138,7 @@ void olsr_update_packet_loss(union olsr_ip_addr *rem, struct interface *loc,
   entry->loss_timeout = GET_TIMESTAMP(entry->loss_hello_int * 1500.0);
 }
 
-static void olsr_time_out_packet_loss()
+static void olsr_time_out_packet_loss(void)
 {
   struct link_entry *walker;
 
@@ -1167,7 +1167,7 @@ static void olsr_time_out_packet_loss()
     }
 }
 
-void olsr_update_dijkstra_link_qualities()
+void olsr_update_dijkstra_link_qualities(void)
 {
   struct link_entry *walker;
 
index 758d9ac..337fa69 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: apm.c,v 1.16 2006/11/26 00:32:14 bernd67 Exp $
+ * $Id: apm.c,v 1.17 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 /*
@@ -135,12 +135,13 @@ apm_init(void)
   else if(apm_read_apm(&ainfo))
     method = USE_APM;
   
-  if(method)
+  if(method != -1)
     apm_printinfo(&ainfo);
 
   return method;
 }
 
+
 void
 apm_printinfo(struct olsr_apm_info *ainfo)
 {
@@ -148,36 +149,25 @@ apm_printinfo(struct olsr_apm_info *ainfo)
              ainfo->ac_line_status,
              ainfo->battery_percentage,
              ainfo->battery_time_left)
-
-  return;
 }
 
 
-
 int
 apm_read(struct olsr_apm_info *ainfo)
 {
-
   switch(method)
     {
     case(USE_APM):
       return apm_read_apm(ainfo);
-      break;
     case(USE_ACPI):
       return apm_read_acpi(ainfo);
-      break;
     default:
-      return 0;
       break;
     }
-
-  return 1;
+  return 0;
 }
 
 
-
-
-
 static int
 apm_read_apm(struct olsr_apm_info *ainfo)
 {
@@ -190,14 +180,14 @@ apm_read_apm(struct olsr_apm_info *ainfo)
   if((apm_procfile = fopen(APM_PROC, "r")) == NULL)
     return 0;
 
-  if(fgets(buffer, sizeof(buffer) - 1, apm_procfile) == NULL)
+  if(fgets(buffer, sizeof(buffer), apm_procfile) == NULL)
     {
       fclose(apm_procfile);
       /* Try re-opening the file */
       if((apm_procfile = fopen(APM_PROC, "r")) != NULL)
        return 0;
 
-      if(fgets(buffer, sizeof(buffer) - 1, apm_procfile) == NULL)
+      if(fgets(buffer, sizeof(buffer), apm_procfile) == NULL)
        {
          /* Giving up */
          fprintf(stderr, "OLSRD: Could not read APM info - setting willingness to default");
@@ -205,8 +195,8 @@ apm_read_apm(struct olsr_apm_info *ainfo)
          return 0;
        }
     }
+  fclose(apm_procfile);
 
-  buffer[sizeof(buffer) - 1] = '\0';
 
   //printf("READ: %s\n", buffer);
 
@@ -245,18 +235,14 @@ apm_read_apm(struct olsr_apm_info *ainfo)
   ainfo->battery_percentage = lainfo.battery_percentage;
   ainfo->battery_time_left = lainfo.battery_time;
   
-  fclose(apm_procfile);
-
   return 1;
 }
 
 
-
 static int
 apm_read_acpi(struct olsr_apm_info *ainfo)
 {
   FILE *fd;
-  char s1[32], s2[32], s3[32], s4[32], inbuff[127];
   int bat_max = 5000; /* Find some sane value */
   int bat_val = 0;
   
@@ -271,22 +257,19 @@ apm_read_acpi(struct olsr_apm_info *ainfo)
       return 1;
     }
 
-
   /* Get maxvalue */
   if((fd = fopen(acpi_info[fd_index], "r")) == NULL) 
     return 0;
-
-  fgets(inbuff, 127, fd);
-  while(!feof(fd)) 
+  
+  for(;;) 
     {
-      sscanf(inbuff, "%s %s %s %s", s1, s2, s3, s4);
-       
+      char s1[32], s2[32], s3[32], s4[32], inbuff[127];
+      if (fgets(inbuff, sizeof(inbuff), fd) == NULL)
+        break;
+
+      sscanf(inbuff, "%s %s %s %s", s1, s2, s3, s4);   
       if (!strcasecmp(s2, "full")) 
-       {
          bat_max = atoi(s4);
-       }
-
-      fgets(inbuff, 127, fd);
     }
   fclose(fd);
 
@@ -295,23 +278,19 @@ apm_read_acpi(struct olsr_apm_info *ainfo)
     return 0;
 
   /* Extract battery status */
-  fgets(inbuff, 127, fd);
-  while(!feof(fd)) 
+  for(;;)
     {
+      char s1[32], s2[32], s3[32], s4[32], inbuff[127];
+      if (fgets(inbuff, sizeof(inbuff), fd) == NULL)
+        break;
       sscanf(inbuff, "%s %s %s %s", s1, s2, s3, s4);
 
       /* find remaining juice */
       if(!strcasecmp(s1, "Remaining")) 
-       {
          bat_val = atoi(s3);
-       }
-
-      fgets(inbuff, 127, fd);
     }
-
   fclose(fd);
 
-
   ainfo->ac_line_status = OLSR_BATTERY_POWERED;
   ainfo->battery_percentage = bat_val * 100 / bat_max;
 
@@ -319,57 +298,56 @@ apm_read_acpi(struct olsr_apm_info *ainfo)
 }
 
 
-
 static int
 acpi_probe(void)
 {
-  char s1[32], s2[32];
-  FILE *fd;
-  olsr_u16_t i;
+  unsigned int i;
   
-
   /* First check for AC power */
 
   for(i = 0; i < ACPI_AC_CNT; i++)
     {
+      char s1[32], s2[32];
+      int rc;
+      FILE *fd = fopen(acpi_ac[i], "r");
+  
       /* Try opening the info file */
-      if((fd = fopen(acpi_ac[i], "r")) == NULL)
+      if(fd == NULL)
        continue;
       
       /* Extract info */
-      if(fscanf(fd, "%s %s", s1, s2) < 2)
-       {
-         fclose(fd);
-         continue;
-       }
-      
+      rc = fscanf(fd, "%s %s", s1, s2);
+
       /* Close info entry */
       fclose(fd);
 
+      if(rc < 2)
+         continue;
+      
       /* Running on AC power */
       if(!strcasecmp(s2, "on-line"))
        return -1;
-
     }
 
-  /* Only checking the first found batery entry... */
+  /* Only checking the first found battery entry... */
   for(i = 0; i < ACPI_BT_CNT; i ++)
     {
-
+      char s1[32], s2[32];
+      int rc;
+      FILE *fd = fopen(acpi_info[i], "r");
+  
       /* Try opening the info file */
-      if((fd = fopen(acpi_info[i], "r")) == NULL)
+      if(fd == NULL)
        continue;
 
       /* Extract info */
-      if(fscanf(fd, "%s %s", s1, s2) < 2)
-       {
-         fclose(fd);
-         continue;
-       }
-
+      rc = fscanf(fd, "%s %s", s1, s2);
       /* Close info entry */
       fclose(fd);
 
+      if(rc < 2)
+         continue;
+
       /* Check if battery is present */
       if((!strcasecmp(s1, "present:")) && (!strcasecmp(s2, "no"))) 
        continue;
@@ -379,11 +357,9 @@ acpi_probe(void)
        continue;
 
       fclose(fd);
-
       return i;
     }
 
   /* No battery found */
   return -1;
 }
-
index d742877..c564a50 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.20 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: kernel_routes.c,v 1.21 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -346,10 +346,8 @@ olsr_ioctl_del_route6(struct rt_entry *destination)
 
 
 int
-delete_all_inet_gws()
-{
-  struct rtentry kernel_route;
-  
+delete_all_inet_gws(void)
+{  
   int s;
   char buf[BUFSIZ], *cp, *cplim;
   struct ifconf ifc;
@@ -375,10 +373,10 @@ delete_all_inet_gws()
     }
 
   ifr = ifc.ifc_req;
-#define size(p) (sizeof (p))
   cplim = buf + ifc.ifc_len; /*skip over if's with big ifr_addr's */
-  for (cp = buf; cp < cplim;cp += sizeof (ifr->ifr_name) + size(ifr->ifr_addr)) 
+  for (cp = buf; cp < cplim;cp += sizeof (ifr->ifr_name) + sizeof(ifr->ifr_addr)) 
     {
+      struct rtentry kernel_route;
       ifr = (struct ifreq *)cp;
       
       
index e3589b9..cd558c1 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net.c,v 1.32 2007/02/04 23:36:35 bernd67 Exp $
+ * $Id: net.c,v 1.33 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -78,7 +78,6 @@ bind_socket_to_device(int sock, char *dev_name)
    */
   OLSR_PRINTF(3, "Binding socket %d to device %s\n", sock, dev_name)
   return setsockopt(sock, SOL_SOCKET, SO_BINDTODEVICE, dev_name, strlen(dev_name)+1);
-
 }
 
 
@@ -98,20 +97,9 @@ int
 enable_ip_forwarding(int version)
 {
   FILE *proc_fwd;
-  char procfile[FILENAME_MAX];
-
-  if(version == AF_INET)
-    {
-      strncpy(procfile, "/proc/sys/net/ipv4/ip_forward", FILENAME_MAX);
-    }
-  else
-    if(version == AF_INET6)
-      {
-       strncpy(procfile, "/proc/sys/net/ipv6/conf/all/forwarding", FILENAME_MAX);
-      }
-    else
-      return -1;
-
+  const char * const procfile = version == AF_INET
+      ? "/proc/sys/net/ipv4/ip_forward"
+      : "/proc/sys/net/ipv6/conf/all/forwarding";
 
   if ((proc_fwd=fopen(procfile, "r"))==NULL)
     {
@@ -126,48 +114,40 @@ enable_ip_forwarding(int version)
       return 0;
     }
   
+  orig_fwd_state = fgetc(proc_fwd);
+  fclose(proc_fwd);
+  if(orig_fwd_state == '1')
+    {
+      OLSR_PRINTF(3, "\nIP forwarding is enabled on this system\n")
+    }
   else
     {
-      orig_fwd_state = fgetc(proc_fwd);
-      fclose(proc_fwd);
-      if(orig_fwd_state == '1')
-       {
-         OLSR_PRINTF(3, "\nIP forwarding is enabled on this system\n")
-       }
+      if ((proc_fwd=fopen(procfile, "w"))==NULL)
+        {
+          fprintf(stderr, "Could not open %s for writing!\n", procfile);
+          fprintf(stderr, "I will continue(in 3 sec) - but you should mannually ensure that IP forwarding is enabeled!\n\n");
+          sleep(3);
+          return 0;
+        }
       else
-       {
-         if ((proc_fwd=fopen(procfile, "w"))==NULL)
-           {
-             fprintf(stderr, "Could not open %s for writing!\n", procfile);
-             fprintf(stderr, "I will continue(in 3 sec) - but you should mannually ensure that IP forwarding is enabeled!\n\n");
-             sleep(3);
-             return 0;
-           }
-         else
-           {
-             syslog(LOG_INFO, "Writing \"1\" to %s\n", procfile);
-             fputs("1", proc_fwd);
-           }
-         fclose(proc_fwd);
-
-       }
+        {
+          syslog(LOG_INFO, "Writing \"1\" to %s\n", procfile);
+          fputs("1", proc_fwd);
+        }
+      fclose(proc_fwd);
     }
-  return 1;
-      
+  return 1;      
 }
 
-
 int
 disable_redirects_global(int version)
 {
   FILE *proc_redirect;
-  char procfile[FILENAME_MAX];
+  const char * const procfile = "/proc/sys/net/ipv4/conf/all/send_redirects";
 
   if(version == AF_INET6)
     return -1;
 
-  strcpy(procfile, "/proc/sys/net/ipv4/conf/all/send_redirects");
-
   if((proc_redirect = fopen(procfile, "r")) == NULL)
     {
       fprintf(stderr, "WARNING! Could not open the %s file to check/disable ICMP redirects!\nAre you using the procfile filesystem?\nDoes your system support IPv4?\nI will continue(in 3 sec) - but you should mannually ensure that ICMP redirects are disabled!\n\n", procfile);
@@ -175,16 +155,11 @@ disable_redirects_global(int version)
       sleep(3);
       return -1;
     }
-  else
-    {
-      orig_global_redirect_state = fgetc(proc_redirect);
-      fclose(proc_redirect);
-    }
+  orig_global_redirect_state = fgetc(proc_redirect);
+  fclose(proc_redirect);
 
   if(orig_global_redirect_state == '0')
-    {
       return 0;
-    }
 
   if ((proc_redirect = fopen(procfile, "w"))==NULL)
     {
@@ -193,17 +168,12 @@ disable_redirects_global(int version)
       sleep(3);
       return 0;
     }
-  else
-    {
-      syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
-      fputs("0", proc_redirect);
-    }
-  fclose(proc_redirect);
-  
+  syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
+  fputs("0", proc_redirect);
+  fclose(proc_redirect);  
   return 1;
 }
 
-
 /**
  *
  *@return 1 on sucess 0 on failiure
@@ -218,22 +188,17 @@ disable_redirects(const char *if_name, struct interface *iface, int version)
     return -1;
 
   /* Generate the procfile name */
-  sprintf(procfile, REDIRECT_PROC, if_name);
+  snprintf(procfile, sizeof(procfile), REDIRECT_PROC, if_name);
 
 
   if((proc_redirect = fopen(procfile, "r")) == NULL)
     {
-      fprintf(stderr, "WARNING! Could not open the %s file to check/disable ICMP redirects!\nAre you using the procfile filesystem?\nDoes your system support IPv4?\nI will continue(in 3 sec) - but you should mannually ensure that ICMP redirects are disabled!\n\n", procfile);
-      
+      fprintf(stderr, "WARNING! Could not open the %s file to check/disable ICMP redirects!\nAre you using the procfile filesystem?\nDoes your system support IPv4?\nI will continue(in 3 sec) - but you should mannually ensure that ICMP redirects are disabled!\n\n", procfile);      
       sleep(3);
       return 0;
     }
-  else
-    {
-      iface->nic_state.redirect = fgetc(proc_redirect);
-      fclose(proc_redirect);
-      
-    }
+  iface->nic_state.redirect = fgetc(proc_redirect);
+  fclose(proc_redirect);      
 
   if ((proc_redirect = fopen(procfile, "w"))==NULL)
     {
@@ -242,18 +207,12 @@ disable_redirects(const char *if_name, struct interface *iface, int version)
       sleep(3);
       return 0;
     }
-  else
-    {
-      syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
-      fputs("0", proc_redirect);
-    }
+  syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
+  fputs("0", proc_redirect);
   fclose(proc_redirect);
-
   return 1;
 }
 
-
-
 /**
  *
  *@return 1 on sucess 0 on failiure
@@ -267,11 +226,9 @@ deactivate_spoof(const char *if_name, struct interface *iface, int version)
   if(version == AF_INET6)
     return -1;
 
-
   /* Generate the procfile name */
   sprintf(procfile, SPOOF_PROC, if_name);
 
-
   if((proc_spoof = fopen(procfile, "r")) == NULL)
     {
       fprintf(stderr, "WARNING! Could not open the %s file to check/disable the IP spoof filter!\nAre you using the procfile filesystem?\nDoes your system support IPv4?\nI will continue(in 3 sec) - but you should mannually ensure that IP spoof filtering is disabled!\n\n", procfile);
@@ -279,12 +236,8 @@ deactivate_spoof(const char *if_name, struct interface *iface, int version)
       sleep(3);
       return 0;
     }
-  else
-    {
-      iface->nic_state.spoof = fgetc(proc_spoof);
-      fclose(proc_spoof);
-      
-    }
+  iface->nic_state.spoof = fgetc(proc_spoof);
+  fclose(proc_spoof);
 
   if ((proc_spoof = fopen(procfile, "w")) == NULL)
     {
@@ -293,27 +246,18 @@ deactivate_spoof(const char *if_name, struct interface *iface, int version)
       sleep(3);
       return 0;
     }
-  else
-    {
-      syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
-      fputs("0", proc_spoof);
-    }
+  syslog(LOG_INFO, "Writing \"0\" to %s", procfile);
+  fputs("0", proc_spoof);
   fclose(proc_spoof);
-
   return 1;
 }
 
-
-
 /**
  *Resets the spoof filter and ICMP redirect settings
  */
-
 int
 restore_settings(int version)
 {
-  FILE *proc_fd;
-  char procfile[FILENAME_MAX];
   struct interface *ifs;
 
   OLSR_PRINTF(1, "Restoring network state\n")
@@ -321,14 +265,10 @@ restore_settings(int version)
   /* Restore IP forwarding to "off" */
   if(orig_fwd_state == '0')
     {
-      if(version == AF_INET)
-       {
-         strcpy(procfile, "/proc/sys/net/ipv4/ip_forward");
-       }
-      else if(version == AF_INET6)
-       {
-         strcpy(procfile, "/proc/sys/net/ipv6/conf/all/forwarding");
-       }
+      const char * const procfile = version == AF_INET
+        ? "/proc/sys/net/ipv4/ip_forward"
+        : "/proc/sys/net/ipv6/conf/all/forwarding";
+      FILE *proc_fd;
 
       if ((proc_fd = fopen(procfile, "w")) == NULL)
        {
@@ -340,7 +280,6 @@ restore_settings(int version)
          fputc(orig_fwd_state, proc_fd);
          fclose(proc_fd);
        }
-
     }
 
   /* Restore global ICMP redirect setting */
@@ -348,7 +287,8 @@ restore_settings(int version)
     {
       if(version == AF_INET)
        {
-         strcpy(procfile, "/proc/sys/net/ipv4/conf/all/send_redirects");
+         const char * const procfile = "/proc/sys/net/ipv4/conf/all/send_redirects";
+          FILE *proc_fd;
 
          if ((proc_fd = fopen(procfile, "w")) == NULL)
            {
@@ -363,24 +303,23 @@ restore_settings(int version)
        }
     }
 
-
   if(version == AF_INET6)
     return 0;
 
   for(ifs = ifnet; ifs != NULL; ifs = ifs->int_next)
     {
+      char procfile[FILENAME_MAX];
+      FILE *proc_fd;
       /* Discard host-emulation interfaces */
       if(ifs->is_hcif)
        continue;
       /* ICMP redirects */
       
       /* Generate the procfile name */
-      sprintf(procfile, REDIRECT_PROC, ifs->int_name);
+      snprintf(procfile, sizeof(procfile), REDIRECT_PROC, ifs->int_name);
       
       if ((proc_fd = fopen(procfile, "w")) == NULL)
-       {
          fprintf(stderr, "Could not open %s for writing!\nSettings not restored!\n", procfile);
-       }
       else
        {
          syslog(LOG_INFO, "Resetting %s to %c\n", procfile, ifs->nic_state.redirect);
@@ -388,17 +327,13 @@ restore_settings(int version)
          fputc(ifs->nic_state.redirect, proc_fd);
          fclose(proc_fd);
        }
-
       
       /* Spoof filter */
       
       /* Generate the procfile name */
       sprintf(procfile, SPOOF_PROC, ifs->int_name);
-
       if ((proc_fd = fopen(procfile, "w")) == NULL)
-       {
          fprintf(stderr, "Could not open %s for writing!\nSettings not restored!\n", procfile);
-       }
       else
        {
          syslog(LOG_INFO, "Resetting %s to %c\n", procfile, ifs->nic_state.spoof);
@@ -406,13 +341,10 @@ restore_settings(int version)
          fputc(ifs->nic_state.spoof, proc_fd);
          fclose(proc_fd);
        }
-
     }
   return 1;
-
 }
 
-
 /**
  *Creates a nonblocking broadcast socket.
  *@param sa sockaddr struct. Used for bind(2).
@@ -424,20 +356,18 @@ gethemusocket(struct sockaddr_in *pin)
   int sock, on = 1;
 
   OLSR_PRINTF(1, "       Connecting to switch daemon port 10150...");
-
-
   if ((sock = socket(AF_INET, SOCK_STREAM, 0)) < 0) 
     {
       perror("hcsocket");
       syslog(LOG_ERR, "hcsocket: %m");
-      return (-1);
+      return -1;
     }
 
   if (setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on)) < 0) 
     {
       perror("SO_REUSEADDR failed");
       close(sock);
-      return (-1);
+      return -1;
     }
   /* connect to PORT on HOST */
   if (connect(sock,(struct sockaddr *) pin, sizeof(*pin)) < 0) 
@@ -446,13 +376,13 @@ gethemusocket(struct sockaddr_in *pin)
       fprintf(stderr, "Error connecting %d - %s\n", errno, strerror(errno));
       printf("connection refused\n");
       close(sock);
-      return (-1);
+      return -1;
     }
 
   printf("OK\n");
 
   /* Keep TCP socket blocking */  
-  return (sock);
+  return sock;
 }
 
 
@@ -467,24 +397,20 @@ getsocket(struct sockaddr *sa, int bufspace, char *int_name)
   struct sockaddr_in *sin=(struct sockaddr_in *)sa;
   int sock, on = 1;
 
-
-
   if ((sock = socket(AF_INET, SOCK_DGRAM, 0)) < 0) 
     {
       perror("socket");
       syslog(LOG_ERR, "socket: %m");
-      return (-1);
+      return -1;
     }
 
-
-
 #ifdef SO_BROADCAST
   if (setsockopt(sock, SOL_SOCKET, SO_BROADCAST, &on, sizeof (on)) < 0)
     {
       perror("setsockopt");
       syslog(LOG_ERR, "setsockopt SO_BROADCAST: %m");
       close(sock);
-      return (-1);
+      return -1;
     }
 #endif
 
@@ -492,17 +418,13 @@ getsocket(struct sockaddr *sa, int bufspace, char *int_name)
     {
       perror("SO_REUSEADDR failed");
       close(sock);
-      return (-1);
+      return -1;
     }
 
-
-
 #ifdef SO_RCVBUF
-
   for (on = bufspace; ; on -= 1024) 
     {
-      if (setsockopt(sock, SOL_SOCKET, SO_RCVBUF,
-                    &on, sizeof (on)) == 0)
+      if (setsockopt(sock, SOL_SOCKET, SO_RCVBUF, &on, sizeof (on)) == 0)
        break;
       if (on <= 8*1024) 
        {
@@ -511,11 +433,8 @@ getsocket(struct sockaddr *sa, int bufspace, char *int_name)
          break;
        }
     }
-
-
 #endif
 
-
   /*
    * WHEN USING KERNEL 2.6 THIS MUST HAPPEN PRIOR TO THE PORT BINDING!!!!
    */
@@ -529,23 +448,21 @@ getsocket(struct sockaddr *sa, int bufspace, char *int_name)
       return -1;
     }
 
-
   if (bind(sock, (struct sockaddr *)sin, sizeof (*sin)) < 0) 
     {
       perror("bind");
       syslog(LOG_ERR, "bind: %m");
       close(sock);
-      return (-1);
+      return -1;
     }
-
   /*
-   *One should probably fetch the flags first
+   *FIXME: One should probably fetch the flags first
    *using F_GETFL....
    */
   if (fcntl(sock, F_SETFL, O_NONBLOCK) == -1)
     syslog(LOG_ERR, "fcntl O_NONBLOCK: %m\n");
 
-  return (sock);
+  return sock;
 }
 
 
@@ -558,9 +475,6 @@ int
 getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
 {
   int sock, on = 1;
-
-
-
   if ((sock = socket(AF_INET6, SOCK_DGRAM, 0)) < 0) 
     {
       perror("socket");
@@ -589,14 +503,10 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
   */
   //#endif
 
-
-
-
 #ifdef SO_RCVBUF
   for (on = bufspace; ; on -= 1024) 
     {
-      if (setsockopt(sock, SOL_SOCKET, SO_RCVBUF,
-                    &on, sizeof (on)) == 0)
+      if (setsockopt(sock, SOL_SOCKET, SO_RCVBUF, &on, sizeof (on)) == 0)
        break;
       if (on <= 8*1024) 
        {
@@ -605,8 +515,6 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
          break;
        }
     }
-
-
 #endif
 
   if (setsockopt(sock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on)) < 0) 
@@ -616,7 +524,6 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
       return (-1);
     }
 
-
   /*
    * WHEN USING KERNEL 2.6 THIS MUST HAPPEN PRIOR TO THE PORT BINDING!!!!
    */
@@ -630,7 +537,6 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
       return -1;
     }
 
-
   if (bind(sock, (struct sockaddr *)sin, sizeof (*sin)) < 0) 
     {
       perror("bind");
@@ -638,7 +544,6 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
       close(sock);
       return (-1);
     }
-
   /*
    *One should probably fetch the flags first
    *using F_GETFL....
@@ -646,12 +551,9 @@ getsocket6(struct sockaddr_in6 *sin, int bufspace, char *int_name)
   if (fcntl(sock, F_SETFL, O_NONBLOCK) == -1)
     syslog(LOG_ERR, "fcntl O_NONBLOCK: %m\n");
 
-
-
-  return (sock);
+  return sock;
 }
 
-
 int
 join_mcast(struct interface *ifs, int sock)
 {
@@ -700,9 +602,8 @@ join_mcast(struct interface *ifs, int sock)
     {
       perror("Join multicast send");
       return -1;
-    }
+    }  
 
-  
   if(setsockopt(sock, 
                IPPROTO_IPV6, 
                IPV6_MULTICAST_IF, 
@@ -719,13 +620,10 @@ join_mcast(struct interface *ifs, int sock)
   return 0;
 }
 
-
-
 /*
  *From net-tools lib/interface.c
  *
  */
-
 int
 get_ipv6_address(char *ifname, struct sockaddr_in6 *saddr6, int scope_in)
 {
@@ -762,8 +660,7 @@ get_ipv6_address(char *ifname, struct sockaddr_in6 *saddr6, int scope_in)
            }
        }
       fclose(f);
-    }
-  
+    }  
   return 0;
 }
 
@@ -771,7 +668,6 @@ get_ipv6_address(char *ifname, struct sockaddr_in6 *saddr6, int scope_in)
 /**
  * Wrapper for sendto(2)
  */
-
 ssize_t
 olsr_sendto(int s, 
            const void *buf, 
@@ -822,7 +718,7 @@ olsr_select(int nfds,
 }
 
 int
-check_wireless_interface(char *ifname)
+check_wireless_interface(char * ifname)
 {
   struct ifreq ifr;
 
index d227652..a499b12 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: local_hna_set.c,v 1.12 2007/02/10 19:27:32 bernd67 Exp $
+ * $Id: local_hna_set.c,v 1.13 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -106,9 +106,9 @@ remove_local_hna4_entry(union olsr_ip_addr *net, union olsr_ip_addr *mask)
 int
 remove_local_hna6_entry(union olsr_ip_addr *net, olsr_u16_t prefix_len)
 {
-  struct hna6_entry *h6 = olsr_cnf->hna6_entries, *h6prev = NULL;
+  struct hna6_entry *h6, *h6prev = NULL;
 
-  while(h6)
+  for(h6 = olsr_cnf->hna6_entries; h6; h6 = h6->next)
     {
       if((memcmp(net, &h6->net, olsr_cnf->ipsize) == 0) && 
         (prefix_len == h6->prefix_len))
@@ -123,9 +123,7 @@ remove_local_hna6_entry(union olsr_ip_addr *net, olsr_u16_t prefix_len)
          return 1;
        }
       h6prev = h6;
-      h6 = h6->next;
     }
-
   return 0;
 }
 
@@ -171,19 +169,16 @@ find_local_hna6_entry(union olsr_ip_addr *net, olsr_u16_t prefix_len)
 
 
 int
-check_inet_gw()
+check_inet_gw(void)
 {
-  struct hna4_entry *h4 = olsr_cnf->hna4_entries;
-
   if(olsr_cnf->ip_version == AF_INET)
     {
-      while(h4)
+      struct hna4_entry *h4;
+      for(h4 = olsr_cnf->hna4_entries; h4; h4 = h4->next)
        {
          if(h4->netmask.v4 == 0 && h4->net.v4 == 0)
            return 1;
-         h4 = h4->next;
        }
-      return 0;
     }
   return 0;
 
index 22d146c..5b12e31 100644 (file)
--- a/src/log.h
+++ b/src/log.h
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: log.h,v 1.5 2004/11/21 11:28:56 kattemat Exp $
+ * $Id: log.h,v 1.6 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /* 
@@ -55,7 +55,7 @@ void
 olsr_openlog(const char *ident);
 
 void
-olsr_syslog(int level, char *format, ...);
+olsr_syslog(int level, char *format, ...) __attribute__((format(printf,2,3)));
 
 
 #endif
index de12203..063cea0 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: main.c,v 1.93 2007/04/20 10:38:01 bernd67 Exp $
+ * $Id: main.c,v 1.94 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include <unistd.h>
@@ -69,14 +69,14 @@ olsr_bool olsr_win32_end_request = OLSR_FALSE;
 olsr_bool olsr_win32_end_flag = OLSR_FALSE;
 #else
 static void
-olsr_shutdown(int);
+olsr_shutdown(int) __attribute__((noreturn));
 #endif
 
 /*
  * Local function prototypes
  */
 void
-olsr_reconfigure(int);
+olsr_reconfigure(int) __attribute__((noreturn));
 
 static void
 print_usage(void);
@@ -407,9 +407,11 @@ main(int argc, char *argv[])
   /* Starting scheduler */
   scheduler();
 
+  /* Stop the compiler from complaining */
+  (void)copyright_string;
+
   /* Like we're ever going to reach this ;-) */
   return 1;
-
 } /* main */
 
 
@@ -421,7 +423,7 @@ main(int argc, char *argv[])
  */
 #ifndef WIN32
 void
-olsr_reconfigure(int signal)
+olsr_reconfigure(int signal __attribute__((unused)))
 {
   if(!fork())
     {
@@ -502,18 +504,19 @@ olsr_shutdown(int signal)
  * Print the command line usage
  */
 static void
-print_usage()
+print_usage(void)
 {
 
-  fprintf(stderr, "An error occured somewhere between your keyboard and your chair!\n"); 
-  fprintf(stderr, "usage: olsrd [-f <configfile>] [ -i interface1 interface2 ... ]\n");
-  fprintf(stderr, "  [-d <debug_level>] [-ipv6] [-multi <IPv6 multicast address>]\n"); 
-  fprintf(stderr, "  [-lql <LQ level>] [-lqw <LQ winsize>]\n"); 
-  fprintf(stderr, "  [-bcast <broadcastaddr>] [-ipc] [-dispin] [-dispout] [-delgw]\n");
-  fprintf(stderr, "  [-hint <hello interval (secs)>] [-tcint <tc interval (secs)>]\n");
-  fprintf(stderr, "  [-midint <mid interval (secs)>] [-hnaint <hna interval (secs)>]\n");
-  fprintf(stderr, "  [-T <Polling Rate (secs)>] [-nofork] [-hemu <ip_address>] \n"); 
-  fprintf(stderr, "  [-lql <LQ level>] [-lqw <LQ winsize>]\n");
+  fprintf(stderr,
+          "An error occured somwhere between your keyboard and your chair!\n"
+          "usage: olsrd [-f <configfile>] [ -i interface1 interface2 ... ]\n"
+          "  [-d <debug_level>] [-ipv6] [-multi <IPv6 multicast address>]\n"
+          "  [-lql <LQ level>] [-lqw <LQ winsize>]\n"
+          "  [-bcast <broadcastaddr>] [-ipc] [-dispin] [-dispout] [-delgw]\n"
+          "  [-hint <hello interval (secs)>] [-tcint <tc interval (secs)>]\n"
+          "  [-midint <mid interval (secs)>] [-hnaint <hna interval (secs)>]\n"
+          "  [-T <Polling Rate (secs)>] [-nofork] [-hemu <ip_address>]\n"
+          "  [-lql <LQ level>] [-lqw <LQ winsize>]\n");
 }
 
 
index 5ba3049..b2c1bed 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: mid_set.c,v 1.17 2007/02/10 19:27:32 bernd67 Exp $
+ * $Id: mid_set.c,v 1.18 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -60,7 +60,7 @@ struct mid_entry *mid_lookup_entry_bymain(union olsr_ip_addr *adr);
  */
 
 int
-olsr_init_mid_set()
+olsr_init_mid_set(void)
 {
   int index;
 
@@ -461,7 +461,7 @@ olsr_prune_aliases(union olsr_ip_addr *m_addr, struct mid_alias *declared_aliase
  *@return nada
  */
 void
-olsr_time_out_mid_set(void *foo)
+olsr_time_out_mid_set(void *foo __attribute__((unused)))
 {
   int index;
 
@@ -527,9 +527,8 @@ mid_delete_node(struct mid_entry *entry)
  *For debuging purposes
  */
 void
-olsr_print_mid_set()
+olsr_print_mid_set(void)
 {
-
   int index;
 
   OLSR_PRINTF(1, "mid set: %02d:%02d:%02d.%06lu\n",nowtm->tm_hour, nowtm->tm_min, nowtm->tm_sec, now.tv_usec)
@@ -538,17 +537,14 @@ olsr_print_mid_set()
     {
       struct mid_entry *tmp_list = mid_set[index].next;
       /*Traverse MID list*/
-      for(tmp_list = mid_set[index].next;
-         tmp_list != &mid_set[index];
-         tmp_list = tmp_list->next)
+      for(tmp_list = mid_set[index].next; tmp_list != &mid_set[index]; tmp_list = tmp_list->next)
        {
-         struct mid_address *tmp_addr = tmp_list->aliases;
-
+         struct mid_address *tmp_addr;
+          
          OLSR_PRINTF(1, "%s: ", olsr_ip_to_string(&tmp_list->main_addr))
-         while(tmp_addr)
+          for(tmp_addr = tmp_list->aliases;tmp_addr;tmp_addr = tmp_addr->next_alias)
            {
              OLSR_PRINTF(1, " %s ", olsr_ip_to_string(&tmp_addr->alias))
-             tmp_addr = tmp_addr->next_alias;
            }
          OLSR_PRINTF(1, "\n")    
        }
index 2e689be..8e3d3e9 100644 (file)
--- a/src/mpr.c
+++ b/src/mpr.c
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: mpr.c,v 1.15 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: mpr.c,v 1.16 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -276,18 +276,18 @@ olsr_find_maximum_covered(int willingness)
  *Remove all MPR registrations
  */
 static void
-olsr_clear_mprs()
+olsr_clear_mprs(void)
 {
   olsr_u32_t index;
-  struct neighbor_entry   *a_neighbor;
-  struct neighbor_2_list_entry *two_hop_list;
   
   for (index=0;index<HASHSIZE;index++)
     {
+      struct neighbor_entry   *a_neighbor;
       for(a_neighbor = neighbortable[index].next;
          a_neighbor != &neighbortable[index];
          a_neighbor = a_neighbor->next)
        {
+          struct neighbor_2_list_entry *two_hop_list;
          /* Clear MPR selection */
          if(a_neighbor->is_mpr)
            {
@@ -304,7 +304,6 @@ olsr_clear_mprs()
            }
        }
     }
-
 }
 
 
@@ -314,16 +313,16 @@ olsr_clear_mprs()
  *@return 1 if changes occured 0 if not
  */
 static int
-olsr_check_mpr_changes()
+olsr_check_mpr_changes(void)
 {
   olsr_u32_t index;
-  struct neighbor_entry       *a_neighbor;
   int retval;
 
   retval = 0;
   
   for (index=0;index<HASHSIZE;index++)
     {
+      struct neighbor_entry *a_neighbor;
       for(a_neighbor = neighbortable[index].next;
          a_neighbor != &neighbortable[index];
          a_neighbor = a_neighbor->next)
@@ -346,13 +345,13 @@ olsr_check_mpr_changes()
  *on two hop neighbors
  */
 static void
-olsr_clear_two_hop_processed()
+olsr_clear_two_hop_processed(void)
 {
-  struct neighbor_2_entry  *neighbor_2;
   int index;
   
   for(index=0;index<HASHSIZE;index++)
     {
+      struct neighbor_2_entry  *neighbor_2;
       for(neighbor_2 = two_hop_neighbortable[index].next;
          neighbor_2 != &two_hop_neighbortable[index];
          neighbor_2 = neighbor_2->next)
@@ -369,32 +368,23 @@ olsr_clear_two_hop_processed()
  *This function calculates the number of two hop neighbors
  */
 static olsr_u16_t
-olsr_calculate_two_hop_neighbors()
+olsr_calculate_two_hop_neighbors(void)
 {
   olsr_u8_t                    index;
-  struct neighbor_entry        *a_neighbor, *dup_neighbor;
-  olsr_u16_t                   count, n_count, sum;
-  struct neighbor_2_list_entry *twohop_neighbors;
+  olsr_u16_t                   sum = 0;
   
-  //dup_neighbor = NULL;
-  
-  n_count = 0;
-  count = 0;
-  sum = 0;
-
   /* Clear 2 hop neighs */
   olsr_clear_two_hop_processed();
 
   for(index=0;index<HASHSIZE;index++)
     {
-      for(a_neighbor = neighbortable[index].next;
-         a_neighbor != &neighbortable[index];
-         a_neighbor = a_neighbor->next)
+      struct neighbor_entry *a_neighbor;
+      for(a_neighbor = neighbortable[index].next; a_neighbor != &neighbortable[index]; a_neighbor = a_neighbor->next)
        { 
-         count = 0;
-         n_count = 0;
-         
-         if(a_neighbor->status == NOT_SYM)
+          struct neighbor_2_list_entry *twohop_neighbors;
+          olsr_u16_t                   count = 0;
+          olsr_u16_t                   n_count = 0;
+          if(a_neighbor->status == NOT_SYM)
            {       
              a_neighbor->neighbor_2_nocov = count;
              continue;
@@ -405,8 +395,7 @@ olsr_calculate_two_hop_neighbors()
              twohop_neighbors = twohop_neighbors->next)
            {
              //printf("\tChecking %s....", olsr_ip_to_string(&twohop_neighbors->neighbor_2->neighbor_2_addr));
-             
-             dup_neighbor = olsr_lookup_neighbor_table(&twohop_neighbors->neighbor_2->neighbor_2_addr);
+              struct neighbor_entry *dup_neighbor = olsr_lookup_neighbor_table(&twohop_neighbors->neighbor_2->neighbor_2_addr);
              
              if((dup_neighbor == NULL) || (dup_neighbor->status != SYM))
                {
@@ -436,19 +425,16 @@ olsr_calculate_two_hop_neighbors()
  * Adds all nodes with willingness set to WILL_ALWAYS
  */
 static olsr_u16_t
-add_will_always_nodes()
+add_will_always_nodes(void)
 {
-
+  olsr_u16_t                   count = 0;
   olsr_u8_t                    index;
-  struct neighbor_entry        *a_neighbor;
-  olsr_u16_t                   count;
-
-  count = 0;
 
   //printf("\nAdding WILL ALWAYS nodes....\n");
 
   for(index=0;index<HASHSIZE;index++)
     {
+      struct neighbor_entry        *a_neighbor;
       for(a_neighbor = neighbortable[index].next;
          a_neighbor != &neighbortable[index];
          a_neighbor = a_neighbor->next)
@@ -472,22 +458,16 @@ add_will_always_nodes()
  *@return nada
  */
 void
-olsr_calculate_mpr()     
-{
-  
-  olsr_u16_t                   two_hop_covered_count=0;
-  olsr_u16_t                   two_hop_count=0;  
-  struct neighbor_2_list_entry *two_hop_list=NULL;
-  struct neighbor_2_list_entry *tmp;
-  struct neighbor_entry        *mprs; 
+olsr_calculate_mpr(void)     
+{  
+  olsr_u16_t                   two_hop_covered_count;
+  olsr_u16_t                   two_hop_count;
   int i;
 
   OLSR_PRINTF(3, "\n**RECALCULATING MPR**\n\n")
 
   olsr_clear_mprs();
-
   two_hop_count = olsr_calculate_two_hop_neighbors();
-
   two_hop_covered_count += add_will_always_nodes();
 
   /*
@@ -496,10 +476,12 @@ olsr_calculate_mpr()
 
   for(i = WILL_ALWAYS - 1; i > WILL_NEVER; i--)
     {
-      two_hop_list = olsr_find_2_hop_neighbors_with_1_link(i);
+      struct neighbor_entry        *mprs; 
+      struct neighbor_2_list_entry *two_hop_list = olsr_find_2_hop_neighbors_with_1_link(i);
 
       while(two_hop_list != NULL)
        {
+          struct neighbor_2_list_entry *tmp;
          //printf("CHOSEN FROM 1 LINK\n");
          if(!two_hop_list->neighbor_2->neighbor_2_nblist.next->neighbor->is_mpr)
            olsr_chosen_mpr(two_hop_list->neighbor_2->neighbor_2_nblist.next->neighbor, &two_hop_covered_count); 
@@ -557,20 +539,18 @@ olsr_calculate_mpr()
  *@return nada
  */
 static void
-olsr_optimize_mpr_set()
+olsr_optimize_mpr_set(void)
 {
-  int i, remove, index;
-  struct neighbor_2_list_entry *two_hop_list;
-  struct neighbor_entry *a_neighbor, *dup_neighbor;
+  int i, remove;
 
   //printf("\n**MPR OPTIMIZING**\n\n");
 
   for(i = WILL_NEVER + 1; i < WILL_ALWAYS; i++)
     {
-
+      int index;
       for(index=0;index<HASHSIZE;index++)
        {
-
+          struct neighbor_entry *a_neighbor;
          for(a_neighbor = neighbortable[index].next;
              a_neighbor != &neighbortable[index];
              a_neighbor = a_neighbor->next)
@@ -581,6 +561,7 @@ olsr_optimize_mpr_set()
              
              if(a_neighbor->is_mpr)
                {
+                  struct neighbor_2_list_entry *two_hop_list;
                  //printf("\tChecking %s\n", olsr_ip_to_string(&a_neighbor->neighbor_main_addr));
                  remove = 1;
 
@@ -589,7 +570,7 @@ olsr_optimize_mpr_set()
                      two_hop_list = two_hop_list->next)
                    {
                      
-                     dup_neighbor = olsr_lookup_neighbor_table(&two_hop_list->neighbor_2->neighbor_2_addr);
+                      const struct neighbor_entry * const dup_neighbor = olsr_lookup_neighbor_table(&two_hop_list->neighbor_2->neighbor_2_addr);
                      
                      if((dup_neighbor != NULL) && (dup_neighbor->status != NOT_SYM))
                        continue;
@@ -605,35 +586,28 @@ olsr_optimize_mpr_set()
                      OLSR_PRINTF(3, "MPR OPTIMIZE: removiong mpr %s\n\n", olsr_ip_to_string(&a_neighbor->neighbor_main_addr))
                      a_neighbor->is_mpr = OLSR_FALSE;
                    }
-
-
                }
            }
-         
        }
-
-
     }
-
 }
 
 
 
 void
-olsr_print_mpr_set()
+olsr_print_mpr_set(void)
 {
-  int                   index;
-  struct neighbor_entry *a_neighbor;
+  int index;
 
   OLSR_PRINTF(1, "MPR SET: ")
 
   for(index=0;index<HASHSIZE;index++)
     {
+      struct neighbor_entry *a_neighbor;
       for(a_neighbor = neighbortable[index].next;
          a_neighbor != &neighbortable[index];
          a_neighbor = a_neighbor->next)
        { 
-         
          /* 
           * Remove MPR settings
           */
@@ -641,7 +615,5 @@ olsr_print_mpr_set()
            OLSR_PRINTF(1, "[%s] ", olsr_ip_to_string(&a_neighbor->neighbor_main_addr))
        }
     }
-
   OLSR_PRINTF(1, "\n")
-
 }
index e540beb..6b148ff 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: mpr_selector_set.c,v 1.15 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: mpr_selector_set.c,v 1.16 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -55,7 +55,7 @@ static struct mpr_selector mprs_list;
  */
 
 int
-olsr_init_mprs_set()
+olsr_init_mprs_set(void)
 {
   OLSR_PRINTF(5, "MPRS: Init\n")
   /* Initial values */
@@ -65,20 +65,19 @@ olsr_init_mprs_set()
   
   mprs_list.next = &mprs_list;
   mprs_list.prev = &mprs_list;
-  
 
   return 1;
 }
 
 
 olsr_u16_t 
-get_local_ansn()
+get_local_ansn(void)
 {
   return ansn;
 }
 
 void
-increase_local_ansn()
+increase_local_ansn(void)
 {
   ansn++;
 }
@@ -88,7 +87,7 @@ increase_local_ansn()
  * neighbors. If the list is empty we are not MPR.
  */
 olsr_bool
-olsr_is_mpr()
+olsr_is_mpr(void)
 {
     return ((mprs_list.next == &mprs_list) ? OLSR_FALSE : OLSR_TRUE);
 }
@@ -195,25 +194,18 @@ olsr_update_mprs_set(union olsr_ip_addr *addr, float vtime)
   return retval;
 }
 
-
-
-
-
 /**
  *Time out MPR selector entries
  *
  *@return nada
  */
 void
-olsr_time_out_mprs_set()
+olsr_time_out_mprs_set(void)
 {
-  struct mpr_selector *mprs;
-
-  mprs = mprs_list.next;
+  struct mpr_selector *mprs = mprs_list.next;
 
   while(mprs != &mprs_list)
     {
-
       if(TIMED_OUT(mprs->MS_time))
        {
          /* Dequeue */
@@ -231,27 +223,20 @@ olsr_time_out_mprs_set()
       else
        mprs = mprs->next;
     }
-
 }
 
-
-
 /**
  *Print the current MPR selector set to STDOUT
  */
 void
-olsr_print_mprs_set()
+olsr_print_mprs_set(void)
 {
   struct mpr_selector *mprs;
-
-
-  mprs = mprs_list.next;
   OLSR_PRINTF(1, "MPR SELECTORS: ")
 
-  while(mprs != &mprs_list)
+    for(mprs = mprs_list.next; mprs != &mprs_list; mprs = mprs->next)
     {
       OLSR_PRINTF(1, "%s ", olsr_ip_to_string(&mprs->MS_main_addr))
-      mprs = mprs->next;
     }
   OLSR_PRINTF(1, "\n")
 }
index 43a8ce8..7792907 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: neighbor_table.c,v 1.30 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: neighbor_table.c,v 1.31 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -56,12 +56,11 @@ struct neighbor_entry neighbortable[HASHSIZE];
 
 
 void
-olsr_init_neighbor_table()
+olsr_init_neighbor_table(void)
 {
   int i;
 
   olsr_register_timeout_function(&olsr_time_out_neighborhood_tables);
-
   for(i = 0; i < HASHSIZE; i++)
     {
       neighbortable[i].next = &neighbortable[i];
@@ -437,31 +436,21 @@ olsr_time_out_two_hop_neighbors(struct neighbor_entry  *neighbor)
     }
 }
 
-
-
-
-
 void
-olsr_time_out_neighborhood_tables()
+olsr_time_out_neighborhood_tables(void)
 {
-  olsr_u8_t              index;
+  olsr_u8_t index;
   
   for(index=0;index<HASHSIZE;index++)
     {
-      struct neighbor_entry *entry = neighbortable[index].next;
-
-      while(entry != &neighbortable[index])
+      struct neighbor_entry *entry;
+      for(entry = neighbortable[index].next; entry != &neighbortable[index]; entry = entry->next)
        {         
          olsr_time_out_two_hop_neighbors(entry);
-         entry = entry->next;
        }
     }
 }
 
-
-
-
-
 /**
  *Prints the registered neighbors and two hop neighbors
  *to STDOUT.
@@ -469,7 +458,7 @@ olsr_time_out_neighborhood_tables()
  *@return nada
  */
 void
-olsr_print_neighbor_table()
+olsr_print_neighbor_table(void)
 {
   int i;
   char *fstr;
@@ -481,39 +470,34 @@ olsr_print_neighbor_table()
               (int)now.tv_usec/10000)
 
   if (olsr_cnf->ip_version == AF_INET)
-  {
-    OLSR_PRINTF(1, "IP address       LQ     NLQ    SYM   MPR   MPRS  will\n")
-    fstr = "%-15s  %5.3f  %5.3f  %s  %s  %s  %d\n";
-  }
-
+    {
+      OLSR_PRINTF(1, "IP address       LQ     NLQ    SYM   MPR   MPRS  will\n")
+      fstr = "%-15s  %5.3f  %5.3f  %s  %s  %s  %d\n";
+    }
   else
-  {
-    OLSR_PRINTF(1, "IP address                               LQ     NLQ    SYM   MPR   MPRS  will\n")
-    fstr = "%-39s  %5.3f  %5.3f  %s  %s  %s  %d\n";
-  }
+    {
+      OLSR_PRINTF(1, "IP address                               LQ     NLQ    SYM   MPR   MPRS  will\n")
+      fstr = "%-39s  %5.3f  %5.3f  %s  %s  %s  %d\n";
+    }
 
   for (i = 0; i < HASHSIZE; i++)
     {
       struct neighbor_entry *neigh;
-      for(neigh = neighbortable[i].next; neigh != &neighbortable[i];
-         neigh = neigh->next)
+      for(neigh = neighbortable[i].next; neigh != &neighbortable[i]; neigh = neigh->next)
        {
-         struct link_entry *link =
-            get_best_link_to_neighbor(&neigh->neighbor_main_addr);
-         double best_lq, inv_best_lq;
-
-         if(!link) 
-           continue;
-
-         best_lq = link->neigh_link_quality;
-         inv_best_lq = link->loss_link_quality;
-
-          OLSR_PRINTF(1, fstr, olsr_ip_to_string(&neigh->neighbor_main_addr),
-                      inv_best_lq, best_lq,
-                      (neigh->status == SYM) ? "YES " : "NO  ",
-                      neigh->is_mpr ? "YES " : "NO  ", 
-                     olsr_lookup_mprs_set(&neigh->neighbor_main_addr) == NULL ? "NO  " : "YES ",
-                      neigh->willingness)
+         struct link_entry *link = get_best_link_to_neighbor(&neigh->neighbor_main_addr);
+         if(link)
+            {
+              double best_lq = link->neigh_link_quality;
+              double inv_best_lq = link->loss_link_quality;
+
+              OLSR_PRINTF(1, fstr, olsr_ip_to_string(&neigh->neighbor_main_addr),
+                          inv_best_lq, best_lq,
+                          (neigh->status == SYM) ? "YES " : "NO  ",
+                          neigh->is_mpr ? "YES " : "NO  ", 
+                          olsr_lookup_mprs_set(&neigh->neighbor_main_addr) == NULL ? "NO  " : "YES ",
+                          neigh->willingness)
+            }
         }
     }
 }
index 50c65b0..3ac6646 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net_olsr.c,v 1.20 2007/04/20 10:38:01 bernd67 Exp $
+ * $Id: net_olsr.c,v 1.21 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "net_olsr.h"
@@ -77,8 +77,6 @@ struct ptf
 
 static struct ptf *ptf_list;
 
-static char ipv6_buf[100]; /* for address coversion */
-
 static struct deny_address_entry *deny_entries;
 
 static const char * const deny_ipv4_defaults[] =
@@ -104,12 +102,11 @@ net_set_disp_pack_out(olsr_bool val)
 void
 init_net(void)
 {
-  union olsr_ip_addr addr;
-  int i;
-
   /* Block invalid addresses */
   if(olsr_cnf->ip_version == AF_INET)
     {
+      union olsr_ip_addr addr;
+      int i;
       /* IPv4 */
       for(i = 0; deny_ipv4_defaults[i] != NULL; i++)
        {
@@ -124,6 +121,8 @@ init_net(void)
     }
   else 
     {
+      union olsr_ip_addr addr;
+      int i;
       /* IPv6 */
       for(i = 0; deny_ipv6_defaults[i] != NULL; i++)
        {
@@ -417,11 +416,9 @@ net_output(struct interface *ifp)
   /*
    *Call possible packet transform functions registered by plugins  
    */
-  tmp_ptf_list = ptf_list;
-  while(tmp_ptf_list != NULL)
+  for (tmp_ptf_list = ptf_list; tmp_ptf_list != NULL; tmp_ptf_list = tmp_ptf_list->next)
     {
       tmp_ptf_list->function(ifp->netbuf.buff, &ifp->netbuf.pending);
-      tmp_ptf_list = tmp_ptf_list->next;
     }
 
   /*
@@ -585,19 +582,15 @@ sockaddr_to_string(struct sockaddr *address_to_convert)
  *@return a char pointer to the string containing the IP
  */
 
-char *
-ip_to_string(olsr_u32_t *address)
+const char *
+ip_to_string(const olsr_u32_t *address)
 {
-
   struct in_addr in;
   in.s_addr=*address;
   return(inet_ntoa(in));
   
 }
 
-
-
-
 /**
  *Converts the 32bit olsr_u32_t datatype to
  *a char array.
@@ -608,36 +601,36 @@ ip_to_string(olsr_u32_t *address)
  *@return a char pointer to the string containing the IP
  */
 
-char *
-ip6_to_string(struct in6_addr *addr6)
+const char *
+ip6_to_string(const struct in6_addr *addr6)
 {
-  return (char *)inet_ntop(AF_INET6, addr6, ipv6_buf, sizeof(ipv6_buf));
+  static char ipv6_buf[INET6_ADDRSTRLEN]; /* for address coversion */
+  return inet_ntop(AF_INET6, addr6, ipv6_buf, sizeof(ipv6_buf));
 }
 
 
-char *
-olsr_ip_to_string(union olsr_ip_addr *addr)
+const char *
+olsr_ip_to_string(const union olsr_ip_addr *addr)
 {
   static int index = 0;
-  static char buff[4][100];
-  char *ret;
-  struct in_addr in;
+  static char buff[4][INET6_ADDRSTRLEN > INET_ADDRSTRLEN ? INET6_ADDRSTRLEN : INET_ADDRSTRLEN];
+  const char *ret;
   
   if(olsr_cnf->ip_version == AF_INET)
     {
-      in.s_addr=addr->v4;
-      ret = inet_ntoa(in);
+#if 0
+      struct in_addr in;
+      in.s_addr = addr->v4;
+      ret = inet_ntop(AF_INET, &in, buff[index], sizeof(buff[index]));
+#else
+      ret = inet_ntop(AF_INET, &addr->v6, buff[index], sizeof(buff[index]));
+#endif
     }
   else
     {
       /* IPv6 */
-      ret = (char *)inet_ntop(AF_INET6, &addr->v6, ipv6_buf, sizeof(ipv6_buf));
+      ret = inet_ntop(AF_INET6, &addr->v6, buff[index], sizeof(buff[index]));
     }
-
-  strncpy(buff[index], ret, 100);
-
-  ret = buff[index];
-
   index = (index + 1) & 3;
 
   return ret;
index 3138199..1607346 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net_olsr.h,v 1.7 2007/02/10 19:59:51 bernd67 Exp $
+ * $Id: net_olsr.h,v 1.8 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -93,14 +93,14 @@ olsr_netmask_to_prefix(union olsr_ip_addr *);
 char *
 sockaddr_to_string(struct sockaddr *);
 
-char *
-ip_to_string(olsr_u32_t *);
+const char *
+ip_to_string(const olsr_u32_t *);
 
-char *
-ip6_to_string(struct in6_addr *);
+const char *
+ip6_to_string(const struct in6_addr *);
 
-char *
-olsr_ip_to_string(union olsr_ip_addr *);
+const char *
+olsr_ip_to_string(const union olsr_ip_addr *);
 
 int
 add_ptf(packet_transform_function);
index 9af6461..76fb80b 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsr.c,v 1.50 2007/02/04 22:47:45 bernd67 Exp $
+ * $Id: olsr.c,v 1.51 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /**
@@ -88,7 +88,7 @@ static olsr_u16_t message_seqno;
  *Initialize the message sequence number as a random value
  */
 void
-init_msg_seqno()
+init_msg_seqno(void)
 {
   message_seqno = random() & 0xFFFF;
 }
@@ -98,8 +98,8 @@ init_msg_seqno()
  *
  *@return the seqno
  */
-inline olsr_u16_t
-get_msg_seqno()
+olsr_u16_t
+get_msg_seqno(void)
 {
   return message_seqno++;
 }
@@ -129,9 +129,8 @@ register_pcf(int (*f)(int, int, int))
  *@return 0
  */
 void
-olsr_process_changes()
+olsr_process_changes(void)
 {
-
   struct pcf *tmp_pc_list;
 
 #ifdef DEBUG
@@ -255,9 +254,8 @@ olsr_process_changes()
  *Also initalizes other variables
  */
 void
-olsr_init_tables()
-{
-  
+olsr_init_tables(void)
+{  
   changes_topology = OLSR_FALSE;
   changes_neighborhood = OLSR_FALSE;
   changes_hna = OLSR_FALSE;
@@ -290,15 +288,9 @@ olsr_init_tables()
   olsr_init_mid_set();
 
   /* Initialize HNA set */
-  olsr_init_hna_set();
-  
+  olsr_init_hna_set();  
 }
 
-
-
-
-
-
 /**
  *Check if a message is to be forwarded and forward
  *it if necessary.
@@ -331,11 +323,12 @@ olsr_forward_message(union olsr_message *m,
     }
 
   /* Lookup sender address */
-  if(!(src = mid_lookup_main_addr(from_addr)))
+  src = mid_lookup_main_addr(from_addr);
+  if(!src)
     src = from_addr;
 
-
-  if(NULL == (neighbor=olsr_lookup_neighbor_table(src)))
+  neighbor=olsr_lookup_neighbor_table(src);
+  if(!neighbor)
     return 0;
 
   if(neighbor->status != SYM)
@@ -354,7 +347,6 @@ olsr_forward_message(union olsr_message *m,
       return 0;
     }
 
-
   /* Treat TTL hopcnt */
   if(olsr_cnf->ip_version == AF_INET)
     {
@@ -369,14 +361,10 @@ olsr_forward_message(union olsr_message *m,
       m->v6.ttl--; 
     }
 
-
-
   /* Update dup forwarded */
   olsr_set_dup_forward(originator, seqno);
 
   /* Update packet data */
-
-
   msgsize = ntohs(m->v4.olsr_msgsize);
 
   /* looping trough interfaces */
@@ -399,10 +387,8 @@ olsr_forward_message(union olsr_message *m,
                  OLSR_PRINTF(1, "Received message to big to be forwarded in %s(%d bytes)!", ifn->int_name, msgsize)
                  olsr_syslog(OLSR_LOG_ERR, "Received message to big to be forwarded on %s(%d bytes)!", ifn->int_name, msgsize);
                }
-
            }
        }
-      
       else
        {
          /* No forwarding pending */
@@ -415,9 +401,7 @@ olsr_forward_message(union olsr_message *m,
            }
        }
     }
-
   return 1;
-
 }
 
 
@@ -431,13 +415,10 @@ set_buffer_timer(struct interface *ifn)
   jitter *= olsr_cnf->max_jitter;
 
   ifn->fwdtimer = GET_TIMESTAMP(jitter*1000);
-
 }
 
-
-
 void
-olsr_init_willingness()
+olsr_init_willingness(void)
 {
   if(olsr_cnf->willingness_auto)
     olsr_register_scheduler_event(&olsr_update_willingness, 
@@ -445,11 +426,9 @@ olsr_init_willingness()
 }
 
 void
-olsr_update_willingness(void *foo)
+olsr_update_willingness(void *foo __attribute__((unused)))
 {
-  int tmp_will;
-
-  tmp_will = olsr_cnf->willingness;
+  int tmp_will = olsr_cnf->willingness;
 
   /* Re-calculate willingness */
   olsr_cnf->willingness = olsr_calculate_willingness();
@@ -470,7 +449,7 @@ olsr_update_willingness(void *foo)
  */
 
 olsr_u8_t
-olsr_calculate_willingness()
+olsr_calculate_willingness(void)
 {
   struct olsr_apm_info ainfo;
 
@@ -624,17 +603,12 @@ olsr_malloc(size_t size, const char *id)
 int
 olsr_printf(int loglevel, char *format, ...)
 {
-  va_list arglist;
-
   if((loglevel <= olsr_cnf->debug_level) && debug_handle)
     {
+      va_list arglist;
       va_start(arglist, format);
-      
       vfprintf(debug_handle, format, arglist);
-      
       va_end(arglist);
     }
-
-
   return 0;
 }
index 245db62..9a5f5bb 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsr.h,v 1.25 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: olsr.h,v 1.26 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -60,7 +60,7 @@ olsr_process_changes(void);
 void
 init_msg_seqno(void);
 
-inline olsr_u16_t
+olsr_u16_t
 get_msg_seqno(void);
 
 int
@@ -101,6 +101,6 @@ void *
 olsr_malloc(size_t, const char *);
 
 int
-olsr_printf(int, char *, ...);
+olsr_printf(int, char *, ...) __attribute__((format(printf,2,3)));
 
 #endif
index 26dc749..d749374 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: olsr_types.h,v 1.5 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: olsr_types.h,v 1.6 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 /*
@@ -92,6 +92,9 @@ typedef int             olsr_32_t;
 
 union olsr_ip_addr
 {
+  /*
+  struct in_addr v4;
+  */
   olsr_u32_t v4;
   olsr_u8_t v4x[4];
   struct in6_addr v6;
index 3710a80..37e1e74 100644 (file)
@@ -29,7 +29,7 @@
  *
  */
 
-/* $Id: olsrd_plugin.h,v 1.1 2005/05/29 12:47:45 br1 Exp $ */
+/* $Id: olsrd_plugin.h,v 1.2 2007/04/20 13:46:04 bernd67 Exp $ */
 
 /*
  * Example plugin for olsrd.org OLSR daemon
  *                Functions that the plugin MUST provide                    *
  ****************************************************************************/
 
+#if 1
+/* We hide them from the compiler here to allow the plugins itself to declare them
+ * as they also implement them if we activate -Wredundant-decls.
+ * Normally we leave it seen so that we enforce a check by the compiler if they are
+ * identical.
+ */
+
 /**
  * Plugin interface version
  * Used by main olsrd to check plugin interface version
@@ -70,6 +77,6 @@ olsrd_plugin_register_param(char *key, char *value);
  */
 int
 olsrd_plugin_init(void);
-
+#endif
 
 #endif
index f6fe3d6..f14f418 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: parser.c,v 1.32 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: parser.c,v 1.33 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "parser.h"
@@ -87,7 +87,7 @@ parser_set_disp_pack_in(olsr_bool val)
  *@return nada
  */
 void
-olsr_init_parser()
+olsr_init_parser(void)
 {
   OLSR_PRINTF(3, "Initializing parser...\n")
 
index 4236d29..df24cdd 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: plugin_loader.c,v 1.23 2005/05/29 12:47:45 br1 Exp $
+ * $Id: plugin_loader.c,v 1.24 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "plugin_loader.h"
@@ -60,22 +60,17 @@ static struct olsr_plugin *olsr_plugins;
  *@return the number of plugins loaded
  */
 int
-olsr_load_plugins()
+olsr_load_plugins(void)
 {
-  struct plugin_entry *entry;
-  int loaded;
-
-  entry = olsr_cnf->plugins;
-  loaded = 0;
-
+  struct plugin_entry *entry = olsr_cnf->plugins;
+  int loaded = 0;
   OLSR_PRINTF(1, "Loading plugins...\n\n")
-
   while(entry)
     {  
       if(olsr_load_dl(entry->name, entry->params) < 0)
        OLSR_PRINTF(1, "-- PLUGIN LOADING FAILED! --\n\n")
       else
-       loaded ++;
+       loaded++;
 
       entry = entry->next;
     }
@@ -207,23 +202,20 @@ olsr_load_dl(char *libname, struct plugin_param *params)
 void
 init_olsr_plugin(struct olsr_plugin *entry)
 {
-  struct plugin_param *params = entry->params;
-  int retval;
-
   if(entry->register_param)
     {
+      struct plugin_param *params;
       OLSR_PRINTF(1, "Sending parameters...\n")
-      while(params)
+        for(params = entry->params;params;params = params->next)
         {
+          int retval = entry->register_param(params->key, params->value);
           OLSR_PRINTF(1, "\"%s\"/\"%s\"... ", params->key, params->value)
-          if((retval = entry->register_param(params->key, params->value)) < 0)
+          if(retval < 0)
             {
               fprintf(stderr, "\nFatal error in plugin parameter \"%s\"/\"%s\"\n", params->key, params->value);
               exit(EXIT_FAILURE);
             }
           OLSR_PRINTF(1, "%s\n", retval == 0 ? "FAILED" : "OK")
-
-          params = params->next;
         }
     }
     
@@ -236,15 +228,11 @@ init_olsr_plugin(struct olsr_plugin *entry)
  *Close all loaded plugins
  */
 void
-olsr_close_plugins()
+olsr_close_plugins(void)
 {
   struct olsr_plugin *entry;
 
   OLSR_PRINTF(1, "Closing plugins...\n")
-  for(entry = olsr_plugins; 
-      entry != NULL ; 
-      entry = entry->next)
-    {
+  for(entry = olsr_plugins; entry != NULL; entry = entry->next)
       dlclose(entry->dlhandle);
-    }
 }
index 2f07089..95c60e6 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: process_package.c,v 1.38 2007/02/10 19:27:32 bernd67 Exp $
+ * $Id: process_package.c,v 1.39 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -60,7 +60,7 @@
  *Initializing the parser functions we are using
  */
 void
-olsr_init_package_process()
+olsr_init_package_process(void)
 {
   if (olsr_cnf->lq_level == 0)
     {
@@ -81,21 +81,19 @@ void
 olsr_hello_tap(struct hello_message *message, struct interface *in_if,
                union olsr_ip_addr *from_addr)
 {
-  struct link_entry         *link;
   struct neighbor_entry     *neighbor;
-  struct hello_neighbor *walker;
-  double saved_lq;
-  double rel_lq;
 
   /*
    * Update link status
    */
-  link = update_link_entry(&in_if->ip_addr, from_addr, message, in_if);
+  struct link_entry         *link = update_link_entry(&in_if->ip_addr, from_addr, message, in_if);
 
   if (olsr_cnf->lq_level > 0)
     {
+      double saved_lq;
+      double rel_lq;
+      struct hello_neighbor *walker;
       // just in case our neighbor has changed its HELLO interval
-
       olsr_update_packet_loss_hello_int(link, message->htime);
 
       // find the input interface in the list of neighbor interfaces
index f5e36be..53af658 100644 (file)
@@ -39,7 +39,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: process_routes.c,v 1.30 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: process_routes.c,v 1.31 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include "defs.h"
@@ -148,7 +148,7 @@ olsr_delroute_remove_function(int (*function) (struct rt_entry*), olsr_u8_t type
 }
 
 void 
-olsr_init_export_route(
+olsr_init_export_route(void)
 {
   olsr_addroute_add_function(&olsr_ioctl_add_route, AF_INET);
   olsr_addroute_add_function(&olsr_ioctl_add_route6, AF_INET6);
@@ -211,7 +211,7 @@ olsr_export_del_route6 (struct rt_entry *e)
 
 
 int
-olsr_init_old_table()
+olsr_init_old_table(void)
 {
   int index;
 
@@ -238,14 +238,10 @@ olsr_init_old_table()
 int
 olsr_find_up_route(struct rt_entry *dst, struct rt_entry *table)
 { 
-  struct rt_entry *destination;
-  olsr_u32_t      hash;
-  hash = olsr_hashing(&dst->rt_dst);
+  struct rt_entry  *destination;
+  const olsr_u32_t  hash = olsr_hashing(&dst->rt_dst);
 
-  for(destination = table[hash].next;
-      destination != &table[hash];
-      destination = destination->next)
+  for(destination = table[hash].next; destination != &table[hash]; destination = destination->next)
     {
       //printf("Checking %s hc: %d ", olsr_ip_to_string(&dst->rt_dst), dst->rt_metric);
       //printf("vs %s hc: %d ... ", olsr_ip_to_string(&destination->rt_dst), destination->rt_metric);      
@@ -263,9 +259,7 @@ olsr_find_up_route(struct rt_entry *dst, struct rt_entry *table)
            }
        }
     }
-
   return 0;
-
 }
 
 
@@ -318,42 +312,31 @@ olsr_build_update_list(struct rt_entry *from_table,struct rt_entry *in_table)
  *@return 1
  */
 int
-olsr_delete_all_kernel_routes()
+olsr_delete_all_kernel_routes(void)
 { 
-  struct destination_n *delete_kernel_list = NULL;
-  struct destination_n *tmp = NULL;
-  union olsr_ip_addr *tmp_addr;
+  struct destination_n *delete_kernel_list;
+  struct destination_n *tmp;
 
   OLSR_PRINTF(1, "Deleting all routes...\n")
-
   delete_kernel_list = olsr_build_update_list(hna_routes, old_hna);
-
-  tmp = delete_kernel_list;
-
   OLSR_PRINTF(1, "HNA list:\n")
-  while(tmp)
+  for(tmp = delete_kernel_list;tmp;tmp = tmp->next)
     {
-      tmp_addr = &tmp->destination->rt_dst;
+      union olsr_ip_addr *tmp_addr = &tmp->destination->rt_dst;
       OLSR_PRINTF(1, "Dest: %s\n", olsr_ip_to_string(tmp_addr))
-      tmp = tmp->next;
     }
 
   olsr_delete_routes_from_kernel(delete_kernel_list);
 
   delete_kernel_list = olsr_build_update_list(routingtable,old_routes);
 
-  tmp = delete_kernel_list;
-
   OLSR_PRINTF(1, "Route list:\n")
-  while(tmp)
+  for(tmp = delete_kernel_list;tmp;tmp = tmp->next)
     {
-      tmp_addr = &tmp->destination->rt_dst;
-      OLSR_PRINTF(1, "Dest: %s\n", olsr_ip_to_string(tmp_addr))
-      tmp = tmp->next;
+      union olsr_ip_addr *tmp_addr = &tmp->destination->rt_dst;
+      OLSR_PRINTF(1, "Dest: %s\n", olsr_ip_to_string(tmp_addr))      
     }
-
   olsr_delete_routes_from_kernel(delete_kernel_list);
-
   return 1;
 }
 
@@ -365,10 +348,10 @@ olsr_delete_all_kernel_routes()
  *@return nada
  */
 void
-olsr_update_kernel_routes()
+olsr_update_kernel_routes(void)
 {
-  struct destination_n *delete_kernel_list = NULL;
-  struct destination_n *add_kernel_list = NULL;
+  struct destination_n *delete_kernel_list;
+  struct destination_n *add_kernel_list;
 
   OLSR_PRINTF(3, "Updating kernel routes...\n")
   delete_kernel_list = olsr_build_update_list(old_routes, routingtable);
@@ -387,15 +370,13 @@ olsr_update_kernel_routes()
  *@return nada
  */
 void
-olsr_update_kernel_hna_routes()
+olsr_update_kernel_hna_routes(void)
 {
-  struct destination_n *delete_kernel_list = NULL;
-  //struct destination_n *delete_kernel_list2;
-  struct destination_n *add_kernel_list = NULL;
+  struct destination_n *delete_kernel_list;
+  struct destination_n *add_kernel_list;
 
   OLSR_PRINTF(3, "Updating kernel HNA routes...\n")
 
-
   delete_kernel_list = olsr_build_update_list(old_hna, hna_routes);
   add_kernel_list = olsr_build_update_list(hna_routes, old_hna);
 
index b2c4ae8..5affeb8 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: routing_table.c,v 1.25 2007/02/10 19:27:32 bernd67 Exp $
+ * $Id: routing_table.c,v 1.26 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -48,6 +48,7 @@
 #include "neighbor_table.h"
 #include "olsr.h"
 #include "link_set.h"
+#include "routing_table.h"
 
 
 struct rt_entry routingtable[HASHSIZE];
@@ -82,10 +83,9 @@ two_hop_neighbor_reachable(struct neighbor_2_list_entry *);
  *Initialize the routing table
  */
 int
-olsr_init_routing_table()
+olsr_init_routing_table(void)
 {
   int index;
-
   /*
    *The hna routes hash will almost always
    *be indexed to 0
@@ -116,9 +116,7 @@ olsr_lookup_routing_table(union olsr_ip_addr *dst)
 {
 
   struct rt_entry *rt_table;
-  olsr_u32_t      hash;
-
-  hash = olsr_hashing(dst);
+  olsr_u32_t      hash = olsr_hashing(dst);
 
   for(rt_table = routingtable[hash].next;
       rt_table != &routingtable[hash];
@@ -126,10 +124,10 @@ olsr_lookup_routing_table(union olsr_ip_addr *dst)
     {
       if (COMP_IP(&rt_table->rt_dst, dst))
        {
-         return(rt_table);
+         return rt_table;
        }
     }
-  return(NULL);
+  return NULL;
   
 }
 
@@ -249,9 +247,9 @@ olsr_insert_routing_table(union olsr_ip_addr *dst,
  *@return
  */
 static int
-olsr_fill_routing_table_with_neighbors()
+olsr_fill_routing_table_with_neighbors(void)
 {
-  olsr_u8_t              index;
+  int index;
 
 #ifdef DEBUG
   OLSR_PRINTF(7, "FILL ROUTING TABLE WITH NEIGHBORS\n")
@@ -260,12 +258,10 @@ olsr_fill_routing_table_with_neighbors()
   for(index=0;index<HASHSIZE;index++)
     {
       struct neighbor_entry *neighbor;
-
       for(neighbor = neighbortable[index].next;
          neighbor != &neighbortable[index];
          neighbor=neighbor->next)     
        {
-
          if(neighbor->status == SYM)
            {
              static struct mid_address addrs;
@@ -277,9 +273,8 @@ olsr_fill_routing_table_with_neighbors()
 
              COPY_IP(&addrs.alias, &neighbor->neighbor_main_addr);
              addrs.next_alias = mid_lookup_aliases(&neighbor->neighbor_main_addr);
-             addrs2 = &addrs;
 
-             while(addrs2!=NULL)
+             for(addrs2 = &addrs;addrs2!=NULL;addrs2 = addrs2->next_alias)
                {
                  struct link_entry *link = get_best_link_to_neighbor(&addrs2->alias);
 #ifdef DEBUG
@@ -298,14 +293,10 @@ olsr_fill_routing_table_with_neighbors()
                                                    -1.0);
                        }
                    }
-             
-                 addrs2 = addrs2->next_alias;
                }
            }
        }
     }
-
-
   return 1;
 }
 
@@ -342,10 +333,10 @@ two_hop_neighbor_reachable(struct neighbor_2_list_entry *neigh_2_list)
  */
 
 static struct destination_n *
-olsr_fill_routing_table_with_two_hop_neighbors()
+olsr_fill_routing_table_with_two_hop_neighbors(void)
 {
   struct destination_n *list_destination_n=NULL;
-  olsr_u8_t            index;
+  int            index;
 
   //printf("FILL ROUTING TABLE WITH TWO HOP NEIGHBORS\n");
 
@@ -392,9 +383,8 @@ olsr_fill_routing_table_with_two_hop_neighbors()
 
              COPY_IP(&addrs.alias, n2_addr);
              addrs.next_alias = mid_lookup_aliases(n2_addr);
-             addrsp = &addrs;
 
-             while(addrsp!=NULL)
+             for(addrsp = &addrs; addrsp; addrsp = addrsp->next_alias)
                {
                  struct link_entry *link = get_best_link_to_neighbor(&neighbor->neighbor_main_addr);
 #ifdef DEBUG
@@ -402,7 +392,7 @@ olsr_fill_routing_table_with_two_hop_neighbors()
 #endif
                  if(link)
                    {
-                   struct interface *iface = link->if_name ? if_ifwithname(link->if_name) :
+                      struct interface *iface = link->if_name ? if_ifwithname(link->if_name) :
                                                if_ifwithaddr(&link->local_iface_addr);
                      if(iface)
                        {
@@ -415,47 +405,39 @@ olsr_fill_routing_table_with_two_hop_neighbors()
                          
                          if(new_route_entry != NULL)
                            {
-                             struct destination_n *list_destination_tmp;
-                             list_destination_tmp = olsr_malloc(sizeof(struct destination_n), 
-                                                                "Fill rt table 2 hop tmp");
-                             
-                             list_destination_tmp->destination = new_route_entry;
-                             list_destination_tmp->next = list_destination_n;
-                             list_destination_n = list_destination_tmp;
+                             struct destination_n *tmp = olsr_malloc(sizeof(struct destination_n), 
+                                                                      "Fill rt table 2 hop tmp");
+                             tmp->destination = new_route_entry;
+                             tmp->next = list_destination_n;
+                             list_destination_n = tmp;
                            }
                        }
                    }
-                 addrsp = addrsp->next_alias; 
                }
            }
-       }
-      
+       }      
     }
-  
   return list_destination_n;
 }
 
-
-
-
 /**
  *Recalculate the routing table
  *
  *@return nada
  */
 void 
-olsr_calculate_routing_table()
+olsr_calculate_routing_table(void)
 {
-  struct destination_n *list_destination_n_1 = NULL;
+  struct destination_n *list_destination_n_1;
 
   olsr_move_route_table(routingtable, old_routes);
 
   /* Add neighbors */
   olsr_fill_routing_table_with_neighbors();
   /* Add two hop enighbors - now they are the "outer rim" */
+  
   list_destination_n_1 = olsr_fill_routing_table_with_two_hop_neighbors();
-
-  while(list_destination_n_1!=NULL)
+  while(list_destination_n_1)
     {
       /* List_destination_n_1 holds the "outer rim" */
       struct destination_n *list_destination_n = list_destination_n_1;
@@ -643,9 +625,9 @@ olsr_check_for_lower_quality(struct rt_entry *routes, struct hna_net *net, float
  *
  */
 void
-olsr_calculate_hna_routes()
+olsr_calculate_hna_routes(void)
 {
-  olsr_u32_t index;
+  int index;
 
 #ifdef DEBUG
   OLSR_PRINTF(3, "Calculating HNA routes\n")
@@ -653,7 +635,6 @@ olsr_calculate_hna_routes()
 
   olsr_move_route_table(hna_routes, old_hna);
 
-  
   for(index=0;index<HASHSIZE;index++)
     {
       struct hna_entry *tmp_hna;
@@ -674,15 +655,11 @@ olsr_calculate_hna_routes()
 
              /* If no route to gateway - skip */
              if((tmp_rt = olsr_lookup_routing_table(&tmp_hna->A_gateway_addr)) == NULL)
-               {
                  continue;
-               }
 
              /* If there exists a better or equal entry - skip */
              if(olsr_check_for_higher_quality(hna_routes, tmp_net, tmp_rt->rt_etx) != NULL)
-               {
                  continue;
-               }
 
              /* If we find an entry with lower quality we just edit it */
              if((new_rt = olsr_check_for_lower_quality(hna_routes, tmp_net, tmp_rt->rt_etx)) != NULL)
@@ -744,7 +721,6 @@ olsr_calculate_hna_routes()
     }
 
   olsr_free_routing_table(old_hna);
-
 }
 
 
@@ -759,12 +735,11 @@ olsr_calculate_hna_routes()
 void
 olsr_print_routing_table(struct rt_entry *table)
 {
-
-  olsr_u8_t index;
+  int index;
 
   printf("ROUTING TABLE\n");
   printf("DESTINATION\tNEXT HOP\tHOPCNT\tINTERFACE\n");
-  for(index=0;index<HASHSIZE;index++)
+  for(index = 0; index < HASHSIZE; index++)
     {
       struct rt_entry *destination;
       for(destination = table[index].next;
@@ -779,8 +754,3 @@ olsr_print_routing_table(struct rt_entry *table)
        }
     }
 }
-
-
-
-
-
index 39dd064..2130206 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: scheduler.c,v 1.37 2007/01/31 12:36:50 bernd67 Exp $
+ * $Id: scheduler.c,v 1.38 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -77,7 +77,7 @@ signal_link_changes(olsr_bool val)
 }
 
 static void 
-trigger_dijkstra(void *dummy)
+trigger_dijkstra(void *foo __attribute__((unused)))
 {
   OLSR_PRINTF(3, "Triggering Dijkstra\n");
 
@@ -98,7 +98,7 @@ trigger_dijkstra(void *dummy)
  */
 
 void
-scheduler()
+scheduler(void)
 {
   struct timespec remainder_spec;
   struct timespec sleeptime_spec;
@@ -138,7 +138,6 @@ scheduler()
   /* Main scheduler event loop */
   for(;;)
     {
-
       /* Update now_times */
       now_times = times(&tms_buf);
 
@@ -314,7 +313,7 @@ olsr_register_scheduler_event(void (*event_function)(void *),
 
 /*
  *
- *@param initial how long utnil the first generation
+ *@param initial how long until the first generation
  *@param trigger pointer to a boolean indicating that
  *this function should be triggered immediatley
  */
@@ -322,7 +321,7 @@ int
 olsr_remove_scheduler_event(void (*event_function)(void *), 
                            void *par,
                            float interval, 
-                           float initial, 
+                           float initial __attribute__((unused))
                            olsr_u8_t *trigger)
 {
   struct event_entry *entry, *prev;
index 9f5432a..938f9ee 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: scheduler.h,v 1.12 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: scheduler.h,v 1.13 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -84,6 +84,6 @@ int
 olsr_remove_scheduler_event(void (*)(void *), void *, float, float, olsr_u8_t *);
 
 void
-scheduler(void);
+scheduler(void) __attribute__((noreturn));
 
 #endif
index 5da6fed..76d5a8b 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: socket_parser.c,v 1.23 2005/05/29 12:47:45 br1 Exp $
+ * $Id: socket_parser.c,v 1.24 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 #include <unistd.h>
@@ -157,11 +157,10 @@ remove_olsr_socket(int fd, void(*pf)(int))
 
 
 void
-poll_sockets()
+poll_sockets(void)
 {
   int n;
   struct olsr_socket_entry *olsr_sockets;
-  static struct tms tms_buf;
 
   /* If there are no registered sockets we
    * do not call select(2)
@@ -172,11 +171,10 @@ poll_sockets()
   FD_ZERO(&ibits);
   
   /* Adding file-descriptors to FD set */
-  olsr_sockets = olsr_socket_entries;
-  while(olsr_sockets)
+  
+  for(olsr_sockets = olsr_socket_entries; olsr_sockets; olsr_sockets = olsr_sockets->next)
     {
-      FD_SET(olsr_sockets->fd, &ibits);
-      olsr_sockets = olsr_sockets->next;
+      FD_SET(olsr_sockets->fd, &ibits);      
     }
       
   /* Runnig select on the FD set */
@@ -197,17 +195,12 @@ poll_sockets()
 
   /* Update time since this is much used by the parsing functions */
   gettimeofday(&now, NULL);      
-  now_times = times(&tms_buf);
+  now_times = times(NULL);
 
-  olsr_sockets = olsr_socket_entries;
-  while(olsr_sockets)
+  for(olsr_sockets = olsr_socket_entries;olsr_sockets;olsr_sockets = olsr_sockets->next)
     {
       if(FD_ISSET(olsr_sockets->fd, &ibits))
-       {
          olsr_sockets->process_function(olsr_sockets->fd);
-       }
-      olsr_sockets = olsr_sockets->next;
-    }
-       
+    }          
 }
 
index d034382..5cf7f84 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: tc_set.c,v 1.24 2006/01/07 08:16:20 kattemat Exp $
+ * $Id: tc_set.c,v 1.25 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -55,7 +55,7 @@ struct tc_entry tc_table[HASHSIZE];
  */
 
 int
-olsr_init_tc()
+olsr_init_tc(void)
 {
   int index;
  
@@ -364,11 +364,6 @@ olsr_tc_lookup_dst(struct tc_entry *entry, union olsr_ip_addr *dst_addr)
   return NULL;
 }
 
-
-
-
-
-
 /**
  * Time out entries
  *
@@ -376,28 +371,27 @@ olsr_tc_lookup_dst(struct tc_entry *entry, union olsr_ip_addr *dst_addr)
  */
 
 void
-olsr_time_out_tc_set()
+olsr_time_out_tc_set(void)
 {
-  int index, deleted;
-  struct tc_entry *entry, *entry2;
-  struct topo_dst *dst_entry, *dst_to_delete;
-
+  int index;
 
   for(index=0;index<HASHSIZE;index++)
     {
       /* For all TC entries */
-      entry = tc_table[index].next;
+      struct tc_entry *entry = tc_table[index].next;
       while(entry != &tc_table[index])
        {
+          struct tc_entry *entry2;
          //printf("INDEX: %d\n", index);
          /* For all destination entries of that TC entry */
-         deleted = 0;
-         dst_entry = entry->destinations.next;
+         int deleted = 0;
+         struct topo_dst *dst_entry = entry->destinations.next;
          while(dst_entry != &entry->destinations)
            {
              /* If timed out - delete */
              if(TIMED_OUT(dst_entry->T_time))
                {
+                  struct topo_dst *dst_to_delete;
                  deleted = 1;
                  /* Dequeue */
                  DEQUEUE_ELEM(dst_entry);
@@ -420,8 +414,6 @@ olsr_time_out_tc_set()
            olsr_tc_delete_entry_if_empty(entry2);
        }
     }
-
-  return;
 }
 
 
@@ -429,11 +421,9 @@ olsr_time_out_tc_set()
  *Print the topology table to stdout
  */
 int
-olsr_print_tc_table()
+olsr_print_tc_table(void)
 {
   int i;
-  struct tc_entry *entry;
-  struct topo_dst *dst_entry;
   char *fstr;
   float etx;
   
@@ -444,46 +434,36 @@ olsr_print_tc_table()
               (int)now.tv_usec / 10000)
 
   if (olsr_cnf->ip_version == AF_INET)
-  {
-    OLSR_PRINTF(1, "Source IP addr   Dest IP addr     LQ     ILQ    ETX\n")
-    fstr = "%-15s  %-15s  %5.3f  %5.3f  %.2f\n";
-  }
-
+    {
+      OLSR_PRINTF(1, "Source IP addr   Dest IP addr     LQ     ILQ    ETX\n")
+      fstr = "%-15s  %-15s  %5.3f  %5.3f  %.2f\n";
+    }
   else
-  {
-    OLSR_PRINTF(1, "Source IP addr                Dest IP addr                    LQ     ILQ    ETX\n")
-    fstr = "%-30s%-30s  %5.3f  %5.3f  %.2f\n";
-  }
+    {
+      OLSR_PRINTF(1, "Source IP addr                Dest IP addr                    LQ     ILQ    ETX\n")
+      fstr = "%-30s%-30s  %5.3f  %5.3f  %.2f\n";
+    }
 
   for (i = 0; i < HASHSIZE; i++)
-  {
-    entry = tc_table[i].next;
-
-    while (entry != &tc_table[i])
     {
-      dst_entry = entry->destinations.next;
-
-      while(dst_entry != &entry->destinations)
-      {
-        if (dst_entry->link_quality < MIN_LINK_QUALITY ||
-            dst_entry->inverse_link_quality < MIN_LINK_QUALITY)
-          etx = 0.0;
-
-        else
-          etx = 1.0 / (dst_entry->link_quality *
-                       dst_entry->inverse_link_quality);
-
-        OLSR_PRINTF(1, fstr, olsr_ip_to_string(&entry->T_last_addr),
-                    olsr_ip_to_string(&dst_entry->T_dest_addr),
-                    dst_entry->link_quality, dst_entry->inverse_link_quality,
-                    etx)
-
-        dst_entry = dst_entry->next;
-      }
-
-      entry = entry->next;
+      struct tc_entry *entry;
+      for(entry = tc_table[i].next;entry != &tc_table[i];entry = entry->next)
+        {
+          struct topo_dst *dst_entry;
+          for(dst_entry = entry->destinations.next;dst_entry != &entry->destinations;dst_entry = dst_entry->next)
+            {
+              if (dst_entry->link_quality < MIN_LINK_QUALITY ||
+                  dst_entry->inverse_link_quality < MIN_LINK_QUALITY)
+                etx = 0.0;
+              else
+                etx = 1.0 / (dst_entry->link_quality * dst_entry->inverse_link_quality);
+
+              OLSR_PRINTF(1, fstr, olsr_ip_to_string(&entry->T_last_addr),
+                          olsr_ip_to_string(&dst_entry->T_dest_addr),
+                          dst_entry->link_quality, dst_entry->inverse_link_quality,
+                          etx)                
+            }
+        }
     }
-  }
-
   return 1;
 }
index 4e6688c..10ae611 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: two_hop_neighbor_table.c,v 1.16 2005/05/29 12:47:46 br1 Exp $
+ * $Id: two_hop_neighbor_table.c,v 1.17 2007/04/20 13:46:04 bernd67 Exp $
  */
 
 
@@ -54,7 +54,7 @@ struct neighbor_2_entry two_hop_neighbortable[HASHSIZE];
  *Initialize 2 hop neighbor table
  */
 int
-olsr_init_two_hop_table()
+olsr_init_two_hop_table(void)
 {
   int index;
 
@@ -242,7 +242,7 @@ olsr_lookup_two_hop_neighbor_table_mid(union olsr_ip_addr *dest)
  *@return nada
  */
 void
-olsr_print_two_hop_neighbor_table()
+olsr_print_two_hop_neighbor_table(void)
 {
   int i;
 
index a5a4710..ed6e820 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: ifnet.c,v 1.46 2007/03/14 14:01:14 bernd67 Exp $
+ * $Id: ifnet.c,v 1.47 2007/04/20 13:46:05 bernd67 Exp $
  */
 
 
@@ -72,7 +72,7 @@ int bufspace = 127*1024;      /* max. input buffer size to request */
 
 
 int
-set_flag(char *ifname, short flag)
+set_flag(char *ifname, short flag __attribute__((unused)))
 {
   struct ifreq ifr;
 
@@ -108,7 +108,7 @@ set_flag(char *ifname, short flag)
 
 
 void
-check_interface_updates(void *foo)
+check_interface_updates(void *foo __attribute__((unused)))
 {
   struct olsr_if *tmp_if;