olsr_exit: use EXIT_SUCCESS or EXIT_FAILURE exclusively
authorFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 30 Nov 2015 11:28:28 +0000 (12:28 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 30 Nov 2015 13:04:48 +0000 (14:04 +0100)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
13 files changed:
lib/dyn_gw/src/olsrd_dyn_gw.c
lib/httpinfo/src/olsrd_httpinfo.c
lib/httpinfo/src/olsrd_plugin.c
lib/pgraph/src/olsrd_pgraph.c
lib/secure/src/olsrd_secure.c
src/linux/net.c
src/lq_plugin.c
src/main.c
src/olsr_cookie.c
src/plugin_loader.c
src/scheduler.c
src/win32/ifnet.c
src/win32/net.c

index 89d453d..44e7dbc 100644 (file)
@@ -581,7 +581,7 @@ add_to_ping_list(const char *ping_address, struct ping_list *the_ping_list)
 {
   struct ping_list *new = calloc(1, sizeof(struct ping_list));
   if (!new) {
-    olsr_exit("DYN GW: Out of memory", 0);
+    olsr_exit("DYN GW: Out of memory", EXIT_SUCCESS);
   }
   new->ping_address = strdup(ping_address);
   new->next = the_ping_list;
@@ -603,7 +603,7 @@ add_to_hna_list(struct hna_list *list_root, union olsr_ip_addr *hna_addr, uint8_
 {
   struct hna_list *new = calloc(1, sizeof(struct hna_list));
   if (new == NULL) {
-    olsr_exit("DYN GW: Out of memory", 0);
+    olsr_exit("DYN GW: Out of memory", EXIT_SUCCESS);
   }
 
   new->hna_addr.v4 = hna_addr->v4;
@@ -626,7 +626,7 @@ add_to_hna_group(struct hna_group *list_root)
 {
   struct hna_group *new = calloc(1, sizeof(struct hna_group));
   if (new == NULL) {
-    olsr_exit("DYN GW: Out of memory", 0);
+    olsr_exit("DYN GW: Out of memory", EXIT_SUCCESS);
   }
        
   new->next =  list_root;
index 043159a..de23338 100644 (file)
@@ -318,7 +318,7 @@ olsrd_plugin_init(void)
   http_socket = get_http_socket(http_port != 0 ? http_port : DEFAULT_TCP_PORT);
 
   if (http_socket < 0) {
-    olsr_exit("HTTPINFO: could not initialize HTTP socket", 1);
+    olsr_exit("HTTPINFO: could not initialize HTTP socket", EXIT_FAILURE);
   }
 
   /* Register socket */
index 71469d4..7c262d3 100644 (file)
@@ -127,12 +127,12 @@ add_plugin_access(const char *value, void *data, set_plugin_parameter_addon addo
   if (olsr_string_to_prefix(olsr_cnf->ip_version, &prefix, value)) {
     char buf[1024];
     snprintf(buf, sizeof(buf), "HTTPINFO: unknown access restriction parameter: %s", value);
-    olsr_exit(buf, 0);
+    olsr_exit(buf, EXIT_SUCCESS);
   }
 
   an = olsr_malloc(sizeof(*an), __func__);
   if (an == NULL) {
-    olsr_exit("HTTPINFO: register param net out of memory", 0);
+    olsr_exit("HTTPINFO: register param net out of memory", EXIT_SUCCESS);
   }
 
   an->prefix = prefix;
index acd9d72..672274a 100644 (file)
@@ -274,7 +274,7 @@ ipc_action(int fd __attribute__ ((unused)), void *data __attribute__ ((unused)),
   if ((ipc_connection = accept(ipc_socket, (struct sockaddr *)&pin, &addrlen)) == -1) {
     char buf2[1024];
     snprintf(buf2, sizeof(buf2), "(DOT DRAW)IPC accept error: %s", strerror(errno));
-    olsr_exit(buf2, 1);
+    olsr_exit(buf2, EXIT_FAILURE);
   } else {
     struct ipaddr_str main_addr;
     addr = inet_ntoa(pin.sin_addr);
index d7fd031..1d0226b 100644 (file)
@@ -181,12 +181,12 @@ secure_plugin_init(void)
   if (i < 0) {
     char buf[1024];
     snprintf(buf, sizeof(buf), "SECURE: Could not read key from file %s", keyfile);
-    olsr_exit(buf, 1);
+    olsr_exit(buf, EXIT_FAILURE);
   }
   if (i == 0) {
     char buf[1024];
     snprintf(buf, sizeof(buf), "SECURE: There was a problem reading key from file %s. Is the key long enough?", keyfile);
-    olsr_exit(buf, 1);
+    olsr_exit(buf, EXIT_FAILURE);
   }
 
   /* Register the packet transform function */
index 9642003..144139d 100644 (file)
@@ -225,7 +225,7 @@ net_os_set_global_ifoptions(void) {
     olsr_startup_sleep(3);
   }
   else if ((!olsr_cnf->set_ip_forward) && (orig_fwd_state != OLSRD_FORWARD_VALUE)) {
-    olsr_exit("IP forwarding not activated", 1);
+    olsr_exit("IP forwarding not activated", EXIT_FAILURE);
   }
 
   if (olsr_cnf->smart_gw_active) {
index 6d0f4d6..492431f 100644 (file)
@@ -87,7 +87,7 @@ activate_lq_handler(const char *name)
   if (node == NULL) {
     char buf[1024];
     snprintf(buf, sizeof(buf), "Error, unknown lq_handler '%s'", name);
-    olsr_exit(buf, 1);
+    olsr_exit(buf, EXIT_FAILURE);
   }
 
   OLSR_PRINTF(1, "Using '%s' algorithm for lq calculation.\n", name);
index 170528e..5bf0801 100644 (file)
@@ -128,7 +128,7 @@ static void olsr_reconfigure(int signo __attribute__ ((unused))) {
 #ifndef _WIN32
   errno = errNr;
 #endif
-  olsr_exit(NULL, 0);
+  olsr_exit(NULL, EXIT_SUCCESS);
 }
 #endif /* _WIN32 */
 
@@ -363,11 +363,11 @@ int main(int argc, char *argv[]) {
     if (!strcmp(argv[1], "-h") || !strcmp(argv[1], "/?")) {
       /* help */
       print_usage(false);
-      olsr_exit(NULL, 0);
+      olsr_exit(NULL, EXIT_SUCCESS);
     }
     if (!strcmp(argv[1], "-v")) {
       /* version */
-      olsr_exit(NULL, 0);
+      olsr_exit(NULL, EXIT_SUCCESS);
     }
   }
 
@@ -446,7 +446,7 @@ int main(int argc, char *argv[]) {
   if (olsr_cnf->ioctl_s < 0) {
     char buf2[1024];
     snprintf(buf2, sizeof(buf2), "ioctl socket: %s", strerror(errno));
-    olsr_exit(buf2, 0);
+    olsr_exit(buf2, EXIT_SUCCESS);
   }
 
   /* create a socket for netlink calls */
@@ -455,7 +455,7 @@ int main(int argc, char *argv[]) {
   if (olsr_cnf->rtnl_s < 0) {
     char buf2[1024];
     snprintf(buf2, sizeof(buf2), "rtnetlink socket: %s", strerror(errno));
-    olsr_exit(buf2, 0);
+    olsr_exit(buf2, EXIT_SUCCESS);
   }
 
   if (fcntl(olsr_cnf->rtnl_s, F_SETFL, O_NONBLOCK)) {
@@ -465,7 +465,7 @@ int main(int argc, char *argv[]) {
   if ((olsr_cnf->rt_monitor_socket = rtnetlink_register_socket(RTMGRP_LINK)) < 0) {
     char buf2[1024];
     snprintf(buf2, sizeof(buf2), "rtmonitor socket: %s", strerror(errno));
-    olsr_exit(buf2, 0);
+    olsr_exit(buf2, EXIT_SUCCESS);
   }
 #endif /* __linux__ */
 
@@ -475,14 +475,14 @@ int main(int argc, char *argv[]) {
   if (olsr_cnf->rts < 0) {
     char buf2[1024];
     snprintf(buf2, sizeof(buf2), "routing socket: %s", strerror(errno));
-    olsr_exit(buf2, 0);
+    olsr_exit(buf2, EXIT_SUCCESS);
   }
 #endif /* defined __FreeBSD__ || defined __FreeBSD_kernel__ || defined __APPLE__ || defined __NetBSD__ || defined __OpenBSD__ */
 
   /* initialise gateway system */
 #ifdef __linux__
   if (olsr_cnf->smart_gw_active && olsr_init_gateways()) {
-    olsr_exit("Cannot initialise gateway tunnels", 1);
+    olsr_exit("Cannot initialise gateway tunnels", EXIT_FAILURE);
   }
 #endif /* __linux__ */
 
@@ -547,13 +547,13 @@ int main(int argc, char *argv[]) {
 #ifdef __linux__
   /* startup gateway system */
   if (olsr_cnf->smart_gw_active && olsr_startup_gateways()) {
-    olsr_exit("Cannot startup gateway tunnels", 1);
+    olsr_exit("Cannot startup gateway tunnels", EXIT_FAILURE);
   }
 #endif /* __linux__ */
 
   /* initialise the IPC socket */
   if ((olsr_cnf->ipc_connections > 0) && ipc_init()) {
-    olsr_exit("ipc_init failure", 1);
+    olsr_exit("ipc_init failure", EXIT_FAILURE);
   }
 
   /* Initialisation of different tables to be used. */
index ac31dab..af35b95 100644 (file)
@@ -70,7 +70,7 @@ olsr_alloc_cookie(const char *cookie_name, olsr_cookie_type cookie_type)
   /* 1 <= ci_index <= COOKIE_ID_MAX */
 
   if (ci_index == COOKIE_ID_MAX) {
-    olsr_exit("No more cookies available", 1);
+    olsr_exit("No more cookies available", EXIT_FAILURE);
     return NULL;
   }
 
index ac4b085..e62ea26 100644 (file)
@@ -70,7 +70,7 @@ olsr_load_plugins(void)
     }
   }
   if (rv != 0) {
-    olsr_exit("-- PLUGIN LOADING FAILED --", 1);
+    olsr_exit("-- PLUGIN LOADING FAILED --", EXIT_FAILURE);
   }
   OLSR_PRINTF(0, "-- ALL PLUGINS LOADED! --\n\n");
 }
index 6f7c4b7..bfb96b9 100644 (file)
@@ -91,7 +91,7 @@ olsr_times(void)
   uint32_t t;
 
   if (gettimeofday(&tv, NULL) != 0) {
-    olsr_exit("OS clock is not working, have to shut down OLSR", 1);
+    olsr_exit("OS clock is not working, have to shut down OLSR", EXIT_FAILURE);
   }
 
   /* test if time jumped backward or more than 60 seconds forward */
@@ -578,7 +578,7 @@ olsr_init_timers(void)
 
   /* Grab initial timestamp */
   if (gettimeofday(&first_tv, NULL)) {
-    olsr_exit("OS clock is not working, have to shut down OLSR", 1);
+    olsr_exit("OS clock is not working, have to shut down OLSR", EXIT_FAILURE);
   }
   last_tv = first_tv;
   now_times = olsr_times();
index 9f6fd94..e5d9e18 100644 (file)
@@ -546,7 +546,7 @@ add_hemu_if(struct olsr_if *iface)
     ifp->olsr_socket = gethemusocket(&sin);
 
     if (ifp->olsr_socket < 0) {
-      olsr_exit("Could not initialize socket", 1);
+      olsr_exit("Could not initialize socket", EXIT_FAILURE);
     }
 
   } else {
@@ -841,7 +841,7 @@ chk_if_up(struct olsr_if *iface, int debuglvl __attribute__ ((unused)))
   New->send_socket = getsocket(0, New);
 
   if (New->olsr_socket < 0) {
-    olsr_exit("Could not initialize socket", 1);
+    olsr_exit("Could not initialize socket", EXIT_FAILURE);
   }
 
   add_olsr_socket(New->olsr_socket, &olsr_input, NULL, NULL, SP_PR_READ);
index 012fa4f..84d9459 100644 (file)
@@ -325,7 +325,7 @@ DisableIcmpRedirects(void)
   fprintf(stderr, "I have disabled ICMP redirect processing in the registry for you.\n");
   fprintf(stderr, "REBOOT NOW, so that these changes take effect. Exiting...\n\n");
 
-  olsr_exit(NULL, 0);
+  olsr_exit(NULL, EXIT_SUCCESS);
 }
 
 int