Navigation

Changeset 1119:79dd22145f52 in freeDiameter for libfdcore/p_ce.c


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

Fix a number of compilation warnings

File:
1 edited

Legend:

Unmodified
Added
Removed

  • libfdcore/p_ce.c

    r1113 r1119  
    275275                if (hdr->avp_flags & AVP_FLAG_VENDOR) {
    276276                        /* Ignore all vendor-specific AVPs in CER/CEA because we don't support any currently */
    277                         TRACE_DEBUG(FULL, "Ignored a vendor AVP in CER / CEA");
    278                         fd_msg_dump_one(FULL, avp);
     277                        LOG_A("Ignored a vendor-specific AVP in CER / CEA");
    279278                        goto next;
    280279                }
    … …  
    284283                                if (hdr->avp_value == NULL) {
    285284                                        /* This is a sanity check */
    286                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    287                                         fd_msg_dump_one(NONE, avp);
     285                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    288286                                        ASSERT(0); /* To check if this really happens, and understand why... */
    289287                                        goto next;
    … …  
    297295                                if (hdr->avp_value == NULL) {
    298296                                        /* This is a sanity check */
    299                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    300                                         fd_msg_dump_one(NONE, avp);
     297                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    301298                                        ASSERT(0); /* To check if this really happens, and understand why... */
    302299                                        goto next;
    … …  
    319316                                if (hdr->avp_value == NULL) {
    320317                                        /* This is a sanity check */
    321                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    322                                         fd_msg_dump_one(NONE, avp);
     318                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    323319                                        ASSERT(0); /* To check if this really happens, and understand why... */
    324320                                        goto next;
    … …  
    350346                                if (hdr->avp_value == NULL) {
    351347                                        /* This is a sanity check */
    352                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    353                                         fd_msg_dump_one(NONE, avp);
     348                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    354349                                        ASSERT(0); /* To check if this really happens, and understand why... */
    355350                                        goto next;
    … …  
    376371                                if (hdr->avp_value == NULL) {
    377372                                        /* This is a sanity check */
    378                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    379                                         fd_msg_dump_one(NONE, avp);
     373                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    380374                                        ASSERT(0); /* To check if this really happens, and understand why... */
    381375                                        goto next;
    … …  
    397391                                if (hdr->avp_value == NULL) {
    398392                                        /* This is a sanity check */
    399                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    400                                         fd_msg_dump_one(NONE, avp);
     393                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    401394                                        ASSERT(0); /* To check if this really happens, and understand why... */
    402395                                        goto next;
    … …  
    419412                                if (hdr->avp_value == NULL) {
    420413                                        /* This is a sanity check */
    421                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    422                                         fd_msg_dump_one(NONE, avp);
     414                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    423415                                        ASSERT(0); /* To check if this really happens, and understand why... */
    424416                                        goto next;
    … …  
    440432                                if (hdr->avp_value == NULL) {
    441433                                        /* This is a sanity check */
    442                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    443                                         fd_msg_dump_one(NONE, avp);
     434                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    444435                                        ASSERT(0); /* To check if this really happens, and understand why... */
    445436                                        goto next;
    … …  
    468459
    469460                                                if (inhdr->avp_flags & AVP_FLAG_VENDOR) {
    470                                                         TRACE_DEBUG(FULL, "Ignored a vendor AVP inside Vendor-Specific-Application-Id AVP");
    471                                                         fd_msg_dump_one(FULL, avp);
     461                                                        LOG_A("Ignored a vendor AVP inside Vendor-Specific-Application-Id AVP");
    472462                                                        goto innext;
    473463                                                }
    … …  
    475465                                                if (inhdr->avp_value == NULL) {
    476466                                                        /* This is a sanity check */
    477                                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    478                                                         fd_msg_dump_one(NONE, avp);
     467                                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    479468                                                        ASSERT(0); /* To check if this really happens, and understand why... */
    480469                                                        goto innext;
    … …  
    508497                                               
    509498                                                if (invalid) {
    510                                                         TRACE_DEBUG(FULL, "Invalid Vendor-Specific-Application-Id AVP received, ignored");
    511                                                         fd_msg_dump_one(FULL, avp);
     499                                                        TRACE_DEBUG(FULL, "Invalid Vendor-Specific-Application-Id AVP received");
    512500                                                        error->pei_errcode = "DIAMETER_INVALID_AVP_VALUE";
    513501                                                        error->pei_avp = avp;
    … …  
    533521                                if (hdr->avp_value == NULL) {
    534522                                        /* This is a sanity check */
    535                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    536                                         fd_msg_dump_one(NONE, avp);
     523                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    537524                                        ASSERT(0); /* To check if this really happens, and understand why... */
    538525                                        goto next;
    … …  
    549536                                if (hdr->avp_value == NULL) {
    550537                                        /* This is a sanity check */
    551                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    552                                         fd_msg_dump_one(NONE, avp);
     538                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    553539                                        ASSERT(0); /* To check if this really happens, and understand why... */
    554540                                        goto next;
    … …  
    566552                                if (hdr->avp_value == NULL) {
    567553                                        /* This is a sanity check */
    568                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    569                                         fd_msg_dump_one(NONE, avp);
     554                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    570555                                        ASSERT(0); /* To check if this really happens, and understand why... */
    571556                                        goto next;
    … …  
    578563                                if (hdr->avp_value == NULL) {
    579564                                        /* This is a sanity check */
    580                                         TRACE_DEBUG(NONE, "Ignored an AVP with unset value in CER/CEA");
    581                                         fd_msg_dump_one(NONE, avp);
     565                                        LOG_F("Ignored an AVP (code %x) with unset value in CER/CEA", hdr->avp_code);
    582566                                        ASSERT(0); /* To check if this really happens, and understand why... */
    583567                                        goto next;
Note: See TracChangeset for help on using the changeset viewer.