]> git.puffer.fish Git - mirror/frr.git/commitdiff
pimd: replace grp_str[100] with grp_str[INET_ADDRSTRLEN]
authorDaniel Walton <dwalton@cumulusnetworks.com>
Thu, 20 Oct 2016 14:09:30 +0000 (14:09 +0000)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Thu, 22 Dec 2016 01:26:11 +0000 (20:26 -0500)
Signed-off-by: Daniel Walton <dwalton@cumulusnetworks.com>
26 files changed:
pimd/pim_assert.c
pimd/pim_cmd.c
pimd/pim_hello.c
pimd/pim_iface.c
pimd/pim_ifchannel.c
pimd/pim_igmp.c
pimd/pim_igmpv2.c
pimd/pim_igmpv3.c
pimd/pim_join.c
pimd/pim_mroute.c
pimd/pim_msg.c
pimd/pim_neighbor.c
pimd/pim_oil.c
pimd/pim_pim.c
pimd/pim_register.c
pimd/pim_rp.c
pimd/pim_rpf.c
pimd/pim_sock.c
pimd/pim_ssmpingd.c
pimd/pim_static.c
pimd/pim_str.c
pimd/pim_tlv.c
pimd/pim_upstream.c
pimd/pim_vty.c
pimd/pim_zebra.c
pimd/pim_zlookup.c

