Navigation

Changeset 1107:96f2051215c8 in freeDiameter


Ignore:
Timestamp:
May 12, 2013, 11:39:07 PM (11 years ago)
Author:
Sebastien Decugis <sdecugis@freediameter.net>
Branch:
default
Phase:
public
Message:

Replaced calls to TRACE_sSA and sSA_DUMP_NODE* macros

Files:
6 edited

Legend:

Unmodified
Added
Removed

  • extensions/app_radgw/rgw_clients.c

    r1033 r1107  
    921921        /* Dump the entry in debug mode */
    922922        if (TRACE_BOOL(FULL + 1 )) {
     923                char sa_buf[sSA_DUMP_STRLEN];
     924                fd_sa_sdump_numeric(sa_buf, ip_port);
    923925                TRACE_DEBUG(FULL, "Adding %s:", (type == RGW_CLI_NAS) ? "NAS" : "PROXY"  );
    924                 TRACE_sSA(FD_LOG_DEBUG, FULL,    "\tIP : ", ip_port, NI_NUMERICHOST | NI_NUMERICSERV, "" );
     926                TRACE_DEBUG(FULL, "\tIP : %s", sa_buf );
    925927                TRACE_BUFFER(FD_LOG_DEBUG, FULL, "\tKey: [", *key, keylen, "]" );
    926928        }
    … …  
    941943       
    942944        if (ret == EEXIST) {
     945                char sa_buf[sSA_DUMP_STRLEN];
    943946                /* Check if the key is the same, then skip or return an error */
    944947                if ((keylen == prev->key.len ) && ( ! memcmp(*key, prev->key.data, keylen) ) && (type == prev->type)) {
    … …  
    950953                fd_log_error("ERROR: Conflicting RADIUS clients descriptions!");
    951954                TRACE_ERROR("Previous entry: %s", (prev->type == RGW_CLI_NAS) ? "NAS" : "PROXY");
    952                 TRACE_sSA(FD_LOG_ERROR, NONE,    "\tIP : ", prev->sa, NI_NUMERICHOST | NI_NUMERICSERV, "" );
     955                fd_sa_sdump_numeric(sa_buf, prev->sa);
     956                TRACE_ERROR("\tIP : %s", sa_buf);
    953957                TRACE_BUFFER(FD_LOG_ERROR, NONE, "\tKey: [", prev->key.data, prev->key.len, "]" );
    954958                TRACE_ERROR("Conflicting entry: %s", (type == RGW_CLI_NAS) ? "NAS" : "PROXY");
    955                 TRACE_sSA(FD_LOG_ERROR, NONE,    "\tIP : ", ip_port, NI_NUMERICHOST | NI_NUMERICSERV, "" );
     959                fd_sa_sdump_numeric(sa_buf, ip_port);
     960                TRACE_ERROR("\tIP : %s", sa_buf);
    956961                TRACE_BUFFER(FD_LOG_ERROR, NONE, "\tKey: [", *key, keylen, "]" );
    957962        }
    … …  
    969974       
    970975        for (ref = senti->next; ref != senti; ref = ref->next) {
     976                char sa_buf[sSA_DUMP_STRLEN];
    971977                client = (struct rgw_client *)ref;
    972978                /* TODO: use a fct param instead of hardcoded FD_LOG_DEBUG */
    973                 TRACE_sSA(FD_LOG_DEBUG, NONE,    "  - ", client->sa, NI_NUMERICHOST | NI_NUMERICSERV, (client->type == RGW_CLI_NAS) ? "" : " [PROXY]" );
     979                TODO("Replace with appropriate code");
     980/*              TRACE_sSA(FD_LOG_DEBUG, NONE,    "  - ", client->sa, NI_NUMERICHOST | NI_NUMERICSERV, (client->type == RGW_CLI_NAS) ? "" : " [PROXY]" ); */
    974981        }
    975982}

  • extensions/app_radgw/rgw_servers.c

    r1027 r1107  
    111111        while (1) {
    112112                struct sockaddr_storage from;
     113                char sa_buf[sSA_DUMP_STRLEN];
    113114                socklen_t fromlen = sizeof(from);
    114115                int len;
    … …  
    133134                }
    134135               
    135                 {
    136                         char __buf[1024];
    137                         sSA_DUMP_NODE_SERV(__buf, sizeof(__buf), &from, NI_NUMERICHOST | NI_NUMERICSERV );
    138                         TRACE_DEBUG(FULL, "Received %d bytes from %s", len, __buf);
    139                 }
     136                fd_sa_sdump_numeric(sa_buf, (sSA*)&from);
     137                TRACE_DEBUG(FULL, "Received %d bytes from %s", len, sa_buf);
    140138               
    141139                /* Search the associated client definition, if any */
    142140                CHECK_FCT_DO( rgw_clients_search((struct sockaddr *) &from, &nas_info),
    143141                        {
    144                                 TRACE_NOTICE("Discarding %d bytes received from unknown IP:", len);
    145                                 TRACE_sSA(FD_LOG_NOTICE, INFO, " ", &from, NI_NUMERICHOST | NI_NUMERICSERV, "" );
     142                                TRACE_NOTICE("Discarding %d bytes received from unknown IP: %s", len, sa_buf);
    146143                                continue;
    147144                        } );
    … …  
    250247        int ret = 0;
    251248        struct sockaddr_storage sto;
     249        char sa_buf[sSA_DUMP_STRLEN];
    252250       
    253251        /* Find the appropriate socket to use (not sure if it is important) */
    … …  
    276274        }
    277275       
    278         {
    279                 char __buf[1024];
    280                 sSA_DUMP_NODE_SERV(__buf, sizeof(__buf), &sto, NI_NUMERICHOST | NI_NUMERICSERV );
    281                 TRACE_DEBUG(FULL, "Sending %zd bytes to %s", buflen, __buf);
    282         }
     276        fd_sa_sdump_numeric(sa_buf, (sSA*)&sto);
     277        TRACE_DEBUG(FULL, "Sending %zd bytes to %s", buflen, sa_buf);
    283278               
    284279        /* Send */

  • include/freeDiameter/libfdproto.h

    r1103 r1107  
    473473static __inline__ void replace_me() MARK_DEPRECATED { }
    474474
    475 #define TRACE_sSA(...) replace_me();
    476 #define sSA_DUMP_NODE_SERV(...) replace_me();
    477 #define sSA_DUMP_NODE(...) replace_me();
    478475#define TRACE_BUFFER(...) replace_me();
    479476#define TRACE_NOTICE(...) replace_me();

  • libfdcore/config.c

    r1093 r1107  
    347347                        if ( (fd_g_config->cnf_flags.no_ip4 && (ep->sa.sa_family == AF_INET))
    348348                           ||(fd_g_config->cnf_flags.no_ip6 && (ep->sa.sa_family == AF_INET6)) ) {
     349                                char sa_buf[sSA_DUMP_STRLEN];;
    349350                                li = li->prev;
    350351                                fd_list_unlink(&ep->chain);
    351                                 if (TRACE_BOOL(INFO)) {
    352                                         char buf[1024];
    353                                         sSA_DUMP_NODE( buf, sizeof(buf), &ep->sa, NI_NUMERICHOST );
    354                                         fd_log_debug("Info: Removing local address conflicting with the flags no_IP / no_IP6 : %s", buf);
    355                                 }
     352                                fd_sa_sdump_numeric(sa_buf, &ep->sa);
     353                                LOG_N("Info: Removing local address conflicting with the flags no_IP / no_IP6 : %s", sa_buf);
    356354                                free(ep);
    357355                        }

  • libfdcore/core.c

    r1098 r1107  
    220220        /* Display configuration */
    221221        b = fd_conf_dump(&buf, &len, NULL);
    222         LOG_N("%s\n", b ?: "Error during configuration dump...");
     222        LOG_N("%s", b ?: "Error during configuration dump...");
    223223       
    224224        /* Display registered triggers for FDEV_TRIGGER */
    225225        b = fd_event_trig_dump(&buf, &len, &offset);
    226226        if (!b || offset) {
    227                 LOG_N("%s\n", b ?: "Error during triggers dump...");
     227                LOG_N("%s", b ?: "Error during triggers dump...");
    228228        }
    229229       

  • libfdcore/sctp.c

    r1102 r1107  
    739739                }
    740740               
    741                 if (TRACE_BOOL(SCTP_LEVEL)) {
     741                #if 0
    742742                        union {
    743743                                sSA     *sa;
    … …  
    751751                                ptr.buf += (ptr.sa->sa_family == AF_INET) ? sizeof(sSA4) : sizeof(sSA6) ;
    752752                        }
    753                 }
     753                #endif
    754754               
    755755                /* Bind to this array */
    … …  
    764764       
    765765        /* Debug: show all local listening addresses */
    766         if (TRACE_BOOL(SCTP_LEVEL)) {
     766        #if 0
    767767                sSA *sar;
    768768                union {
    … …  
    779779                }
    780780                sctp_freeladdrs(sar);
    781         }
     781        #endif
    782782
    783783        return 0;
    … …  
    830830       
    831831        /* Try connecting */
    832         if (TRACE_BOOL(FULL)) {
    833                 TRACE_DEBUG(FULL, "Attempting SCTP connection (%d addresses attempted) :", count);
     832        LOG_A("Attempting SCTP connection (%d addresses attempted) ", count);
     833               
     834#if 0
    834835                /* Dump the SAs */
    835836                union {
    … …  
    845846                        ptr.buf += (ptr.sa->sa_family == AF_INET) ? sizeof(sSA4) : sizeof(sSA6);
    846847                }
    847         }
     848#endif
    848849       
    849850#ifdef SCTP_CONNECTX_4_ARGS
    … …  
    908909                return ENOTSUP;
    909910        }
    910         if (TRACE_BOOL(SCTP_LEVEL)) {
    911                 char buf[1024];
    912                 sSA_DUMP_NODE_SERV(buf, sizeof(buf), &status.sstat_primary.spinfo_address, NI_NUMERICHOST | NI_NUMERICSERV );
     911#if 0
     912                char sa_buf[sSA_DUMP_STRLEN];
     913                fd_sa_sdump_numeric(sa_buf, &status.sstat_primary.spinfo_address);
    913914                fd_log_debug( "SCTP_STATUS : sstat_state                  : %i" , status.sstat_state);
    914915                fd_log_debug( "              sstat_rwnd                   : %u" , status.sstat_rwnd);
    … …  
    918919                fd_log_debug( "              sstat_outstrms               : %hu", status.sstat_outstrms);
    919920                fd_log_debug( "              sstat_fragmentation_point    : %u" , status.sstat_fragmentation_point);
    920                 fd_log_debug( "              sstat_primary.spinfo_address : %s" , buf);
     921                fd_log_debug( "              sstat_primary.spinfo_address : %s" , sa_buf);
    921922                fd_log_debug( "              sstat_primary.spinfo_state   : %d" , status.sstat_primary.spinfo_state);
    922923                fd_log_debug( "              sstat_primary.spinfo_cwnd    : %u" , status.sstat_primary.spinfo_cwnd);
    … …  
    924925                fd_log_debug( "              sstat_primary.spinfo_rto     : %u" , status.sstat_primary.spinfo_rto);
    925926                fd_log_debug( "              sstat_primary.spinfo_mtu     : %u" , status.sstat_primary.spinfo_mtu);
    926         }
     927#endif /* 0 */
    927928       
    928929        *in = status.sstat_instrms;
    … …  
    11501151                        case SCTP_PEER_ADDR_CHANGE:
    11511152                                TRACE_DEBUG(FULL, "Received SCTP_PEER_ADDR_CHANGE notification");
    1152                                 TRACE_sSA(FD_LOG_DEBUG, SCTP_LEVEL, "    intf_change : ", &(notif->sn_paddr_change.spc_aaddr), NI_NUMERICHOST | NI_NUMERICSERV, "" );
     1153                                /* TRACE_sSA(FD_LOG_DEBUG, SCTP_LEVEL, "    intf_change : ", &(notif->sn_paddr_change.spc_aaddr), NI_NUMERICHOST | NI_NUMERICSERV, "" ); */
    11531154                                TRACE_DEBUG(SCTP_LEVEL, "          state : %d", notif->sn_paddr_change.spc_state);
    11541155                                TRACE_DEBUG(SCTP_LEVEL, "          error : %d", notif->sn_paddr_change.spc_error);
Note: See TracChangeset for help on using the changeset viewer.