More descriptive messages
authorVasilis Tsiligiannis <b_tsiligiannis@silverton.gr>
Wed, 10 Mar 2010 08:20:55 +0000 (10:20 +0200)
committerVasilis Tsiligiannis <b_tsiligiannis@silverton.gr>
Wed, 10 Mar 2010 08:20:55 +0000 (10:20 +0200)
lib/quagga/src/client.c
lib/quagga/src/packet.c
lib/quagga/src/parse.c
lib/quagga/src/plugin.c
lib/quagga/src/quagga.c

index 2046dbc..f050d0f 100644 (file)
@@ -40,8 +40,8 @@ my_realloc(void *buf, size_t s, const char *c)
 
   buf = realloc(buf, s);
   if (!buf) {
-    OLSR_PRINTF(1, "(QUAGGA) OUT OF MEMORY: %s\n", strerror(errno));
-    olsr_syslog(OLSR_LOG_ERR, "olsrd: out of memory!: %m\n");
+    OLSR_PRINTF(1, "(QUAGGA) Out of memory: %s!\n", strerror(errno));
+    olsr_syslog(OLSR_LOG_ERR, "(QUAGGA) Out of memory!\n");
     olsr_exit(c, EXIT_FAILURE);
   }
 
@@ -127,7 +127,7 @@ zclient_write(unsigned char *options)
         ret = 0;
         continue;
       } else {
-        OLSR_PRINTF(1, "(QUAGGA) Disconnected from zebra\n");
+        OLSR_PRINTF(1, "(QUAGGA) Disconnected from zebra.\n");
         zebra.status &= ~STATUS_CONNECTED;
         /* TODO: Remove HNAs added from redistribution */
         free(options);
@@ -166,7 +166,7 @@ zclient_read(ssize_t * size)
     /* (re)allocate buffer */
     if (*size == bufsize) {
       bufsize += BUFSIZE;
-      buf = my_realloc(buf, bufsize, "Zebra try_read");
+      buf = my_realloc(buf, bufsize, "QUAGGA: Grow read buffer");
     }
 
     /* read from socket */
index 7a99cab..5a8eb13 100644 (file)
@@ -32,7 +32,7 @@ unsigned char
   uint32_t ind, metric;
   unsigned char *cmdopt, *t;
 
-  cmdopt = olsr_malloc(ZEBRA_MAX_PACKET_SIZ, "zebra add_v4_route");
+  cmdopt = olsr_malloc(ZEBRA_MAX_PACKET_SIZ, "QUAGGA: New route packet");
 
   t = &cmdopt[2];
   if (zebra.version) {
@@ -94,7 +94,7 @@ unsigned char
   unsigned char *data, *pnt;
   uint16_t size;
 
-  data = olsr_malloc(ZEBRA_MAX_PACKET_SIZ , "zebra_redistribute_packet");
+  data = olsr_malloc(ZEBRA_MAX_PACKET_SIZ , "QUAGGA: New redistribute packet");
 
   pnt = &data[2];
   if (zebra.version) {
index 6f7d5d8..5220849 100644 (file)
@@ -51,7 +51,7 @@ static struct zroute
   memcpy(&length, opt, sizeof length);
   length = ntohs (length);
 
-  r = olsr_malloc(sizeof *r, "zebra_parse_route");
+  r = olsr_malloc(sizeof *r, "QUAGGA: New zebra route");
   pnt = (zebra.version ? &opt[6] : &opt[3]);
   r->type = *pnt++;
   r->flags = *pnt++;
@@ -70,7 +70,7 @@ static struct zroute
     case 1:
       if (r->message & ZAPI_MESSAGE_NEXTHOP) {
         r->nexthop_num = *pnt++;
-        r->nexthop = olsr_malloc((sizeof *r->nexthop) * r->nexthop_num, "quagga: zebra_parse_route");
+        r->nexthop = olsr_malloc((sizeof *r->nexthop) * r->nexthop_num, "QUAGGA: New zebra route nexthop");
         for (c = 0; c < r->nexthop_num; c++) {
           if (olsr_cnf->ip_version == AF_INET) {
             memcpy(&r->nexthop[c].v4.s_addr, pnt, sizeof r->nexthop[c].v4.s_addr);
@@ -84,7 +84,7 @@ static struct zroute
 
       if (r->message & ZAPI_MESSAGE_IFINDEX) {
         r->ifindex_num = *pnt++;
-        r->ifindex = olsr_malloc(sizeof(uint32_t) * r->ifindex_num, "quagga: zebra_parse_route");
+        r->ifindex = olsr_malloc(sizeof(uint32_t) * r->ifindex_num, "QUAGGA: New zebra route ifindex");
         for (c = 0; c < r->ifindex_num; c++) {
           memcpy(&r->ifindex[c], pnt, sizeof r->ifindex[c]);
           r->ifindex[c] = ntohl (r->ifindex[c]);
@@ -110,7 +110,7 @@ static struct zroute
 //  }
 
   if (pnt - opt != length) {
-    olsr_exit("(QUAGGA) length does not match ??? ", EXIT_FAILURE);
+    olsr_exit("(QUAGGA) Length does not match!", EXIT_FAILURE);
   }
 
   return r;
@@ -136,7 +136,7 @@ zparse(void *foo __attribute__ ((unused)))
       memcpy(&length, f, sizeof length);
       length = ntohs (length);
       if (!length) // something weired happened
-        olsr_exit("(QUAGGA) Zero message length??? ", EXIT_FAILURE);
+        olsr_exit("(QUAGGA) Zero message length!", EXIT_FAILURE);
       if (zebra.version) {
         if ((f[2] != ZEBRA_HEADER_MARKER) || (f[3] != zebra.version))
           olsr_exit("(QUAGGA) Invalid zebra header received!", EXIT_FAILURE);
index 97a9edd..66454b9 100644 (file)
@@ -37,8 +37,8 @@ static void
 
   buf = realloc(buf, s);
   if (!buf) {
-    OLSR_PRINTF(1, "(QUAGGA) OUT OF MEMORY: %s\n", strerror(errno));
-    olsr_syslog(OLSR_LOG_ERR, "olsrd: out of memory!: %m\n");
+    OLSR_PRINTF(1, "(QUAGGA) Out of memory: %s!\n", strerror(errno));
+    olsr_syslog(OLSR_LOG_ERR, "(QUAGGA) Out of memory!\n");
     olsr_exit(c, EXIT_FAILURE);
   }
 
@@ -119,7 +119,7 @@ zplugin_sockpath(const char *value, void *data __attribute__ ((unused)), set_plu
   if (set_plugin_string(value, &sockpath, addon))
     return 1;
   len = strlen(sockpath) + 1;
-  zebra.sockpath = my_realloc(zebra.sockpath, len, "zebra_sockpath");
+  zebra.sockpath = my_realloc(zebra.sockpath, len, "QUAGGA: Grow socket path");
   memcpy(zebra.sockpath, sockpath, len);
 
   return 0;
index 8e5f891..04af3c7 100644 (file)
@@ -33,7 +33,7 @@ zebra_init(void)
 {
 
   memset(&zebra, 0, sizeof zebra);
-  zebra.sockpath = olsr_malloc(sizeof ZEBRA_SOCKPATH  + 1, "zebra_sockpath");
+  zebra.sockpath = olsr_malloc(sizeof ZEBRA_SOCKPATH  + 1, "QUAGGA: New socket path");
   strscpy(zebra.sockpath, ZEBRA_SOCKPATH, sizeof ZEBRA_SOCKPATH);
 
 }
@@ -81,11 +81,11 @@ zebra_addroute(const struct rt_entry *r)
     return 0;                  /* Quagga BUG workaround: don't add routes with destination = gateway
                                   see http://lists.olsr.org/pipermail/olsr-users/2006-June/001726.html */
     route.ifindex_num++;
-    route.ifindex = olsr_malloc(sizeof *route.ifindex, "zebra_add_route");
+    route.ifindex = olsr_malloc(sizeof *route.ifindex, "QUAGGA: New zebra route ifindex");
     *route.ifindex = r->rt_best->rtp_nexthop.iif_index;
   } else {
     route.nexthop_num++;
-    route.nexthop = olsr_malloc(sizeof *route.nexthop, "zebra_add_route");
+    route.nexthop = olsr_malloc(sizeof *route.nexthop, "QUAGGA: New zebra route nexthop");
     if (olsr_cnf->ip_version == AF_INET)
       route.nexthop->v4.s_addr = r->rt_best->rtp_nexthop.gateway.v4.s_addr;
     else
@@ -132,11 +132,11 @@ zebra_delroute(const struct rt_entry *r)
     return 0;                  /* Quagga BUG workaround: don't delete routes with destination = gateway
                                   see http://lists.olsr.org/pipermail/olsr-users/2006-June/001726.html */
     route.ifindex_num++;
-    route.ifindex = olsr_malloc(sizeof *route.ifindex, "zebra_del_route");
+    route.ifindex = olsr_malloc(sizeof *route.ifindex, "QUAGGA: New zebra route ifindex");
     *route.ifindex = r->rt_nexthop.iif_index;
   } else {
     route.nexthop_num++;
-    route.nexthop = olsr_malloc(sizeof *route.nexthop, "zebra_del_route");
+    route.nexthop = olsr_malloc(sizeof *route.nexthop, "QUAGGA: New zebra route nexthop");
     if (olsr_cnf->ip_version == AF_INET)
       route.nexthop->v4.s_addr = r->rt_nexthop.gateway.v4.s_addr;
     else
@@ -163,7 +163,7 @@ zebra_enable_redistribute(void)
   for (type = 0; type < ZEBRA_ROUTE_MAX; type++)
     if (zebra.redistribute[type]) {
       if (zclient_write(zpacket_redistribute(ZEBRA_REDISTRIBUTE_ADD, type)) < 0)
-        olsr_exit("(QUAGGA) could not send redistribute add command", EXIT_FAILURE);
+        olsr_exit("(QUAGGA) Could not write redistribute packet!", EXIT_FAILURE);
     }
 
 }
@@ -176,7 +176,7 @@ zebra_disable_redistribute(void)
   for (type = 0; type < ZEBRA_ROUTE_MAX; type++)
     if (zebra.redistribute[type]) {
       if (zclient_write(zpacket_redistribute(ZEBRA_REDISTRIBUTE_DELETE, type)) < 0)
-        olsr_exit("(QUAGGA) could not send redistribute delete command", EXIT_FAILURE);
+        olsr_exit("(QUAGGA) Could not write redistribute packet!", EXIT_FAILURE);
     }
 
 }