index 481a2851d55c9976669e6c31cf8dc2981ffae4b6..fced5dcd476fb2c055d657eb8cb16ef06f36ea23 100644 (file)
@@ -64,8 +64,8 @@ void pim_ifassert_winner_set(struct pim_ifchannel     *ch,
     }
 
     if (winner_changed) {
-      char was_str[100];
-      char winner_str[100];
+      char was_str[INET_ADDRSTRLEN];
+      char winner_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<was?>", ch->ifassert_winner, was_str, sizeof(was_str));
       pim_inet4_dump("<winner?>", winner, winner_str, sizeof(winner_str));
       zlog_debug("%s: (S,G)=%s assert winner changed from %s to %s on interface %s",
@@ -91,7 +91,7 @@ static void on_trace(const char *label,
                     struct interface *ifp, struct in_addr src)
 {
   if (PIM_DEBUG_PIM_TRACE) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src, src_str, sizeof(src_str));
     zlog_debug("%s: from %s on %s",
               label, src_str, ifp->name);
@@ -243,7 +243,7 @@ int pim_assert_recv(struct interface *ifp,
    */
   offset = pim_parse_addr_group (&msg_group_addr, curr, curr_size);
   if (offset < 1) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: pim_parse_addr_group() failure: from %s on %s",
              __PRETTY_FUNCTION__,
@@ -258,7 +258,7 @@ int pim_assert_recv(struct interface *ifp,
   */
   offset = pim_parse_addr_ucast (&msg_source_addr, curr, curr_size);
   if (offset < 1) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: pim_parse_addr_ucast() failure: from %s on %s",
              __PRETTY_FUNCTION__,
@@ -269,7 +269,7 @@ int pim_assert_recv(struct interface *ifp,
   curr_size -= offset;
 
   if (curr_size != 8) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: preference/metric size is not 8: size=%d from %s on interface %s",
              __PRETTY_FUNCTION__,
@@ -296,9 +296,9 @@ int pim_assert_recv(struct interface *ifp,
   msg_metric.route_metric = pim_read_uint32_host(curr);
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char neigh_str[100];
-    char source_str[100];
-    char group_str[100];
+    char neigh_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<neigh?>", src_addr, neigh_str, sizeof(neigh_str));
     pim_inet4_dump("<src?>", msg_source_addr.u.prefix4, source_str, sizeof(source_str));
     pim_inet4_dump("<grp?>", msg_group_addr.u.prefix4, group_str, sizeof(group_str));
@@ -384,7 +384,7 @@ int pim_assert_build_msg(uint8_t *pim_msg, int buf_size,
                                                remain,
                                                group_addr);
   if (!pim_msg_curr) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     zlog_warn("%s: failure encoding group address %s: space left=%d",
              __PRETTY_FUNCTION__, group_str, remain);
@@ -397,7 +397,7 @@ int pim_assert_build_msg(uint8_t *pim_msg, int buf_size,
                                                remain,
                                                source_addr);
   if (!pim_msg_curr) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: failure encoding source address %s: space left=%d",
              __PRETTY_FUNCTION__, source_str, remain);
index c6390271f19fc1c95f88556dde89347a9563c920..dbe8c3ef427f232591bfad0128e4b2cc812bfa72 100644 (file)
@@ -184,9 +184,9 @@ static void pim_show_assert(struct vty *vty)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
-      char winner_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
+      char winner_str[INET_ADDRSTRLEN];
       char uptime[10];
       char timer[10];
 
@@ -245,8 +245,8 @@ static void pim_show_assert_internal(struct vty *vty)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
 
       pim_inet4_dump("<ch_src?>", ch->sg.src,
                     ch_src_str, sizeof(ch_src_str));
@@ -289,9 +289,9 @@ static void pim_show_assert_metric(struct vty *vty)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
-      char addr_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
+      char addr_str[INET_ADDRSTRLEN];
       struct pim_assert_metric am;
 
       am = pim_macro_spt_assert_metric(&ch->upstream->rpf, pim_ifp->primary_address);
@@ -340,9 +340,9 @@ static void pim_show_assert_winner_metric(struct vty *vty)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
-      char addr_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
+      char addr_str[INET_ADDRSTRLEN];
       struct pim_assert_metric *am;
       char pref_str[5];
       char metr_str[7];
@@ -438,8 +438,8 @@ static void pim_show_membership(struct vty *vty, u_char uj)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
 
       pim_inet4_dump("<ch_src?>", ch->sg.src,
                     ch_src_str, sizeof(ch_src_str));
@@ -708,7 +708,7 @@ static void igmp_show_interface_join(struct vty *vty)
     struct listnode *join_node;
     struct igmp_join *ij;
     struct in_addr pri_addr;
-    char pri_addr_str[100];
+    char pri_addr_str[INET_ADDRSTRLEN];
 
     pim_ifp = ifp->info;
     
@@ -722,8 +722,8 @@ static void igmp_show_interface_join(struct vty *vty)
     pim_inet4_dump("<pri?>", pri_addr, pri_addr_str, sizeof(pri_addr_str));
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_join_list, join_node, ij)) {
-      char group_str[100];
-      char source_str[100];
+      char group_str[INET_ADDRSTRLEN];
+      char source_str[INET_ADDRSTRLEN];
       char uptime[10];
 
       pim_time_uptime(uptime, sizeof(uptime), now - ij->sock_creation);
@@ -755,14 +755,14 @@ static void pim_show_interfaces_single(struct vty *vty, const char *ifname, u_ch
   struct pim_neighbor *neigh;
   struct pim_upstream *up;
   time_t now;
-  char dr_str[100];
+  char dr_str[INET_ADDRSTRLEN];
   char dr_uptime[10];
   char expire[10];
-  char grp_str[100];
+  char grp_str[INET_ADDRSTRLEN];
   char hello_period[10];
   char hello_timer[10];
-  char neigh_src_str[100];
-  char src_str[100];
+  char neigh_src_str[INET_ADDRSTRLEN];
+  char src_str[INET_ADDRSTRLEN];
   char stat_uptime[10];
   char uptime[10];
   int mloop;
@@ -1096,8 +1096,8 @@ static void pim_show_join(struct vty *vty, u_char uj)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_ifchannel_list, ch_node, ch)) {
-      char ch_src_str[100];
-      char ch_grp_str[100];
+      char ch_src_str[INET_ADDRSTRLEN];
+      char ch_grp_str[INET_ADDRSTRLEN];
       char uptime[10];
       char expire[10];
       char prune[10];
@@ -1169,7 +1169,7 @@ static void pim_show_neighbors_single(struct vty *vty, const char *neighbor, u_c
   int option_t_bit;
   char uptime[10];
   char expire[10];
-  char neigh_src_str[100];
+  char neigh_src_str[INET_ADDRSTRLEN];
 
   json_object *json = NULL;
   json_object *json_ifp = NULL;
@@ -1322,8 +1322,8 @@ pim_show_state(struct vty *vty, const char *src_or_group, const char *group, u_c
   }
 
   for (ALL_LIST_ELEMENTS_RO(pim_channel_oil_list, node, c_oil)) {
-    char grp_str[100];
-    char src_str[100];
+    char grp_str[INET_ADDRSTRLEN];
+    char src_str[INET_ADDRSTRLEN];
     char in_ifname[16];
     char out_ifname[16];
     int oif_vif_index;
@@ -1442,7 +1442,7 @@ static void pim_show_neighbors(struct vty *vty, u_char uj)
   time_t now;
   char uptime[10];
   char expire[10];
-  char neigh_src_str[100];
+  char neigh_src_str[INET_ADDRSTRLEN];
   json_object *json = NULL;
   json_object *json_ifp_rows = NULL;
   json_object *json_row = NULL;
@@ -1530,7 +1530,7 @@ static void pim_show_neighbors_secondary(struct vty *vty)
     ifaddr = pim_ifp->primary_address;
 
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->pim_neighbor_list, neighnode, neigh)) {
-      char neigh_src_str[100];
+      char neigh_src_str[INET_ADDRSTRLEN];
       struct listnode *prefix_node;
       struct prefix *p;
 
@@ -1541,7 +1541,7 @@ static void pim_show_neighbors_secondary(struct vty *vty)
                     neigh_src_str, sizeof(neigh_src_str));
 
       for (ALL_LIST_ELEMENTS_RO(neigh->prefix_list, prefix_node, p)) {
-       char neigh_sec_str[100];
+       char neigh_sec_str[INET_ADDRSTRLEN];
 
        if (p->family != AF_INET)
          continue;
@@ -1599,8 +1599,8 @@ static void pim_show_upstream(struct vty *vty, u_char uj)
     vty_out(vty, "Iif       Source          Group           State       Uptime   JoinTimer RSTimer   KATimer   RefCnt%s", VTY_NEWLINE);
 
   for (ALL_LIST_ELEMENTS_RO(pim_upstream_list, upnode, up)) {
-    char src_str[100];
-    char grp_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
     char uptime[10];
     char join_timer[10];
     char rs_timer[10];
@@ -1661,8 +1661,8 @@ static void pim_show_join_desired(struct vty *vty, u_char uj)
   struct interface     *ifp;
   struct pim_interface *pim_ifp;
   struct pim_ifchannel *ch;
-  char src_str[100];
-  char grp_str[100];
+  char src_str[INET_ADDRSTRLEN];
+  char grp_str[INET_ADDRSTRLEN];
   json_object *json = NULL;
   json_object *json_group = NULL;
   json_object *json_row = NULL;
@@ -1752,10 +1752,10 @@ static void pim_show_upstream_rpf(struct vty *vty, u_char uj)
             VTY_NEWLINE);
 
   for (ALL_LIST_ELEMENTS_RO(pim_upstream_list, upnode, up)) {
-    char src_str[100];
-    char grp_str[100];
-    char rpf_nexthop_str[100];
-    char rpf_addr_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
+    char rpf_nexthop_str[PREFIX_STRLEN];
+    char rpf_addr_str[PREFIX_STRLEN];
     struct pim_rpf *rpf;
     const char *rpf_ifname;
 
@@ -1871,10 +1871,10 @@ static void pim_show_rpf(struct vty *vty, u_char uj)
   }
 
   for (ALL_LIST_ELEMENTS_RO(pim_upstream_list, up_node, up)) {
-    char src_str[100];
-    char grp_str[100];
-    char rpf_addr_str[100];
-    char rib_nexthop_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
+    char rpf_addr_str[PREFIX_STRLEN];
+    char rib_nexthop_str[PREFIX_STRLEN];
     const char *rpf_ifname;
     struct pim_rpf  *rpf = &up->rpf;
     
@@ -1949,7 +1949,7 @@ static void igmp_show_groups(struct vty *vty, u_char uj)
     
     /* scan igmp sockets */
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_socket_list, sock_node, igmp)) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       struct listnode *grpnode;
       struct igmp_group *grp;
 
@@ -1957,7 +1957,7 @@ static void igmp_show_groups(struct vty *vty, u_char uj)
 
       /* scan igmp groups */
       for (ALL_LIST_ELEMENTS_RO(igmp->igmp_group_list, grpnode, grp)) {
-       char group_str[100];
+       char group_str[INET_ADDRSTRLEN];
        char hhmmss[10];
        char uptime[10];
 
@@ -2027,7 +2027,7 @@ static void igmp_show_group_retransmission(struct vty *vty)
     
     /* scan igmp sockets */
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_socket_list, sock_node, igmp)) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       struct listnode *grpnode;
       struct igmp_group *grp;
 
@@ -2035,7 +2035,7 @@ static void igmp_show_group_retransmission(struct vty *vty)
 
       /* scan igmp groups */
       for (ALL_LIST_ELEMENTS_RO(igmp->igmp_group_list, grpnode, grp)) {
-       char group_str[100];
+       char group_str[INET_ADDRSTRLEN];
        char grp_retr_mmss[10];
        struct listnode    *src_node;
        struct igmp_source *src;
@@ -2087,7 +2087,7 @@ static void igmp_show_sources(struct vty *vty)
     
     /* scan igmp sockets */
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_socket_list, sock_node, igmp)) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       struct listnode   *grpnode;
       struct igmp_group *grp;
 
@@ -2095,7 +2095,7 @@ static void igmp_show_sources(struct vty *vty)
 
       /* scan igmp groups */
       for (ALL_LIST_ELEMENTS_RO(igmp->igmp_group_list, grpnode, grp)) {
-       char group_str[100];
+       char group_str[INET_ADDRSTRLEN];
        struct listnode    *srcnode;
        struct igmp_source *src;
 
@@ -2103,7 +2103,7 @@ static void igmp_show_sources(struct vty *vty)
        
        /* scan group sources */
        for (ALL_LIST_ELEMENTS_RO(grp->group_source_list, srcnode, src)) {
-         char source_str[100];
+         char source_str[INET_ADDRSTRLEN];
          char mmss[10];
          char uptime[10];
 
@@ -2147,7 +2147,7 @@ static void igmp_show_source_retransmission(struct vty *vty)
     
     /* scan igmp sockets */
     for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_socket_list, sock_node, igmp)) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       struct listnode   *grpnode;
       struct igmp_group *grp;
 
@@ -2155,7 +2155,7 @@ static void igmp_show_source_retransmission(struct vty *vty)
 
       /* scan igmp groups */
       for (ALL_LIST_ELEMENTS_RO(igmp->igmp_group_list, grpnode, grp)) {
-       char group_str[100];
+       char group_str[INET_ADDRSTRLEN];
        struct listnode    *srcnode;
        struct igmp_source *src;
 
@@ -2163,7 +2163,7 @@ static void igmp_show_source_retransmission(struct vty *vty)
        
        /* scan group sources */
        for (ALL_LIST_ELEMENTS_RO(grp->group_source_list, srcnode, src)) {
-         char source_str[100];
+         char source_str[INET_ADDRSTRLEN];
 
          pim_inet4_dump("<source?>", src->source_addr, source_str, sizeof(source_str));
 
@@ -2248,8 +2248,8 @@ static void mroute_add_all()
   for (ALL_LIST_ELEMENTS_RO(pim_channel_oil_list, node, c_oil)) {
     if (pim_mroute_add(c_oil)) {
       /* just log warning */
-      char source_str[100];
-      char group_str[100];
+      char source_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       zlog_warn("%s %s: (S,G)=(%s,%s) failure writing MFC",
@@ -2267,8 +2267,8 @@ static void mroute_del_all()
   for (ALL_LIST_ELEMENTS_RO(pim_channel_oil_list, node, c_oil)) {
     if (pim_mroute_del(c_oil)) {
       /* just log warning */
-      char source_str[100];
-      char group_str[100];
+      char source_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       zlog_warn("%s %s: (S,G)=(%s,%s) failure clearing MFC",
@@ -2286,8 +2286,8 @@ static void static_mroute_add_all()
   for (ALL_LIST_ELEMENTS_RO(qpim_static_route_list, node, s_route)) {
     if (pim_mroute_add(&s_route->c_oil)) {
       /* just log warning */
-      char source_str[100];
-      char group_str[100];
+      char source_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", s_route->c_oil.oil.mfcc_origin, source_str, sizeof(source_str));
       pim_inet4_dump("<group?>", s_route->c_oil.oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       zlog_warn("%s %s: (S,G)=(%s,%s) failure writing MFC",
@@ -2305,8 +2305,8 @@ static void static_mroute_del_all()
    for (ALL_LIST_ELEMENTS_RO(qpim_static_route_list, node, s_route)) {
      if (pim_mroute_del(&s_route->c_oil)) {
        /* just log warning */
-       char source_str[100];
-       char group_str[100];
+       char source_str[INET_ADDRSTRLEN];
+       char group_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<source?>", s_route->c_oil.oil.mfcc_origin, source_str, sizeof(source_str));
        pim_inet4_dump("<group?>", s_route->c_oil.oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        zlog_warn("%s %s: (S,G)=(%s,%s) failure clearing MFC",
@@ -2843,8 +2843,8 @@ static void show_mroute(struct vty *vty, u_char uj)
 
   /* print list of PIM and IGMP routes */
   for (ALL_LIST_ELEMENTS_RO(pim_channel_oil_list, node, c_oil)) {
-    char grp_str[100];
-    char src_str[100];
+    char grp_str[INET_ADDRSTRLEN];
+    char src_str[INET_ADDRSTRLEN];
     char in_ifname[16];
     char out_ifname[16];
     int oif_vif_index;
@@ -2978,8 +2978,8 @@ static void show_mroute(struct vty *vty, u_char uj)
 
   /* Print list of static routes */
   for (ALL_LIST_ELEMENTS_RO(qpim_static_route_list, node, s_route)) {
-    char grp_str[100];
-    char src_str[100];
+    char grp_str[INET_ADDRSTRLEN];
+    char src_str[INET_ADDRSTRLEN];
     char in_ifname[16];
     char out_ifname[16];
     int oif_vif_index;
@@ -3124,8 +3124,8 @@ static void show_mroute_count(struct vty *vty)
 
   /* Print PIM and IGMP route counts */
   for (ALL_LIST_ELEMENTS_RO(pim_channel_oil_list, node, c_oil)) {
-    char group_str[100]; 
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN]; 
+    char source_str[INET_ADDRSTRLEN];
 
     if (!c_oil->installed)
       continue;
@@ -3147,8 +3147,8 @@ static void show_mroute_count(struct vty *vty)
 
    /* Print static route counts */
   for (ALL_LIST_ELEMENTS_RO(qpim_static_route_list, node, s_route)) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
 
     if (!s_route->c_oil.installed)
       continue;
@@ -3193,7 +3193,7 @@ DEFUN (show_ip_rib,
   struct in_addr addr;
   const char *addr_str;
   struct pim_nexthop nexthop;
-  char nexthop_addr_str[100];
+  char nexthop_addr_str[PREFIX_STRLEN];
   int result;
 
   addr_str = argv[idx_ipv4]->arg;
@@ -3242,11 +3242,11 @@ static void show_ssmpingd(struct vty *vty)
   now = pim_time_monotonic_sec();
 
   for (ALL_LIST_ELEMENTS_RO(qpim_ssmpingd_list, node, ss)) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     char ss_uptime[10];
     struct sockaddr_in bind_addr;
     socklen_t len = sizeof(bind_addr);
-    char bind_addr_str[100];
+    char bind_addr_str[INET_ADDRSTRLEN];
 
     pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
 
@@ -3725,7 +3725,7 @@ static void igmp_sock_query_interval_reconfig(struct igmp_sock *igmp)
   pim_ifp = ifp->info;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char ifaddr_str[100];
+    char ifaddr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
     zlog_debug("%s: Querier %s on %s reconfig query_interval=%d",
               __PRETTY_FUNCTION__,
index 434a0e66c4da3b602f17f55176558e8981b3a607..dd827c43b79b5ab6aace71e6d20e538ca7bed278 100644 (file)
@@ -38,7 +38,7 @@ static void on_trace(const char *label,
                     struct interface *ifp, struct in_addr src)
 {
   if (PIM_DEBUG_PIM_TRACE) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src, src_str, sizeof(src_str));
     zlog_debug("%s: from %s on %s",
               label, src_str, ifp->name);
@@ -50,7 +50,7 @@ static void tlv_trace_bool(const char *label, const char *tlv_name,
                           int isset, int value)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s=%d",
               label, 
@@ -64,7 +64,7 @@ static void tlv_trace_uint16(const char *label, const char *tlv_name,
                             int isset, uint16_t value)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s=%u",
               label, 
@@ -78,7 +78,7 @@ static void tlv_trace_uint32(const char *label, const char *tlv_name,
                             int isset, uint32_t value)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s=%u",
               label, 
@@ -92,7 +92,7 @@ static void tlv_trace_uint32_hex(const char *label, const char *tlv_name,
                                 int isset, uint32_t value)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s=%08x",
               label, 
@@ -107,7 +107,7 @@ static void tlv_trace(const char *label, const char *tlv_name,
                      int isset)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s",
               label, 
@@ -122,7 +122,7 @@ static void tlv_trace_list(const char *label, const char *tlv_name,
                           int isset, struct list *addr_list)
 {
   if (isset) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_debug("%s: PIM hello option from %s on interface %s: %s size=%d list=%p",
               label, 
@@ -182,7 +182,7 @@ int pim_hello_recv(struct interface *ifp,
 
     if (remain < PIM_TLV_MIN_SIZE) {
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_debug("%s: short PIM hello TLV size=%d < min=%d from %s on interface %s",
                   __PRETTY_FUNCTION__,
@@ -199,7 +199,7 @@ int pim_hello_recv(struct interface *ifp,
 
     if ((tlv_curr + option_len) > tlv_pastend) {
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_debug("%s: long PIM hello TLV type=%d length=%d > left=%td from %s on interface %s",
                   __PRETTY_FUNCTION__,
@@ -210,7 +210,7 @@ int pim_hello_recv(struct interface *ifp,
     }
 
     if (PIM_DEBUG_PIM_HELLO) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       zlog_debug("%s: parse left_size=%d: PIM hello TLV type=%d length=%d from %s on %s",
                 __PRETTY_FUNCTION__,
@@ -268,7 +268,7 @@ int pim_hello_recv(struct interface *ifp,
       break;
     case PIM_MSG_OPTION_TYPE_DM_STATE_REFRESH:
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_debug("%s: ignoring PIM hello dense-mode state refresh TLV option type=%d length=%d from %s on interface %s",
                   __PRETTY_FUNCTION__,
@@ -278,7 +278,7 @@ int pim_hello_recv(struct interface *ifp,
       break;
     default:
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_debug("%s: ignoring unknown PIM hello TLV type=%d length=%d from %s on interface %s",
                   __PRETTY_FUNCTION__,
@@ -327,7 +327,7 @@ int pim_hello_recv(struct interface *ifp,
 
   if (!PIM_OPTION_IS_SET(hello_options, PIM_OPTION_MASK_HOLDTIME)) {
     if (PIM_DEBUG_PIM_HELLO) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       zlog_debug("%s: PIM hello missing holdtime from %s on interface %s",
                __PRETTY_FUNCTION__,
@@ -354,7 +354,7 @@ int pim_hello_recv(struct interface *ifp,
                             PIM_NEIGHBOR_SEND_DELAY);
     if (!neigh) {
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_warn("%s: failure creating PIM neighbor %s on interface %s",
                  __PRETTY_FUNCTION__,
@@ -378,7 +378,7 @@ int pim_hello_recv(struct interface *ifp,
       /* GenID mismatch, then replace neighbor */
       
       if (PIM_DEBUG_PIM_HELLO) {
-       char src_str[100];
+       char src_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
        zlog_debug("%s: GenId mismatch new=%08x old=%08x: replacing neighbor %s on %s",
                   __PRETTY_FUNCTION__,
@@ -401,7 +401,7 @@ int pim_hello_recv(struct interface *ifp,
                               PIM_NEIGHBOR_SEND_NOW);
       if (!neigh) {
        if (PIM_DEBUG_PIM_HELLO) {
-         char src_str[100];
+         char src_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
          zlog_debug("%s: failure re-creating PIM neighbor %s on interface %s",
                     __PRETTY_FUNCTION__,
index ceab93edff500e2ad68f41b11ace677bfdfdad93..3f3629395c5afb37eaec0341ef47a51f70af7cb0 100644 (file)
@@ -295,8 +295,8 @@ static int detect_primary_address_change(struct interface *ifp,
   changed = new_prim_addr.s_addr != pim_ifp->primary_address.s_addr;
 
   if (PIM_DEBUG_ZEBRA) {
-    char new_prim_str[100];
-    char old_prim_str[100];
+    char new_prim_str[INET_ADDRSTRLEN];
+    char old_prim_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<new?>", new_prim_addr, new_prim_str, sizeof(new_prim_str));
     pim_inet4_dump("<old?>", pim_ifp->primary_address, old_prim_str, sizeof(old_prim_str));
     zlog_debug("%s: old=%s new=%s on interface %s: %s",
@@ -965,7 +965,7 @@ struct pim_neighbor *pim_if_find_neighbor(struct interface *ifp,
   }
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_debug("%s: neighbor not found for address %s on interface %s",
               __PRETTY_FUNCTION__, 
@@ -1051,8 +1051,8 @@ static struct igmp_join *igmp_join_new(struct interface *ifp,
 
   join_fd = igmp_join_sock(ifp->name, ifp->ifindex, group_addr, source_addr);
   if (join_fd < 0) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: igmp_join_sock() failure for IGMP group %s source %s on interface %s",
@@ -1063,8 +1063,8 @@ static struct igmp_join *igmp_join_new(struct interface *ifp,
 
   ij = XCALLOC(MTYPE_PIM_IGMP_JOIN, sizeof(*ij));
   if (!ij) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_err("%s: XCALLOC(%zu) failure for IGMP group %s source %s on interface %s",
@@ -1111,8 +1111,8 @@ int pim_if_igmp_join_add(struct interface *ifp,
 
   ij = igmp_join_find(pim_ifp->igmp_join_list, group_addr, source_addr);
   if (ij) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: can't re-join existing IGMP group %s source %s on interface %s",
@@ -1123,8 +1123,8 @@ int pim_if_igmp_join_add(struct interface *ifp,
 
   ij = igmp_join_new(ifp, group_addr, source_addr);
   if (!ij) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: igmp_join_new() failure for IGMP group %s source %s on interface %s",
@@ -1134,8 +1134,8 @@ int pim_if_igmp_join_add(struct interface *ifp,
   }
 
   if (PIM_DEBUG_IGMP_EVENTS) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_debug("%s: issued static igmp join for channel (S,G)=(%s,%s) on interface %s",
@@ -1172,8 +1172,8 @@ int pim_if_igmp_join_del(struct interface *ifp,
 
   ij = igmp_join_find(pim_ifp->igmp_join_list, group_addr, source_addr);
   if (!ij) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: could not find IGMP group %s source %s on interface %s",
@@ -1183,8 +1183,8 @@ int pim_if_igmp_join_del(struct interface *ifp,
   }
 
   if (close(ij->sock_fd)) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: failure closing sock_fd=%d for IGMP group %s source %s on interface %s: errno=%d: %s",
index 2f9e9b59814b649433d8ecad2d57a37f3a850221..93a6cad649d65984e92a73cba9e1926dd64a2562 100644 (file)
@@ -524,8 +524,8 @@ static void check_recv_upstream(int is_join,
 
   /* upstream directed to RPF'(S,G) ? */
   if (upstream.s_addr != up->rpf.rpf_addr.u.prefix4.s_addr) {
-    char up_str[100];
-    char rpf_str[100];
+    char up_str[INET_ADDRSTRLEN];
+    char rpf_str[PREFIX_STRLEN];
     pim_inet4_dump("<up?>", upstream, up_str, sizeof(up_str));
     pim_addr_dump("<rpf?>", &up->rpf.rpf_addr, rpf_str, sizeof(rpf_str));
     zlog_warn("%s %s: (S,G)=%s upstream=%s not directed to RPF'(S,G)=%s on interface %s",
@@ -579,7 +579,7 @@ static int nonlocal_upstream(int is_join,
   is_local = (upstream.s_addr == recv_pim_ifp->primary_address.s_addr);
   
   if (PIM_DEBUG_PIM_TRACE) {
-    char up_str[100];
+    char up_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<upstream?>", upstream, up_str, sizeof(up_str));
     zlog_warn("%s: recv %s (S,G)=%s to %s upstream=%s on %s",
              __PRETTY_FUNCTION__,
@@ -638,7 +638,7 @@ void pim_ifchannel_join_add(struct interface *ifp,
     address of the join message is our primary address.
    */
   if (ch->ifassert_state == PIM_IFASSERT_I_AM_LOSER) {
-    char neigh_str[100];
+    char neigh_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<neigh?>", neigh_addr, neigh_str, sizeof(neigh_str));
     zlog_warn("%s: Assert Loser recv Join%s from %s on %s",
              __PRETTY_FUNCTION__,
@@ -881,8 +881,8 @@ void pim_ifchannel_update_could_assert(struct pim_ifchannel *ch)
     return;
 
   if (PIM_DEBUG_PIM_EVENTS) {
-    char src_str[100];
-    char grp_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ch->sg.src, src_str, sizeof(src_str));
     pim_inet4_dump("<grp?>", ch->sg.grp, grp_str, sizeof(grp_str));
     zlog_debug("%s: CouldAssert(%s,%s,%s) changed from %d to %d",
@@ -923,10 +923,10 @@ void pim_ifchannel_update_my_assert_metric(struct pim_ifchannel *ch)
       return;
 
   if (PIM_DEBUG_PIM_EVENTS) {
-    char src_str[100];
-    char grp_str[100];
-    char old_addr_str[100];
-    char new_addr_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
+    char old_addr_str[INET_ADDRSTRLEN];
+    char new_addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ch->sg.src, src_str, sizeof(src_str));
     pim_inet4_dump("<grp?>", ch->sg.grp, grp_str, sizeof(grp_str));
     pim_inet4_dump("<old_addr?>", ch->ifassert_my_metric.ip_address, old_addr_str, sizeof(old_addr_str));
@@ -961,8 +961,8 @@ void pim_ifchannel_update_assert_tracking_desired(struct pim_ifchannel *ch)
     return;
 
   if (PIM_DEBUG_PIM_EVENTS) {
-    char src_str[100];
-    char grp_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char grp_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ch->sg.src, src_str, sizeof(src_str));
     pim_inet4_dump("<grp?>", ch->sg.grp, grp_str, sizeof(grp_str));
     zlog_debug("%s: AssertTrackingDesired(%s,%s,%s) changed from %d to %d",
index 56f2457f97b1e68bd1a7e6ef57124fdd83abb642..5532dc1dd06a0f44c41bcabcfcb45d3c53cb6227 100644 (file)
@@ -161,7 +161,7 @@ static int pim_igmp_other_querier_expire(struct thread *t)
   zassert(!igmp->t_igmp_query_timer);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char ifaddr_str[100];
+    char ifaddr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
     zlog_debug("%s: Querier %s resuming",
               __PRETTY_FUNCTION__,
@@ -197,7 +197,7 @@ void pim_igmp_other_querier_timer_on(struct igmp_sock *igmp)
     */
 
     if (PIM_DEBUG_IGMP_TRACE) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
       zlog_debug("Querier %s resetting TIMER event for Other-Querier-Present",
                 ifaddr_str);
@@ -240,7 +240,7 @@ void pim_igmp_other_querier_timer_on(struct igmp_sock *igmp)
                       pim_ifp->igmp_query_max_response_time_dsec);
   
   if (PIM_DEBUG_IGMP_TRACE) {
-    char ifaddr_str[100];
+    char ifaddr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
     zlog_debug("Querier %s scheduling %ld.%03ld sec TIMER event for Other-Querier-Present",
               ifaddr_str,
@@ -259,7 +259,7 @@ void pim_igmp_other_querier_timer_off(struct igmp_sock *igmp)
 
   if (PIM_DEBUG_IGMP_TRACE) {
     if (igmp->t_other_querier_timer) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
       zlog_debug("IGMP querier %s fd=%d cancelling other-querier-present TIMER event on %s",
                 ifaddr_str, igmp->fd, igmp->interface->name);
@@ -321,7 +321,7 @@ igmp_recv_query(struct igmp_sock *igmp, int query_version,
   }
 
   if (PIM_DEBUG_IGMP_PACKETS) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
     zlog_debug("Recv IGMP query v%d from %s on %s for group %s",
               query_version, from_str, ifp->name, group_str);
@@ -338,7 +338,7 @@ igmp_recv_query(struct igmp_sock *igmp, int query_version,
   if (ntohl(from.s_addr) < ntohl(igmp->ifaddr.s_addr)) {
 
     if (PIM_DEBUG_IGMP_TRACE) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
       zlog_debug("%s: local address %s (%u) lost querier election to %s (%u)",
                 ifp->name,
@@ -361,7 +361,7 @@ static void on_trace(const char *label,
                     struct interface *ifp, struct in_addr from)
 {
   if (PIM_DEBUG_IGMP_TRACE) {
-    char from_str[100];
+    char from_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<from?>", from, from_str, sizeof(from_str));
     zlog_debug("%s: from %s on %s",
               label, from_str, ifp->name);
@@ -410,8 +410,8 @@ int pim_igmp_packet(struct igmp_sock *igmp, char *buf, size_t len)
   char *igmp_msg;
   int igmp_msg_len;
   int msg_type;
-  char from_str[100];
-  char to_str[100];
+  char from_str[INET_ADDRSTRLEN];
+  char to_str[INET_ADDRSTRLEN];
     
   if (len < sizeof(*ip_hdr)) {
     zlog_warn("IGMP packet size=%zu shorter than minimum=%zu",
@@ -560,7 +560,7 @@ void pim_igmp_general_query_on(struct igmp_sock *igmp)
   }
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char ifaddr_str[100];
+    char ifaddr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
     zlog_debug("Querier %s scheduling %d-second (%s) TIMER event for IGMP query on fd=%d",
               ifaddr_str,
@@ -580,7 +580,7 @@ void pim_igmp_general_query_off(struct igmp_sock *igmp)
 
   if (PIM_DEBUG_IGMP_TRACE) {
     if (igmp->t_igmp_query_timer) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
       zlog_debug("IGMP querier %s fd=%d cancelling query TIMER event on %s",
                 ifaddr_str, igmp->fd, igmp->interface->name);
@@ -629,8 +629,8 @@ static int pim_igmp_general_query(struct thread *t)
   group_addr.s_addr = PIM_NET_INADDR_ANY;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char querier_str[100];
-    char dst_str[100];
+    char querier_str[INET_ADDRSTRLEN];
+    char dst_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<querier?>", igmp->ifaddr, querier_str,
                   sizeof(querier_str));
     pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
@@ -719,7 +719,7 @@ static void igmp_group_delete(struct igmp_group *group)
   struct igmp_source *src;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Deleting IGMP group %s from socket %d interface %s",
               group_str,
@@ -887,7 +887,7 @@ static int igmp_group_timer(struct thread *t)
   zassert(group);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("%s: Timer for group %s on interface %s",
               __PRETTY_FUNCTION__,
@@ -926,7 +926,7 @@ static void group_timer_off(struct igmp_group *group)
     return;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Cancelling TIMER event for group %s on %s",
               group_str, group->group_igmp_sock->interface->name);
@@ -942,7 +942,7 @@ void igmp_group_timer_on(struct igmp_group *group,
   group_timer_off(group);
 
   if (PIM_DEBUG_IGMP_EVENTS) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Scheduling %ld.%03ld sec TIMER event for group %s on %s",
               interval_msec / 1000,
@@ -1033,7 +1033,7 @@ struct igmp_group *igmp_add_group_by_addr(struct igmp_sock *igmp,
   listnode_add(igmp->igmp_group_list, group);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Creating new IGMP group %s on socket %d interface %s",
               group_str, igmp->fd, igmp->interface->name);
index cc78f18d7c4966c55edcfdf341eeca57f09b85a0..4f005f01954d8137d34195285e86ea810ec08686 100644 (file)
@@ -33,7 +33,7 @@ on_trace (const char *label,
           struct interface *ifp, struct in_addr from)
 {
   if (PIM_DEBUG_IGMP_TRACE) {
-    char from_str[100];
+    char from_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<from?>", from, from_str, sizeof(from_str));
     zlog_debug("%s: from %s on %s",
                label, from_str, ifp->name);
@@ -69,8 +69,8 @@ igmp_v2_send_query (struct igmp_group *group,
   *(uint16_t *)(query_buf + IGMP_CHECKSUM_OFFSET) = checksum;
 
   if (PIM_DEBUG_IGMP_PACKETS) {
-    char dst_str[100];
-    char group_str[100];
+    char dst_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
     zlog_debug("Send IGMPv2 QUERY to %s on %s for group %s",
@@ -85,8 +85,8 @@ igmp_v2_send_query (struct igmp_group *group,
   sent = sendto(fd, query_buf, msg_size, MSG_DONTWAIT,
                 (struct sockaddr *)&to, tolen);
   if (sent != (ssize_t) msg_size) {
-    char dst_str[100];
-    char group_str[100];
+    char dst_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
     if (sent < 0) {
@@ -108,7 +108,7 @@ igmp_v2_recv_report (struct igmp_sock *igmp,
 {
   struct interface *ifp = igmp->interface;
   struct in_addr group_addr;
-  char group_str[100];
+  char group_str[INET_ADDRSTRLEN];
 
   on_trace(__PRETTY_FUNCTION__, igmp->interface, from);
 
@@ -151,7 +151,7 @@ igmp_v2_recv_leave (struct igmp_sock *igmp,
 {
   struct interface *ifp = igmp->interface;
   struct in_addr group_addr;
-  char group_str[100];
+  char group_str[INET_ADDRSTRLEN];
 
   on_trace(__PRETTY_FUNCTION__, igmp->interface, from);
 
index 21a5176a9d798b63bc3e80b404225148e8914c2f..706f708dd190067e3b76b841f3030865c10a66b9 100644 (file)
@@ -47,8 +47,8 @@ static void on_trace(const char *label,
                     int num_sources, struct in_addr *sources)
 {
   if (PIM_DEBUG_IGMP_TRACE) {
-    char from_str[100];
-    char group_str[100];
+    char from_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
 
     pim_inet4_dump("<from?>", from, from_str, sizeof(from_str));
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
@@ -89,7 +89,7 @@ void igmp_group_reset_gmi(struct igmp_group *group)
                      pim_ifp->igmp_query_max_response_time_dsec);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Resetting group %s timer to GMI=%ld.%03ld sec on %s",
               group_str,
@@ -122,8 +122,8 @@ static int igmp_source_timer(struct thread *t)
   group = source->source_group;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_debug("%s: Source timer expired for group %s source %s on %s",
@@ -187,8 +187,8 @@ static void source_timer_off(struct igmp_group *group,
     return;
   
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_debug("Cancelling TIMER event for group %s source %s on %s",
@@ -207,8 +207,8 @@ static void igmp_source_timer_on(struct igmp_group *group,
   source_timer_off(group, source);
 
   if (PIM_DEBUG_IGMP_EVENTS) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_debug("Scheduling %ld.%03ld sec TIMER event for group %s source %s on %s",
@@ -248,8 +248,8 @@ void igmp_source_reset_gmi(struct igmp_sock *igmp,
                      pim_ifp->igmp_query_max_response_time_dsec);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
 
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
@@ -355,8 +355,8 @@ void igmp_source_delete(struct igmp_source *source)
   group = source->source_group;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_debug("Deleting IGMP source %s for group %s from socket %d interface %s",
@@ -370,8 +370,8 @@ void igmp_source_delete(struct igmp_source *source)
 
   /* sanity check that forwarding has been disabled */
   if (IGMP_SOURCE_TEST_FORWARDING(source->source_flags)) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: forwarding=ON(!) IGMP source %s for group %s from socket %d interface %s",
@@ -440,8 +440,8 @@ source_new (struct igmp_group *group,
   struct igmp_source *src;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", src_addr, source_str, sizeof(source_str));
     zlog_debug("Creating new IGMP source %s for group %s on socket %d interface %s",
@@ -1044,7 +1044,7 @@ static void group_retransmit_group(struct igmp_group *group)
   s_flag = igmp_group_timer_remain_msec(group) > lmqt_msec;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("retransmit_group_specific_query: group %s on %s: s_flag=%d count=%d",
               group_str, igmp->interface->name, s_flag,
@@ -1143,7 +1143,7 @@ static int group_retransmit_sources(struct igmp_group *group,
   num_sources_tosend2 = source_addr2 - (struct in_addr *)(query_buf2 + IGMP_V3_SOURCES_OFFSET);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("retransmit_grp&src_specific_query: group %s on %s: srcs_with_sflag=%d srcs_wo_sflag=%d will_send_sflag=%d retransmit_src_left=%d",
               group_str, igmp->interface->name,
@@ -1163,7 +1163,7 @@ static int group_retransmit_sources(struct igmp_group *group,
 
       query_buf1_max_sources = (sizeof(query_buf1) - IGMP_V3_SOURCES_OFFSET) >> 2;
       if (num_sources_tosend1 > query_buf1_max_sources) {
-       char group_str[100];
+       char group_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
        zlog_warn("%s: group %s on %s: s_flag=1 unable to fit %d sources into buf_size=%zu (max_sources=%d)",
                  __PRETTY_FUNCTION__, group_str, igmp->interface->name,
@@ -1205,7 +1205,7 @@ static int group_retransmit_sources(struct igmp_group *group,
   
     query_buf2_max_sources = (sizeof(query_buf2) - IGMP_V3_SOURCES_OFFSET) >> 2;
     if (num_sources_tosend2 > query_buf2_max_sources) {
-      char group_str[100];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
       zlog_warn("%s: group %s on %s: s_flag=0 unable to fit %d sources into buf_size=%zu (max_sources=%d)",
                __PRETTY_FUNCTION__, group_str, igmp->interface->name,
@@ -1250,7 +1250,7 @@ static int igmp_group_retransmit(struct thread *t)
   zassert(group);
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("group_retransmit_timer: group %s on %s",
               group_str, group->group_igmp_sock->interface->name);
@@ -1316,7 +1316,7 @@ static void group_retransmit_timer_on(struct igmp_group *group)
   lmqi_msec = 100 * pim_ifp->igmp_specific_query_max_response_time_dsec;
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("Scheduling %ld.%03ld sec retransmit timer for group %s on %s",
               lmqi_msec / 1000,
@@ -1545,7 +1545,7 @@ void igmp_group_timer_lower_to_lmqt(struct igmp_group *group)
   lmqt_msec = PIM_IGMP_LMQT_MSEC(lmqi_dsec, lmqc); /* lmqt_msec = (100 * lmqi_dsec) * lmqc */
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     zlog_debug("%s: group %s on %s: LMQC=%d LMQI=%d dsec LMQT=%d msec",
               __PRETTY_FUNCTION__,
@@ -1580,8 +1580,8 @@ void igmp_source_timer_lower_to_lmqt(struct igmp_source *source)
   lmqt_msec = PIM_IGMP_LMQT_MSEC(lmqi_dsec, lmqc); /* lmqt_msec = (100 * lmqi_dsec) * lmqc */
 
   if (PIM_DEBUG_IGMP_TRACE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", group->group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
     zlog_debug("%s: group %s source %s on %s: LMQC=%d LMQI=%d dsec LMQT=%d msec",
@@ -1656,8 +1656,8 @@ igmp_v3_send_query (struct igmp_group *group,
   *(uint16_t *)(query_buf + IGMP_CHECKSUM_OFFSET) = checksum;
 
   if (PIM_DEBUG_IGMP_PACKETS) {
-    char dst_str[100];
-    char group_str[100];
+    char dst_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
     zlog_debug("Send IGMPv3 query to %s on %s for group %s, sources=%d msg_size=%zd s_flag=%x QRV=%u QQI=%u QQIC=%02x",
@@ -1674,8 +1674,8 @@ igmp_v3_send_query (struct igmp_group *group,
   sent = sendto(fd, query_buf, msg_size, MSG_DONTWAIT,
                 (struct sockaddr *)&to, tolen);
   if (sent != (ssize_t) msg_size) {
-    char dst_str[100];
-    char group_str[100];
+    char dst_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
     pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
     if (sent < 0) {
@@ -1703,8 +1703,8 @@ igmp_v3_send_query (struct igmp_group *group,
   if (!s_flag) {
     /* general query? */
     if (PIM_INADDR_IS_ANY(group_addr)) {
-      char dst_str[100];
-      char group_str[100];
+      char dst_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<dst?>", dst_addr, dst_str, sizeof(dst_str));
       pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
       zlog_warn("%s: to %s on %s: group=%s sources=%d: s_flag is clear for general query!",
@@ -1759,7 +1759,7 @@ igmp_v3_recv_query (struct igmp_sock *igmp, const char *from_str, char *igmp_msg
     igmp->querier_query_interval = qqi ? qqi : pim_ifp->igmp_default_query_interval;
 
     if (PIM_DEBUG_IGMP_TRACE) {
-      char ifaddr_str[100];
+      char ifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", igmp->ifaddr, ifaddr_str, sizeof(ifaddr_str));
       zlog_debug("Querier %s new query interval is %s QQI=%u sec (recv QQIC=%02x from %s)",
                  ifaddr_str,
@@ -1823,7 +1823,7 @@ igmp_v3_recv_query (struct igmp_sock *igmp, const char *from_str, char *igmp_msg
           }
         }
       } else {
-        char group_str[100];
+        char group_str[INET_ADDRSTRLEN];
         pim_inet4_dump("<group?>", group_addr, group_str, sizeof(group_str));
         zlog_warn("IGMP query v3 from %s on %s: could not find group %s for timer update",
                   from_str, ifp->name, group_str);
index 0a00be9e60b5602815f73e81e456ad7c556ef9b0..0490efda9e3e8a0bc3f4ff030f94975bd8971b1d 100644 (file)
@@ -45,7 +45,7 @@ on_trace (const char *label,
          struct interface *ifp, struct in_addr src)
 {
   if (PIM_DEBUG_PIM_TRACE) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src, src_str, sizeof(src_str));
     zlog_debug("%s: from %s on %s",
               label, src_str, ifp->name);
@@ -67,8 +67,8 @@ static void recv_join(struct interface *ifp,
   sg.grp = group;
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char up_str[100];
-    char neigh_str[100];
+    char up_str[INET_ADDRSTRLEN];
+    char neigh_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<upstream?>", upstream, up_str, sizeof(up_str));
     pim_inet4_dump("<neigh?>", neigh->source_addr, neigh_str, sizeof(neigh_str));
     zlog_warn("%s: join (S,G)=%s rpt=%d wc=%d upstream=%s holdtime=%d from %s on %s",
@@ -155,8 +155,8 @@ static void recv_prune(struct interface *ifp,
   sg.grp = group;
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char up_str[100];
-    char neigh_str[100];
+    char up_str[INET_ADDRSTRLEN];
+    char neigh_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<upstream?>", upstream, up_str, sizeof(up_str));
     pim_inet4_dump("<neigh?>", neigh->source_addr, neigh_str, sizeof(neigh_str));
     zlog_warn("%s: prune (S,G)=%s rpt=%d wc=%d upstream=%s holdtime=%d from %s on %s",
@@ -253,7 +253,7 @@ int pim_joinprune_recv(struct interface *ifp,
   addr_offset = pim_parse_addr_ucast (&msg_upstream_addr,
                                      buf, pastend - buf);
   if (addr_offset < 1) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: pim_parse_addr_ucast() failure: from %s on %s",
              __PRETTY_FUNCTION__,
@@ -267,7 +267,7 @@ int pim_joinprune_recv(struct interface *ifp,
    */
   if (msg_upstream_addr.family != AF_INET) {
     if (PIM_DEBUG_PIM_J_P) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       zlog_warn("%s: ignoring join/prune directed to unexpected addr family=%d from %s on %s",
                __PRETTY_FUNCTION__,
@@ -278,7 +278,7 @@ int pim_joinprune_recv(struct interface *ifp,
 
   remain = pastend - buf;
   if (remain < 4) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: short join/prune message buffer for group list: size=%d minimum=%d from %s on %s",
              __PRETTY_FUNCTION__,
@@ -294,8 +294,8 @@ int pim_joinprune_recv(struct interface *ifp,
   ++buf;
 
   if (PIM_DEBUG_PIM_J_P) {
-    char src_str[100];
-    char upstream_str[100];
+    char src_str[INET_ADDRSTRLEN];
+    char upstream_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     pim_inet4_dump("<addr?>", msg_upstream_addr.u.prefix4,
                   upstream_str, sizeof(upstream_str));
@@ -323,7 +323,7 @@ int pim_joinprune_recv(struct interface *ifp,
 
     remain = pastend - buf;
     if (remain < 4) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       zlog_warn("%s: short join/prune buffer for source list: size=%d minimum=%d from %s on %s",
                __PRETTY_FUNCTION__,
@@ -337,9 +337,9 @@ int pim_joinprune_recv(struct interface *ifp,
     buf += 2;
 
     if (PIM_DEBUG_PIM_J_P) {
-      char src_str[100];
-      char upstream_str[100];
-      char group_str[100];
+      char src_str[INET_ADDRSTRLEN];
+      char upstream_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       pim_inet4_dump("<addr?>", msg_upstream_addr.u.prefix4,
                     upstream_str, sizeof(upstream_str));
@@ -416,7 +416,7 @@ int pim_joinprune_send(struct interface *ifp,
   }
 
   if (PIM_DEBUG_PIM_J_P) {
-    char dst_str[100];
+    char dst_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", upstream_addr, dst_str, sizeof(dst_str));
     zlog_debug("%s: sending %s(S,G)=%s to upstream=%s on interface %s",
               __PRETTY_FUNCTION__,
@@ -426,7 +426,7 @@ int pim_joinprune_send(struct interface *ifp,
 
   if (PIM_INADDR_IS_ANY(upstream_addr)) {
     if (PIM_DEBUG_PIM_J_P) {
-      char dst_str[100];
+      char dst_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<dst?>", upstream_addr, dst_str, sizeof(dst_str));
       zlog_debug("%s: %s(S,G)=%s: upstream=%s is myself on interface %s",
                 __PRETTY_FUNCTION__,
index 341d733361a7ec1cf64b5e0382ba90904f57ad9d..982aa9477a12e8e3e5269b44d54b496aa3a4966a 100644 (file)
@@ -402,10 +402,10 @@ int pim_mroute_msg(int fd, const char *buf, int buf_size)
   struct pim_interface *pim_ifp;
   const struct ip      *ip_hdr;
   const struct igmpmsg *msg;
-  char ip_src_str[100] = "";
-  char ip_dst_str[100] = "";
-  char src_str[100] = "<src?>";
-  char grp_str[100] = "<grp?>";
+  char ip_src_str[INET_ADDRSTRLEN] = "";
+  char ip_dst_str[INET_ADDRSTRLEN] = "";
+  char src_str[INET_ADDRSTRLEN] = "<src?>";
+  char grp_str[INET_ADDRSTRLEN] = "<grp?>";
   struct in_addr ifaddr;
   struct igmp_sock *igmp;
 
@@ -646,7 +646,7 @@ int pim_mroute_add_vif(struct interface *ifp, struct in_addr ifaddr, unsigned ch
 
   err = setsockopt(qpim_mroute_socket_fd, IPPROTO_IP, MRT_ADD_VIF, (void*) &vc, sizeof(vc));
   if (err) {
-    char ifaddr_str[100];
+    char ifaddr_str[INET_ADDRSTRLEN];
 
     pim_inet4_dump("<ifaddr?>", ifaddr, ifaddr_str, sizeof(ifaddr_str));
 
index 0eadb5eb38574ab8bc0c7a6680490b1b8ad3c8a5..edc7f0ee42fa8369f950a80362303913d7af53dd 100644 (file)
@@ -128,7 +128,7 @@ pim_msg_join_prune_encode (uint8_t *buf, int buf_size, int is_join,
   remain = end - pim_msg_curr;
   pim_msg_curr = pim_msg_addr_encode_ipv4_ucast (pim_msg_curr, buf_size - PIM_MSG_HEADER_LEN, upstream);
   if (!pim_msg_curr) {
-    char dst_str[100];
+    char dst_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", upstream, dst_str, sizeof(dst_str));
     zlog_warn("%s: failure encoding destination address %s: space left=%d",
              __PRETTY_FUNCTION__, dst_str, remain);
@@ -155,7 +155,7 @@ pim_msg_join_prune_encode (uint8_t *buf, int buf_size, int is_join,
   pim_msg_curr = pim_msg_addr_encode_ipv4_group (pim_msg_curr, remain,
                                                  group);
   if (!pim_msg_curr) {
-    char group_str[100];
+    char group_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group, group_str, sizeof(group_str));
     zlog_warn("%s: failure encoding group address %s: space left=%d",
              __PRETTY_FUNCTION__, group_str, remain);
@@ -193,7 +193,7 @@ pim_msg_join_prune_encode (uint8_t *buf, int buf_size, int is_join,
     }
   pim_msg_curr = pim_msg_addr_encode_ipv4_source (pim_msg_curr, remain, stosend, bits);
   if (!pim_msg_curr) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source, source_str, sizeof(source_str));
     zlog_warn("%s: failure encoding source address %s: space left=%d",
              __PRETTY_FUNCTION__, source_str, remain);
index 3696d2d47e187e397352192192cac87fded57f97..abca6094216e052fd26d3b81e1104924d1da42b4 100644 (file)
@@ -129,8 +129,8 @@ int pim_if_dr_election(struct interface *ifp)
   if (old_dr_addr.s_addr != pim_ifp->pim_dr_addr.s_addr) {
 
     if (PIM_DEBUG_PIM_EVENTS) {
-      char dr_old_str[100];
-      char dr_new_str[100];
+      char dr_old_str[INET_ADDRSTRLEN];
+      char dr_new_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<old_dr?>", old_dr_addr, dr_old_str, sizeof(dr_old_str));
       pim_inet4_dump("<new_dr?>", pim_ifp->pim_dr_addr, dr_new_str, sizeof(dr_new_str));
       zlog_debug("%s: DR was %s now is %s on interface %s",
@@ -218,7 +218,7 @@ static int on_neighbor_timer(struct thread *t)
   ifp = neigh->interface;
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", neigh->source_addr, src_str, sizeof(src_str));
     zlog_debug("Expired %d sec holdtime for neighbor %s on interface %s",
               neigh->holdtime, src_str, ifp->name);
@@ -245,7 +245,7 @@ static void neighbor_timer_off(struct pim_neighbor *neigh)
 {
   if (PIM_DEBUG_PIM_TRACE_DETAIL) {
     if (neigh->t_expire_timer) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", neigh->source_addr, src_str, sizeof(src_str));
       zlog_debug("%s: cancelling timer for neighbor %s on %s",
                 __PRETTY_FUNCTION__,
@@ -270,7 +270,7 @@ void pim_neighbor_timer_reset(struct pim_neighbor *neigh, uint16_t holdtime)
   }
 
   if (PIM_DEBUG_PIM_TRACE_DETAIL) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", neigh->source_addr, src_str, sizeof(src_str));
     zlog_debug("%s: starting %u sec timer for neighbor %s on %s",
               __PRETTY_FUNCTION__,
@@ -294,7 +294,7 @@ static struct pim_neighbor *pim_neighbor_new(struct interface *ifp,
 {
   struct pim_interface *pim_ifp;
   struct pim_neighbor *neigh;
-  char src_str[100];
+  char src_str[INET_ADDRSTRLEN];
 
   zassert(ifp);
   pim_ifp = ifp->info;
@@ -541,7 +541,7 @@ void pim_neighbor_delete(struct interface *ifp,
                         const char *delete_message)
 {
   struct pim_interface *pim_ifp;
-  char src_str[100];
+  char src_str[INET_ADDRSTRLEN];
 
   pim_ifp = ifp->info;
   zassert(pim_ifp);
@@ -679,9 +679,9 @@ static void delete_from_neigh_addr(struct interface *ifp,
       {
        struct prefix *p = pim_neighbor_find_secondary(neigh, addr->u.prefix4);
        if (p) {
-         char addr_str[100];
-         char this_neigh_str[100];
-         char other_neigh_str[100];
+         char addr_str[INET_ADDRSTRLEN];
+         char this_neigh_str[INET_ADDRSTRLEN];
+         char other_neigh_str[INET_ADDRSTRLEN];
          
          pim_inet4_dump("<addr?>", addr->u.prefix4, addr_str, sizeof(addr_str));
          pim_inet4_dump("<neigh1?>", neigh_addr, this_neigh_str, sizeof(this_neigh_str));
index 7fe4d8af323feb2b268c22063fd1943e5c8a7b1d..94165032043d8e72b22c15942451b2825100227b 100644 (file)
@@ -206,8 +206,8 @@ pim_channel_del_oif (struct channel_oil *channel_oil,
 
   if (PIM_DEBUG_MROUTE)
     {
-      char group_str[100];
-      char source_str[100];
+      char group_str[INET_ADDRSTRLEN];
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       zlog_debug("%s %s: (S,G)=(%s,%s): proto_mask=%u OIF=%s vif_index=%d",
@@ -224,8 +224,8 @@ pim_channel_del_oif (struct channel_oil *channel_oil,
     {
       if (PIM_DEBUG_MROUTE)
        {
-         char group_str[100];
-         char source_str[100];
+         char group_str[INET_ADDRSTRLEN];
+         char source_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
          pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
          zlog_debug("%s %s: no existing protocol mask %u(%u) for requested OIF %s (vif_index=%d, min_ttl=%d) for channel (S,G)=(%s,%s)",
@@ -244,8 +244,8 @@ pim_channel_del_oif (struct channel_oil *channel_oil,
     {
       if (PIM_DEBUG_MROUTE)
        {
-         char group_str[100];
-         char source_str[100];
+         char group_str[INET_ADDRSTRLEN];
+         char source_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
          pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
          zlog_debug("%s %s: other protocol masks remain for requested OIF %s (vif_index=%d, min_ttl=%d) for channel (S,G)=(%s,%s)",
@@ -262,8 +262,8 @@ pim_channel_del_oif (struct channel_oil *channel_oil,
   if (pim_mroute_add (channel_oil)) {
     if (PIM_DEBUG_MROUTE)
       {
-        char group_str[100];
-        char source_str[100];
+        char group_str[INET_ADDRSTRLEN];
+        char source_str[INET_ADDRSTRLEN];
         pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
         pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
         zlog_debug("%s %s: could not remove output interface %s (vif_index=%d) for channel (S,G)=(%s,%s)",
@@ -298,8 +298,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   pim_ifp = oif->info;
 
   if (PIM_DEBUG_MROUTE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_debug("%s %s: (S,G)=(%s,%s): proto_mask=%u OIF=%s vif_index=%d",
@@ -323,8 +323,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   if (pim_ifp->mroute_vif_index == channel_oil->oil.mfcc_parent) {
     if (PIM_DEBUG_MROUTE)
       {
-       char group_str[100];
-       char source_str[100];
+       char group_str[INET_ADDRSTRLEN];
+       char source_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        zlog_debug("%s %s: refusing protocol mask %u request for IIF=OIF=%s (vif_index=%d) for channel (S,G)=(%s,%s)",
@@ -341,8 +341,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   if (channel_oil->oif_flags[pim_ifp->mroute_vif_index] & proto_mask) {
     if (PIM_DEBUG_MROUTE)
       {
-       char group_str[100];
-       char source_str[100];
+       char group_str[INET_ADDRSTRLEN];
+       char source_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        zlog_debug("%s %s: existing protocol mask %u requested OIF %s (vif_index=%d, min_ttl=%d) for channel (S,G)=(%s,%s)",
@@ -364,8 +364,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
     if (channel_oil->oil.mfcc_ttls[pim_ifp->mroute_vif_index] < 1) {
       if (PIM_DEBUG_MROUTE)
        {
-         char group_str[100];
-         char source_str[100];
+         char group_str[INET_ADDRSTRLEN];
+         char source_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
          pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
          zlog_debug("%s %s: new protocol mask %u requested nonexistent OIF %s (vif_index=%d, min_ttl=%d) for channel (S,G)=(%s,%s)",
@@ -384,8 +384,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   if (old_ttl > 0) {
     if (PIM_DEBUG_MROUTE)
       {
-       char group_str[100];
-       char source_str[100];
+       char group_str[INET_ADDRSTRLEN];
+       char source_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        zlog_debug("%s %s: interface %s (vif_index=%d) is existing output for channel (S,G)=(%s,%s)",
@@ -401,8 +401,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   if (pim_mroute_add(channel_oil)) {
     if (PIM_DEBUG_MROUTE)
       {
-       char group_str[100];
-       char source_str[100];
+       char group_str[INET_ADDRSTRLEN];
+       char source_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        zlog_debug("%s %s: could not add output interface %s (vif_index=%d) for channel (S,G)=(%s,%s)",
@@ -420,8 +420,8 @@ int pim_channel_add_oif(struct channel_oil *channel_oil,
   channel_oil->oif_flags[pim_ifp->mroute_vif_index] |= proto_mask;
 
   if (PIM_DEBUG_MROUTE) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_debug("%s %s: (S,G)=(%s,%s): proto_mask=%u OIF=%s vif_index=%d: DONE",
index b10528536eb15115ba8ffdc2274955b3a35db011..1e71b0ad781fea48e1bc4070a4a3508cce170458 100644 (file)
@@ -132,8 +132,8 @@ int pim_pim_packet(struct interface *ifp, uint8_t *buf, size_t len)
 {
   struct ip *ip_hdr;
   size_t ip_hlen; /* ip header length in bytes */
-  char src_str[100];
-  char dst_str[100];
+  char src_str[INET_ADDRSTRLEN];
+  char dst_str[INET_ADDRSTRLEN];
   uint8_t *pim_msg;
   int pim_msg_len;
   uint8_t pim_version;
@@ -318,8 +318,8 @@ static int pim_sock_read(struct thread *t)
   }
 
   if (PIM_DEBUG_PIM_PACKETS) {
-    char from_str[100];
-    char to_str[100];
+    char from_str[INET_ADDRSTRLEN];
+    char to_str[INET_ADDRSTRLEN];
     
     if (!inet_ntop(AF_INET, &from.sin_addr, from_str, sizeof(from_str)))
       sprintf(from_str, "<from?>");
@@ -337,8 +337,8 @@ static int pim_sock_read(struct thread *t)
 #ifdef PIM_CHECK_RECV_IFINDEX_SANITY
   /* ifindex sanity check */
   if (ifindex != (int) ifp->ifindex) {
-    char from_str[100];
-    char to_str[100];
+    char from_str[INET_ADDRSTRLEN];
+    char to_str[INET_ADDRSTRLEN];
     struct interface *recv_ifp;
 
     if (!inet_ntop(AF_INET, &from.sin_addr, from_str , sizeof(from_str)))
@@ -489,7 +489,7 @@ pim_msg_send_frame (int fd, char *buf, size_t len,
 
   while (sendto (fd, buf, len, MSG_DONTWAIT, dst, salen) < 0)
     {
-      char dst_str[100];
+      char dst_str[INET_ADDRSTRLEN];
 
       switch (errno)
        {
@@ -559,7 +559,7 @@ pim_msg_send(int fd, struct in_addr src,
   ip->ip_len = htons (sendlen);
 
   if (PIM_DEBUG_PIM_PACKETS) {
-    char dst_str[100];
+    char dst_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", dst, dst_str, sizeof(dst_str));
     zlog_debug("%s: to %s on %s: msg_size=%d checksum=%x",
               __PRETTY_FUNCTION__,
@@ -592,7 +592,7 @@ static int hello_send(struct interface *ifp,
   pim_ifp = ifp->info;
 
   if (PIM_DEBUG_PIM_HELLO) {
-    char dst_str[100];
+    char dst_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<dst?>", qpim_all_pim_routers_addr, dst_str, sizeof(dst_str));
     zlog_debug("%s: to %s on %s: holdt=%u prop_d=%u overr_i=%u dis_join_supp=%d dr_prio=%u gen_id=%08x addrs=%d",
               __PRETTY_FUNCTION__,
index 216e3390a61ee743bde3ff13e5e61519cae8b26f..62973bc9c1b593c2de6650b9304defa17c1b43ca 100644 (file)
@@ -156,7 +156,7 @@ pim_register_send (const uint8_t *buf, int buf_size, struct in_addr src, struct
 
   if (PIM_DEBUG_PIM_REG)
     {
-       char rp_str[100];
+       char rp_str[INET_ADDRSTRLEN];
        strcpy (rp_str, inet_ntoa (rpg->rpf_addr.u.prefix4));
        zlog_debug ("Sending %sRegister Packet to %s", null_register ? "NULL " : "", rp_str);
     }
@@ -259,7 +259,7 @@ pim_register_recv (struct interface *ifp,
 
   if (!pim_rp_check_is_my_ip_address (ip_hdr->ip_dst, dest_addr)) {
     if (PIM_DEBUG_PIM_REG) {
-      char dest[100];
+      char dest[INET_ADDRSTRLEN];
 
       pim_inet4_dump ("<dst?>", dest_addr, dest, sizeof(dest));
       zlog_debug ("%s: Received Register message for %s that I do not own", __func__,
@@ -270,7 +270,7 @@ pim_register_recv (struct interface *ifp,
 
   if (PIM_DEBUG_PIM_REG)
     {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
 
       pim_inet4_dump ("<src?>", src_addr, src_str, sizeof (src_str));
       zlog_debug ("Received Register message from %s on %s", src_str, ifp->name);
index 98fc2eaabd60716020757a53069eef8ae02f96d2..817837afbf0279d941efe48ce8e7a250e4b7b2ef 100644 (file)
@@ -516,9 +516,9 @@ pim_rp_check_rp (struct in_addr old, struct in_addr new)
   for (ALL_LIST_ELEMENTS_RO (qpim_rp_list, node, rp_info))
     {
       if (PIM_DEBUG_ZEBRA) {
-        char sold[100];
-        char snew[100];
-        char rp[100];
+        char sold[INET_ADDRSTRLEN];
+        char snew[INET_ADDRSTRLEN];
+        char rp[PREFIX_STRLEN];
         pim_addr_dump("<rp?>", &rp_info->rp.rpf_addr, rp, sizeof(rp));
         pim_inet4_dump("<old?>", old, sold, sizeof(sold));
         pim_inet4_dump("<new?>", new, snew, sizeof(snew));
index 46a6ef1ca409dddec2abf9996f1878f0a5563589..ffedd84103cebe213800bdd86a68b757bf1be01a 100644 (file)
@@ -51,7 +51,7 @@ int pim_nexthop_lookup(struct pim_nexthop *nexthop, struct in_addr addr, int nei
                                       MULTIPATH_NUM,
                                       addr, PIM_NEXTHOP_LOOKUP_MAX);
   if (num_ifindex < 1) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s %s: could not find nexthop ifindex for address %s",
              __FILE__, __PRETTY_FUNCTION__,
@@ -68,7 +68,7 @@ int pim_nexthop_lookup(struct pim_nexthop *nexthop, struct in_addr addr, int nei
         {
           if (PIM_DEBUG_ZEBRA)
             {
-              char addr_str[100];
+              char addr_str[INET_ADDRSTRLEN];
               pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
               zlog_debug("%s %s: could not find interface for ifindex %d (address %s)",
                          __FILE__, __PRETTY_FUNCTION__,
@@ -79,7 +79,7 @@ int pim_nexthop_lookup(struct pim_nexthop *nexthop, struct in_addr addr, int nei
 
       if (!ifp->info && PIM_DEBUG_ZEBRA)
         {
-          char addr_str[100];
+          char addr_str[INET_ADDRSTRLEN];
           pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
           zlog_debug("%s: multicast not enabled on input interface %s (ifindex=%d, RPF for source %s)",
                     __PRETTY_FUNCTION__,
@@ -105,8 +105,8 @@ int pim_nexthop_lookup(struct pim_nexthop *nexthop, struct in_addr addr, int nei
   if (found)
     {
       if (PIM_DEBUG_ZEBRA) {
-        char nexthop_str[100];
-        char addr_str[100];
+        char nexthop_str[PREFIX_STRLEN];
+        char addr_str[INET_ADDRSTRLEN];
         pim_addr_dump("<nexthop?>", &nexthop_tab[0].nexthop_addr, nexthop_str, sizeof(nexthop_str));
         pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
         zlog_debug("%s %s: found nexthop %s for address %s: interface %s ifindex=%d metric=%d pref=%d",
@@ -167,7 +167,7 @@ enum pim_rpf_result pim_rpf_update(struct pim_upstream *up, struct in_addr *old_
   if (nexthop_mismatch(&rpf->source_nexthop, &save_nexthop)) {
 
     if (PIM_DEBUG_ZEBRA) {
-      char nhaddr_str[100];
+      char nhaddr_str[PREFIX_STRLEN];
       pim_addr_dump("<addr?>", &rpf->source_nexthop.mrib_nexthop_addr, nhaddr_str, sizeof(nhaddr_str));
       zlog_debug("%s %s: (S,G)=%s source nexthop now is: interface=%s address=%s pref=%d metric=%d",
                 __FILE__, __PRETTY_FUNCTION__,
index 8ba514a219569f501ba2177edc416eb87b550aa8..681b5fe5c151e11690109b414c82c4e8ba98a3a7 100644 (file)
@@ -269,8 +269,8 @@ int pim_socket_join(int fd, struct in_addr group,
 
   ret = setsockopt(fd, IPPROTO_IP, IP_ADD_MEMBERSHIP, &opt, sizeof(opt));
   if (ret) {
-    char group_str[100];
-    char ifaddr_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char ifaddr_str[INET_ADDRSTRLEN];
     if (!inet_ntop(AF_INET, &group, group_str , sizeof(group_str)))
       sprintf(group_str, "<group?>");
     if (!inet_ntop(AF_INET, &ifaddr, ifaddr_str , sizeof(ifaddr_str)))
@@ -282,8 +282,8 @@ int pim_socket_join(int fd, struct in_addr group,
   }
 
   if (PIM_DEBUG_TRACE) {
-    char group_str[100];
-    char ifaddr_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char ifaddr_str[INET_ADDRSTRLEN];
     if (!inet_ntop(AF_INET, &group, group_str , sizeof(group_str)))
       sprintf(group_str, "<group?>");
     if (!inet_ntop(AF_INET, &ifaddr, ifaddr_str , sizeof(ifaddr_str)))
@@ -302,8 +302,8 @@ int pim_socket_join_source(int fd, ifindex_t ifindex,
                           const char *ifname)
 {
   if (pim_igmp_join_source(fd, ifindex, group_addr, source_addr)) {
-    char group_str[100];
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<grp?>", group_addr, group_str, sizeof(group_str));
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: setsockopt(fd=%d) failure for IGMP group %s source %s ifindex %d on interface %s: errno=%d: %s",
@@ -380,7 +380,7 @@ int pim_socket_recvfromto(int fd, uint8_t *buf, size_t len,
        *ifindex = i->ipi_ifindex;
 
       if (to && PIM_DEBUG_PACKETS) {
-       char to_str[100];
+       char to_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<to?>", to->sin_addr, to_str, sizeof(to_str));
        zlog_debug("%s: HAVE_IP_PKTINFO to=%s,%d",
                   __PRETTY_FUNCTION__,
@@ -400,7 +400,7 @@ int pim_socket_recvfromto(int fd, uint8_t *buf, size_t len,
        *tolen = sizeof(struct sockaddr_in);
 
       if (to && PIM_DEBUG_PACKETS) {
-       char to_str[100];
+       char to_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<to?>", to->sin_addr, to_str, sizeof(to_str));
        zlog_debug("%s: HAVE_IP_RECVDSTADDR to=%s,%d",
                   __PRETTY_FUNCTION__,
index 40873ca69f3bc1a745947f9c5f4376280eabf954..3d08552de8d187cc5a6a72988536987d09adfd22 100644 (file)
@@ -97,7 +97,7 @@ static int ssmpingd_socket(struct in_addr addr, int port, int mttl)
   sockaddr.sin_port   = htons(port);
 
   if (bind(fd, (struct sockaddr *)&sockaddr, sizeof(sockaddr))) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s: bind(fd=%d,addr=%s,port=%d,len=%zu) failure: errno=%d: %s",
              __PRETTY_FUNCTION__,
@@ -196,7 +196,7 @@ static void ssmpingd_delete(struct ssmpingd_sock *ss)
   THREAD_OFF(ss->t_sock_read);
 
   if (close(ss->sock_fd)) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: failure closing ssmpingd sock_fd=%d for source %s: errno=%d: %s",
              __PRETTY_FUNCTION__,
@@ -219,7 +219,7 @@ static void ssmpingd_sendto(struct ssmpingd_sock *ss,
   sent = sendto(ss->sock_fd, buf, len, MSG_DONTWAIT,
                 (struct sockaddr *)&to, tolen);
   if (sent != len) {
-    char to_str[100];
+    char to_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<to?>", to.sin_addr, to_str, sizeof(to_str));
     if (sent < 0) {
       zlog_warn("%s: sendto() failure to %s,%d: fd=%d len=%d: errno=%d: %s",
@@ -254,7 +254,7 @@ static int ssmpingd_read_msg(struct ssmpingd_sock *ss)
                              &to, &tolen,
                              &ifindex);
   if (len < 0) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: failure receiving ssmping for source %s on fd=%d: errno=%d: %s",
              __PRETTY_FUNCTION__, source_str, ss->sock_fd, errno, safe_strerror(errno));
@@ -264,9 +264,9 @@ static int ssmpingd_read_msg(struct ssmpingd_sock *ss)
   ifp = if_lookup_by_index(ifindex);
 
   if (buf[0] != PIM_SSMPINGD_REQUEST) {
-    char source_str[100];
-    char from_str[100];
-    char to_str[100];
+    char source_str[INET_ADDRSTRLEN];
+    char from_str[INET_ADDRSTRLEN];
+    char to_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
     pim_inet4_dump("<from?>", from.sin_addr, from_str, sizeof(from_str));
     pim_inet4_dump("<to?>", to.sin_addr, to_str, sizeof(to_str));
@@ -282,9 +282,9 @@ static int ssmpingd_read_msg(struct ssmpingd_sock *ss)
   }
 
   if (PIM_DEBUG_SSMPINGD) {
-    char source_str[100];
-    char from_str[100];
-    char to_str[100];
+    char source_str[INET_ADDRSTRLEN];
+    char from_str[INET_ADDRSTRLEN];
+    char to_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
     pim_inet4_dump("<from?>", from.sin_addr, from_str, sizeof(from_str));
     pim_inet4_dump("<to?>", to.sin_addr, to_str, sizeof(to_str));
@@ -356,7 +356,7 @@ static struct ssmpingd_sock *ssmpingd_new(struct in_addr source_addr)
 
   sock_fd = ssmpingd_socket(source_addr, /* port: */ 4321, /* mTTL: */ 64);
   if (sock_fd < 0) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: ssmpingd_socket() failure for source %s",
              __PRETTY_FUNCTION__, source_str);
@@ -365,7 +365,7 @@ static struct ssmpingd_sock *ssmpingd_new(struct in_addr source_addr)
 
   ss = XCALLOC(MTYPE_PIM_SSMPINGD, sizeof(*ss));
   if (!ss) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_err("%s: XCALLOC(%zu) failure for ssmpingd source %s",
             __PRETTY_FUNCTION__,
@@ -398,7 +398,7 @@ int pim_ssmpingd_start(struct in_addr source_addr)
   }
 
   {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_info("%s: starting ssmpingd for source %s",
              __PRETTY_FUNCTION__, source_str);
@@ -406,7 +406,7 @@ int pim_ssmpingd_start(struct in_addr source_addr)
 
   ss = ssmpingd_new(source_addr);
   if (!ss) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: ssmpingd_new() failure for source %s",
              __PRETTY_FUNCTION__, source_str);
@@ -422,7 +422,7 @@ int pim_ssmpingd_stop(struct in_addr source_addr)
 
   ss = ssmpingd_find(source_addr);
   if (!ss) {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_warn("%s: could not find ssmpingd for source %s",
              __PRETTY_FUNCTION__, source_str);
@@ -430,7 +430,7 @@ int pim_ssmpingd_stop(struct in_addr source_addr)
   }
 
   {
-    char source_str[100];
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", source_addr, source_str, sizeof(source_str));
     zlog_info("%s: stopping ssmpingd for source %s",
              __PRETTY_FUNCTION__, source_str);
index a24a3f92a834e4e85b92916775e712f95a796bea..443e7cacaa8a13bf49cf8b0d7682920cd1436fe0 100644 (file)
@@ -111,8 +111,8 @@ int pim_static_add(struct interface *iif, struct interface *oif, struct in_addr
           s_route->source.s_addr == source.s_addr) {
          if (s_route->iif == iif_index &&
              s_route->oif_ttls[oif_index]) {
-            char gifaddr_str[100];
-            char sifaddr_str[100];
+            char gifaddr_str[INET_ADDRSTRLEN];
+            char sifaddr_str[INET_ADDRSTRLEN];
             pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
             pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
             zlog_warn("%s %s: Unable to add static route: Route already exists (iif=%d,oif=%d,group=%s,source=%s)",
@@ -177,8 +177,8 @@ int pim_static_add(struct interface *iif, struct interface *oif, struct in_addr
 
    if (pim_mroute_add(&s_route->c_oil))
    {
-      char gifaddr_str[100];
-      char sifaddr_str[100];
+      char gifaddr_str[INET_ADDRSTRLEN];
+      char sifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
       pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
       zlog_warn("%s %s: Unable to add static route(iif=%d,oif=%d,group=%s,source=%s)",
@@ -210,8 +210,8 @@ int pim_static_add(struct interface *iif, struct interface *oif, struct in_addr
    }
 
    if (PIM_DEBUG_STATIC) {
-     char gifaddr_str[100];
-     char sifaddr_str[100];
+     char gifaddr_str[INET_ADDRSTRLEN];
+     char sifaddr_str[INET_ADDRSTRLEN];
      pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
      pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
      zlog_debug("%s: Static route added(iif=%d,oif=%d,group=%s,source=%s)",
@@ -255,8 +255,8 @@ int pim_static_del(struct interface *iif, struct interface *oif, struct in_addr
          /* If there are no more outputs then delete the whole route, otherwise set the route with the new outputs */
          if (s_route->c_oil.oil_ref_count <= 0 ?
                  pim_mroute_del(&s_route->c_oil) : pim_mroute_add(&s_route->c_oil)) {
-            char gifaddr_str[100];
-            char sifaddr_str[100];
+            char gifaddr_str[INET_ADDRSTRLEN];
+            char sifaddr_str[INET_ADDRSTRLEN];
             pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
             pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
             zlog_warn("%s %s: Unable to remove static route(iif=%d,oif=%d,group=%s,source=%s)",
@@ -281,8 +281,8 @@ int pim_static_del(struct interface *iif, struct interface *oif, struct in_addr
          }
 
          if (PIM_DEBUG_STATIC) {
-           char gifaddr_str[100];
-           char sifaddr_str[100];
+           char gifaddr_str[INET_ADDRSTRLEN];
+           char sifaddr_str[INET_ADDRSTRLEN];
            pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
            pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
            zlog_debug("%s: Static route removed(iif=%d,oif=%d,group=%s,source=%s)",
@@ -298,8 +298,8 @@ int pim_static_del(struct interface *iif, struct interface *oif, struct in_addr
    }
 
    if (!node) {
-      char gifaddr_str[100];
-      char sifaddr_str[100];
+      char gifaddr_str[INET_ADDRSTRLEN];
+      char sifaddr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<ifaddr?>", group, gifaddr_str, sizeof(gifaddr_str));
       pim_inet4_dump("<ifaddr?>", source, sifaddr_str, sizeof(sifaddr_str));
       zlog_warn("%s %s: Unable to remove static route: Route does not exist(iif=%d,oif=%d,group=%s,source=%s)",
@@ -320,8 +320,8 @@ pim_static_write_mroute (struct vty *vty, struct interface *ifp)
   struct listnode *node;
   struct static_route *sroute;
   int count = 0;
-  char sbuf[100];
-  char gbuf[100];
+  char sbuf[INET_ADDRSTRLEN];
+  char gbuf[INET_ADDRSTRLEN];
 
   for (ALL_LIST_ELEMENTS_RO (qpim_static_route_list, node, sroute))
     {
index 2dffe8c2b8908fed938147c0692c8e6afe579125..4c25c00294d54b4846f064de0241e1ed8230fae0 100644 (file)
@@ -65,8 +65,8 @@ void pim_inet4_dump(const char *onfail, struct in_addr addr, char *buf, int buf_
 char *
 pim_str_sg_dump (const struct prefix_sg *sg)
 {
-  char src_str[100];
-  char grp_str[100];
+  char src_str[INET_ADDRSTRLEN];
+  char grp_str[INET_ADDRSTRLEN];
   static char sg_str[200];
 
   pim_inet4_dump ("<src?>", sg->src, src_str, sizeof(src_str));
index 93227d633985a7e532db351a3f489805b40ff4db..d083cd8f40ee3c1f8fb75ae37cb764ea017f7e3d 100644 (file)
@@ -274,7 +274,7 @@ static int check_tlv_length(const char *label, const char *tlv_name,
                            int correct_len, int option_len)
 {
   if (option_len != correct_len) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: PIM hello %s TLV with incorrect value size=%d correct=%d from %s on interface %s",
              label, tlv_name,
@@ -293,7 +293,7 @@ static void check_tlv_redefinition_uint16(const char *label, const char *tlv_nam
                                          uint16_t new, uint16_t old)
 {
   if (PIM_OPTION_IS_SET(options, opt_mask)) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: PIM hello TLV redefined %s=%u old=%u from %s on interface %s",
              label, tlv_name,
@@ -309,7 +309,7 @@ static void check_tlv_redefinition_uint32(const char *label, const char *tlv_nam
                                          uint32_t new, uint32_t old)
 {
   if (PIM_OPTION_IS_SET(options, opt_mask)) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: PIM hello TLV redefined %s=%u old=%u from %s on interface %s",
              label, tlv_name,
@@ -325,7 +325,7 @@ static void check_tlv_redefinition_uint32_hex(const char *label, const char *tlv
                                              uint32_t new, uint32_t old)
 {
   if (PIM_OPTION_IS_SET(options, opt_mask)) {
-    char src_str[100];
+    char src_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
     zlog_warn("%s: PIM hello TLV redefined %s=%08x old=%08x from %s on interface %s",
              label, tlv_name,
@@ -683,7 +683,7 @@ int pim_tlv_parse_addr_list(const char *ifname, struct in_addr src_addr,
      */
     addr_offset = pim_parse_addr_ucast(&tmp, addr, pastend - addr);
     if (addr_offset < 1) {
-      char src_str[100];
+      char src_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
       zlog_warn("%s: pim_parse_addr_ucast() failure: from %s on %s",
                __PRETTY_FUNCTION__,
@@ -700,8 +700,8 @@ int pim_tlv_parse_addr_list(const char *ifname, struct in_addr src_addr,
       switch (tmp.family) {
       case AF_INET:
        {
-         char addr_str[100];
-         char src_str[100];
+         char addr_str[INET_ADDRSTRLEN];
+         char src_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<addr?>", tmp.u.prefix4, addr_str, sizeof(addr_str));
          pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
          zlog_debug("%s: PIM hello TLV option: list_old_size=%d IPv4 address %s from %s on %s",
@@ -713,7 +713,7 @@ int pim_tlv_parse_addr_list(const char *ifname, struct in_addr src_addr,
        break;
       default:
        {
-         char src_str[100];
+         char src_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
          zlog_debug("%s: PIM hello TLV option: list_old_size=%d UNKNOWN address family from %s on %s",
                     __PRETTY_FUNCTION__,
@@ -730,7 +730,7 @@ int pim_tlv_parse_addr_list(const char *ifname, struct in_addr src_addr,
      */
     if (tmp.family == AF_INET) {
       if (tmp.u.prefix4.s_addr == src_addr.s_addr) {
-         char src_str[100];
+         char src_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<src?>", src_addr, src_str, sizeof(src_str));
          zlog_warn("%s: ignoring primary address in secondary list from %s on %s",
                    __PRETTY_FUNCTION__,
index a7e40da9482c834b76a00e23d8d41f4d01a0c68f..192b6609edd797301c8a1402a7e70b3dcbee4717 100644 (file)
@@ -181,7 +181,7 @@ void
 pim_upstream_send_join (struct pim_upstream *up)
 {
   if (PIM_DEBUG_PIM_TRACE) {
-    char rpf_str[100];
+    char rpf_str[PREFIX_STRLEN];
     pim_addr_dump("<rpf?>", &up->rpf.rpf_addr, rpf_str, sizeof(rpf_str));
     zlog_debug ("%s: RPF'%s=%s(%s) for Interface %s", __PRETTY_FUNCTION__,
                pim_str_sg_dump (&up->sg), rpf_str, pim_upstream_state2str (up->join_state),
@@ -279,7 +279,7 @@ void pim_upstream_join_suppress(struct pim_upstream *up,
   join_timer_remain_msec = pim_time_timer_remain_msec(up->t_join_timer);
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char rpf_str[100];
+    char rpf_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<rpf?>", rpf_addr, rpf_str, sizeof(rpf_str));
     zlog_debug("%s %s: detected Join%s to RPF'(S,G)=%s: join_timer=%ld msec t_joinsuppress=%ld msec",
               __FILE__, __PRETTY_FUNCTION__, 
@@ -310,7 +310,7 @@ void pim_upstream_join_timer_decrease_to_t_override(const char *debug_label,
   t_override_msec = pim_if_t_override_msec(up->rpf.source_nexthop.interface);
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char rpf_str[100];
+    char rpf_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<rpf?>", rpf_addr, rpf_str, sizeof(rpf_str));
     zlog_debug("%s: to RPF'%s=%s: join_timer=%ld msec t_override=%d msec",
               debug_label,
@@ -692,8 +692,8 @@ void pim_upstream_rpf_genid_changed(struct in_addr neigh_addr)
   for (ALL_LIST_ELEMENTS(pim_upstream_list, up_node, up_nextnode, up)) {
 
     if (PIM_DEBUG_PIM_TRACE) {
-      char neigh_str[100];
-      char rpf_addr_str[100];
+      char neigh_str[INET_ADDRSTRLEN];
+      char rpf_addr_str[PREFIX_STRLEN];
       pim_inet4_dump("<neigh?>", neigh_addr, neigh_str, sizeof(neigh_str));
       pim_addr_dump("<rpf?>", &up->rpf.rpf_addr, rpf_addr_str, sizeof(rpf_addr_str));
       zlog_debug("%s: matching neigh=%s against upstream (S,G)=%s joined=%d rpf_addr=%s",
index 644fb8e69133fde69b6a2aed5436c8b0cec33f52..f5cdc2bd2c340bd96f38b59b6c5d2c44817f4e8f 100644 (file)
@@ -160,7 +160,7 @@ int pim_global_config_write(struct vty *vty)
     vty_out(vty, "!%s", VTY_NEWLINE);
     ++writes;
     for (ALL_LIST_ELEMENTS_RO(qpim_ssmpingd_list, node, ss)) {
-      char source_str[100];
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<src?>", ss->source_addr, source_str, sizeof(source_str));
       vty_out(vty, "ip ssmpingd %s%s", source_str, VTY_NEWLINE);
       ++writes;
@@ -247,8 +247,8 @@ int pim_interface_config_write(struct vty *vty)
        struct listnode *node;
        struct igmp_join *ij;
        for (ALL_LIST_ELEMENTS_RO(pim_ifp->igmp_join_list, node, ij)) {
-         char group_str[100];
-         char source_str[100];
+         char group_str[INET_ADDRSTRLEN];
+         char source_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<grp?>", ij->group_addr, group_str, sizeof(group_str));
          pim_inet4_dump("<src?>", ij->source_addr, source_str, sizeof(source_str));
          vty_out(vty, " ip igmp join %s %s%s",
index e8460044e12618a16eec212ffc9ac343f26ac741..eac618b4389f7afb29411dcec68ac30e59824471 100644 (file)
@@ -284,7 +284,7 @@ static int pim_zebra_if_address_add(int command, struct zclient *zclient,
        /* but we had a primary address already */
 
        char buf[BUFSIZ];
-       char old[100];
+       char old[INET_ADDRSTRLEN];
 
        prefix2str(p, buf, BUFSIZ);
        pim_inet4_dump("<old?>", primary_addr, old, sizeof(old));
@@ -436,8 +436,8 @@ pim_scan_individual_oil (struct channel_oil *c_oil)
     {
       if (PIM_DEBUG_ZEBRA)
         {
-          char source_str[100];
-          char group_str[100];
+          char source_str[INET_ADDRSTRLEN];
+          char group_str[INET_ADDRSTRLEN];
           pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
           pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
           zlog_debug("%s %s: could not find input interface(%d) for (S,G)=(%s,%s)",
@@ -461,8 +461,8 @@ pim_scan_individual_oil (struct channel_oil *c_oil)
     {
       struct interface *old_iif = pim_if_find_by_vif_index(c_oil->oil.mfcc_parent);
       struct interface *new_iif = pim_if_find_by_vif_index(input_iface_vif_index);
-      char source_str[100];
-      char group_str[100];
+      char source_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       zlog_debug("%s %s: (S,G)=(%s,%s) input interface changed from %s vif_index=%d to %s vif_index=%d",
@@ -478,8 +478,8 @@ pim_scan_individual_oil (struct channel_oil *c_oil)
       struct interface *new_iif = pim_if_find_by_vif_index(input_iface_vif_index);
 
       if (PIM_DEBUG_ZEBRA) {
-       char source_str[100];
-       char group_str[100];
+       char source_str[INET_ADDRSTRLEN];
+       char group_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        zlog_debug("%s %s: (S,G)=(%s,%s) new iif loops to existing oif: %s vif_index=%d",
@@ -501,8 +501,8 @@ pim_scan_individual_oil (struct channel_oil *c_oil)
       /* just log warning */
       struct interface *old_iif = pim_if_find_by_vif_index(old_vif_index);
       struct interface *new_iif = pim_if_find_by_vif_index(input_iface_vif_index);
-      char source_str[100];
-      char group_str[100]; 
+      char source_str[INET_ADDRSTRLEN];
+      char group_str[INET_ADDRSTRLEN]; 
       pim_inet4_dump("<source?>", c_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       pim_inet4_dump("<group?>", c_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       zlog_warn("%s %s: (S,G)=(%s,%s) failure updating input interface from %s vif_index=%d to %s vif_index=%d",
@@ -804,7 +804,7 @@ static int fib_lookup_if_vif_index(struct in_addr addr)
                                       MULTIPATH_NUM, addr,
                                       PIM_NEXTHOP_LOOKUP_MAX);
   if (num_ifindex < 1) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s %s: could not find nexthop ifindex for address %s",
              __FILE__, __PRETTY_FUNCTION__,
@@ -815,7 +815,7 @@ static int fib_lookup_if_vif_index(struct in_addr addr)
   first_ifindex = nexthop_tab[0].ifindex;
   
   if (num_ifindex > 1) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_info("%s %s: FIXME ignoring multiple nexthop ifindex'es num_ifindex=%d for address %s (using only ifindex=%d)",
               __FILE__, __PRETTY_FUNCTION__,
@@ -824,7 +824,7 @@ static int fib_lookup_if_vif_index(struct in_addr addr)
   }
   
   if (PIM_DEBUG_ZEBRA) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<ifaddr?>", addr, addr_str, sizeof(addr_str));
     zlog_debug("%s %s: found nexthop ifindex=%d (interface %s) for address %s",
               __FILE__, __PRETTY_FUNCTION__,
@@ -834,7 +834,7 @@ static int fib_lookup_if_vif_index(struct in_addr addr)
   vif_index = pim_if_find_vifindex_by_ifindex(first_ifindex);
 
   if (vif_index < 0) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s %s: low vif_index=%d < 1 nexthop for address %s",
              __FILE__, __PRETTY_FUNCTION__,
@@ -845,7 +845,7 @@ static int fib_lookup_if_vif_index(struct in_addr addr)
   zassert(qpim_mroute_oif_highest_vif_index < MAXVIFS);
 
   if (vif_index > qpim_mroute_oif_highest_vif_index) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s %s: high vif_index=%d > highest_vif_index=%d nexthop for address %s",
              __FILE__, __PRETTY_FUNCTION__,
@@ -878,8 +878,8 @@ static int del_oif(struct channel_oil *channel_oil,
   zassert(pim_ifp->mroute_vif_index <= qpim_mroute_oif_highest_vif_index);
 
   if (PIM_DEBUG_MROUTE) {
-    char group_str[100]; 
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN]; 
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_debug("%s %s: (S,G)=(%s,%s): proto_mask=%u OIF=%s vif_index=%d",
@@ -893,8 +893,8 @@ static int del_oif(struct channel_oil *channel_oil,
   if (!(channel_oil->oif_flags[pim_ifp->mroute_vif_index] & proto_mask)) {
     if (PIM_DEBUG_MROUTE)
       {
-       char group_str[100]; 
-       char source_str[100];
+       char group_str[INET_ADDRSTRLEN]; 
+       char source_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
        pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
        zlog_warn("%s %s: nonexistent protocol mask %u removed OIF %s (vif_index=%d, min_ttl=%d) from channel (S,G)=(%s,%s)",
@@ -917,8 +917,8 @@ static int del_oif(struct channel_oil *channel_oil,
     /* Check the OIF keeps existing before returning, and only log
        warning otherwise */
     if (channel_oil->oil.mfcc_ttls[pim_ifp->mroute_vif_index] < 1) {
-      char group_str[100]; 
-      char source_str[100];
+      char group_str[INET_ADDRSTRLEN]; 
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       zlog_warn("%s %s: protocol mask %u removing nonexistent OIF %s (vif_index=%d, min_ttl=%d) from channel (S,G)=(%s,%s)",
@@ -934,8 +934,8 @@ static int del_oif(struct channel_oil *channel_oil,
   old_ttl = channel_oil->oil.mfcc_ttls[pim_ifp->mroute_vif_index];
 
   if (old_ttl < 1) {
-    char group_str[100]; 
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN]; 
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_warn("%s %s: interface %s (vif_index=%d) is not output for channel (S,G)=(%s,%s)",
@@ -948,8 +948,8 @@ static int del_oif(struct channel_oil *channel_oil,
   channel_oil->oil.mfcc_ttls[pim_ifp->mroute_vif_index] = 0;
 
   if (pim_mroute_add(channel_oil)) {
-    char group_str[100]; 
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN]; 
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_warn("%s %s: could not remove output interface %s (vif_index=%d) from channel (S,G)=(%s,%s)",
@@ -966,8 +966,8 @@ static int del_oif(struct channel_oil *channel_oil,
   if (channel_oil->oil_size < 1) {
     if (pim_mroute_del(channel_oil)) {
       /* just log a warning in case of failure */
-      char group_str[100]; 
-      char source_str[100];
+      char group_str[INET_ADDRSTRLEN]; 
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
       pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
       zlog_warn("%s %s: failure removing OIL for channel (S,G)=(%s,%s)",
@@ -977,8 +977,8 @@ static int del_oif(struct channel_oil *channel_oil,
   }
 
   if (PIM_DEBUG_MROUTE) {
-    char group_str[100]; 
-    char source_str[100];
+    char group_str[INET_ADDRSTRLEN]; 
+    char source_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<group?>", channel_oil->oil.mfcc_mcastgrp, group_str, sizeof(group_str));
     pim_inet4_dump("<source?>", channel_oil->oil.mfcc_origin, source_str, sizeof(source_str));
     zlog_debug("%s %s: (S,G)=(%s,%s): proto_mask=%u OIF=%s vif_index=%d: DONE",
@@ -1026,7 +1026,7 @@ void igmp_source_forward_start(struct igmp_source *source)
 
     int input_iface_vif_index = fib_lookup_if_vif_index(vif_source);
     if (input_iface_vif_index < 1) {
-      char source_str[100];
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", source->source_addr, source_str, sizeof(source_str));
       zlog_warn("%s %s: could not find input interface for source %s",
                __FILE__, __PRETTY_FUNCTION__,
@@ -1160,9 +1160,9 @@ void pim_forward_start(struct pim_ifchannel *ch)
   struct pim_upstream *up = ch->upstream;
 
   if (PIM_DEBUG_PIM_TRACE) {
-    char source_str[100];
-    char group_str[100]; 
-    char upstream_str[100];
+    char source_str[INET_ADDRSTRLEN];
+    char group_str[INET_ADDRSTRLEN]; 
+    char upstream_str[INET_ADDRSTRLEN];
 
     pim_inet4_dump("<source?>", ch->sg.src, source_str, sizeof(source_str));
     pim_inet4_dump("<group?>", ch->sg.grp, group_str, sizeof(group_str));
@@ -1175,7 +1175,7 @@ void pim_forward_start(struct pim_ifchannel *ch)
   if (!up->channel_oil) {
     int input_iface_vif_index = fib_lookup_if_vif_index(up->upstream_addr);
     if (input_iface_vif_index < 1) {
-      char source_str[100];
+      char source_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<source?>", up->sg.src, source_str, sizeof(source_str));
       zlog_warn("%s %s: could not find input interface for source %s",
                __FILE__, __PRETTY_FUNCTION__,
index 4e6c76f0b0d3c5dcc48e513ef40c625bb6472999..cdaf04566da15602d1967825ff261c6287adb40b 100644 (file)
@@ -165,7 +165,7 @@ static int zclient_read_nexthop(struct zclient *zlookup,
   int i, err;
 
   if (PIM_DEBUG_ZEBRA) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_debug("%s: addr=%s", 
               __PRETTY_FUNCTION__,
@@ -197,8 +197,8 @@ static int zclient_read_nexthop(struct zclient *zlookup,
   raddr.s_addr = stream_get_ipv4(s);
 
   if (raddr.s_addr != addr.s_addr) {
-    char addr_str[100];
-    char raddr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
+    char raddr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     pim_inet4_dump("<raddr?>", raddr, raddr_str, sizeof(raddr_str));
     zlog_warn("%s: address mismatch: addr=%s raddr=%s", 
@@ -223,7 +223,7 @@ static int zclient_read_nexthop(struct zclient *zlookup,
 
     nexthop_type = stream_getc(s);
     if (num_ifindex >= tab_size) {
-      char addr_str[100];
+      char addr_str[INET_ADDRSTRLEN];
       pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
       zlog_warn("%s %s: found too many nexthop ifindexes (%d > %d) for address %s",
                __FILE__, __PRETTY_FUNCTION__,
@@ -262,7 +262,7 @@ static int zclient_read_nexthop(struct zclient *zlookup,
     default:
       /* do nothing */
       {
-       char addr_str[100];
+       char addr_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
        zlog_warn("%s %s: found non-ifindex nexthop type=%d for address %s",
                 __FILE__, __PRETTY_FUNCTION__,
@@ -284,7 +284,7 @@ zclient_lookup_nexthop_once (struct pim_zlookup_nexthop nexthop_tab[],
   int ret;
 
   if (PIM_DEBUG_ZEBRA) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_debug("%s: addr=%s", 
               __PRETTY_FUNCTION__,
@@ -344,7 +344,7 @@ zclient_lookup_nexthop (struct pim_zlookup_nexthop nexthop_tab[],
                                              tab_size, addr);
     if (num_ifindex < 1) {
       if (PIM_DEBUG_ZEBRA) {
-       char addr_str[100];
+       char addr_str[INET_ADDRSTRLEN];
        pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
        zlog_debug("%s %s: lookup=%d/%d: could not find nexthop ifindex for address %s",
                   __FILE__, __PRETTY_FUNCTION__,
@@ -376,7 +376,7 @@ zclient_lookup_nexthop (struct pim_zlookup_nexthop nexthop_tab[],
       if (lookup > 0) {
        /* Report non-recursive success after first lookup */
        if (PIM_DEBUG_ZEBRA) {
-         char addr_str[100];
+         char addr_str[INET_ADDRSTRLEN];
          pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
          zlog_debug("%s %s: lookup=%d/%d: found non-recursive ifindex=%d for address %s dist=%d met=%d",
                     __FILE__, __PRETTY_FUNCTION__,
@@ -397,8 +397,8 @@ zclient_lookup_nexthop (struct pim_zlookup_nexthop nexthop_tab[],
     }
 
     if (PIM_DEBUG_ZEBRA) {
-      char addr_str[100];
-      char nexthop_str[100];
+      char addr_str[INET_ADDRSTRLEN];
+      char nexthop_str[PREFIX_STRLEN];
       pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
       pim_addr_dump("<nexthop?>", &nexthop_addr, nexthop_str, sizeof(nexthop_str));
       zlog_debug("%s %s: lookup=%d/%d: zebra returned recursive nexthop %s for address %s dist=%d met=%d",
@@ -413,7 +413,7 @@ zclient_lookup_nexthop (struct pim_zlookup_nexthop nexthop_tab[],
   } /* for (max_lookup) */
 
   if (PIM_DEBUG_ZEBRA) {
-    char addr_str[100];
+    char addr_str[INET_ADDRSTRLEN];
     pim_inet4_dump("<addr?>", addr, addr_str, sizeof(addr_str));
     zlog_warn("%s %s: lookup=%d/%d: failure searching recursive nexthop ifindex for address %s",
              __FILE__, __PRETTY_FUNCTION__,