olsr_exit: fix some exit codes
authorFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 30 Nov 2015 11:49:07 +0000 (12:49 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 30 Nov 2015 13:04:48 +0000 (14:04 +0100)
EXIT_FAILURE is the correct code to return on failure (duh)

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
lib/dyn_gw/src/olsrd_dyn_gw.c
lib/httpinfo/src/olsrd_plugin.c
src/main.c

index 44e7dbc..21a8dfd 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", EXIT_SUCCESS);
+    olsr_exit("DYN GW: Out of memory", EXIT_FAILURE);
   }
   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", EXIT_SUCCESS);
+    olsr_exit("DYN GW: Out of memory", EXIT_FAILURE);
   }
 
   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", EXIT_SUCCESS);
+    olsr_exit("DYN GW: Out of memory", EXIT_FAILURE);
   }
        
   new->next =  list_root;
index 7c262d3..8bccb13 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, EXIT_SUCCESS);
+    olsr_exit(buf, EXIT_FAILURE);
   }
 
   an = olsr_malloc(sizeof(*an), __func__);
   if (an == NULL) {
-    olsr_exit("HTTPINFO: register param net out of memory", EXIT_SUCCESS);
+    olsr_exit("HTTPINFO: register param net out of memory", EXIT_FAILURE);
   }
 
   an->prefix = prefix;
index 5bf0801..0962ade 100644 (file)
@@ -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, EXIT_SUCCESS);
+    olsr_exit(buf2, EXIT_FAILURE);
   }
 
   /* 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, EXIT_SUCCESS);
+    olsr_exit(buf2, EXIT_FAILURE);
   }
 
   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, EXIT_SUCCESS);
+    olsr_exit(buf2, EXIT_FAILURE);
   }
 #endif /* __linux__ */
 
@@ -475,7 +475,7 @@ 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, EXIT_SUCCESS);
+    olsr_exit(buf2, EXIT_FAILURE);
   }
 #endif /* defined __FreeBSD__ || defined __FreeBSD_kernel__ || defined __APPLE__ || defined __NetBSD__ || defined __OpenBSD__ */