Changeset 5984


Ignore:
Timestamp:
May 9, 2019 8:18:18 AM (14 months ago)
Author:
nanang
Message:

Re #1298: Updated PJSIP* to use PJ_ERROR consistently.

Location:
pjproject/trunk/pjsip/src
Files:
10 edited

Legend:

Unmodified
Added
Removed
  • pjproject/trunk/pjsip/src/pjsip-simple/publishc.c

    r5373 r5984  
    804804        // cause accessing freed memory location. 
    805805        //--pubc->pending_tsx; 
    806         PJ_LOG(4,(THIS_FILE, "Error sending request, status=%d", status)); 
     806        PJ_PERROR(4,(THIS_FILE, status, "Error sending request")); 
    807807    } 
    808808 
  • pjproject/trunk/pjsip/src/pjsip-ua/sip_inv.c

    r5937 r5984  
    19481948    status = pjmedia_sdp_neg_negotiate(inv->pool_prov, inv->neg, 0); 
    19491949 
    1950     PJ_LOG(5,(inv->obj_name, "SDP negotiation done, status=%d", status)); 
     1950    PJ_PERROR(4,(inv->obj_name, status, "SDP negotiation done")); 
    19511951 
    19521952    if (mod_inv.cb.on_media_update && inv->notify) 
  • pjproject/trunk/pjsip/src/pjsip-ua/sip_reg.c

    r5373 r5984  
    14401440        regc->has_tsx = PJ_FALSE; 
    14411441 
    1442         PJ_LOG(4,(THIS_FILE, "Error sending request, status=%d", status)); 
     1442        PJ_PERROR(4,(THIS_FILE, status, "Error sending request")); 
    14431443    } 
    14441444 
  • pjproject/trunk/pjsip/src/pjsip/sip_endpoint.c

    r5971 r5984  
    524524    status = pjsip_resolver_create(endpt->pool, &endpt->resolver); 
    525525    if (status != PJ_SUCCESS) { 
    526         PJ_LOG(4, (THIS_FILE, "Error creating resolver instance")); 
     526        PJ_PERROR(4, (THIS_FILE, status, 
     527                      "Error creating resolver instance")); 
    527528        goto on_error; 
    528529    } 
     
    568569    pj_pool_release( endpt->pool ); 
    569570 
    570     PJ_LOG(4, (THIS_FILE, "Error creating endpoint")); 
     571    PJ_PERROR(4, (THIS_FILE, status, "Error creating endpoint")); 
    571572    return status; 
    572573} 
     
    580581    exit_cb *ecb; 
    581582 
    582     PJ_LOG(5, (THIS_FILE, "Destroying endpoing instance..")); 
     583    PJ_LOG(5, (THIS_FILE, "Destroying endpoint instance..")); 
    583584 
    584585    /* Phase 1: stop all modules */ 
  • pjproject/trunk/pjsip/src/pjsip/sip_resolve.c

    r5723 r5984  
    513513on_error: 
    514514    if (status != PJ_SUCCESS) { 
    515         char errmsg[PJ_ERR_MSG_SIZE]; 
    516         PJ_LOG(4,(THIS_FILE, "Failed to resolve '%.*s'. Err=%d (%s)", 
    517                              (int)target->addr.host.slen, 
    518                              target->addr.host.ptr, 
    519                              status, 
    520                              pj_strerror(status,errmsg,sizeof(errmsg)).ptr)); 
     515        PJ_PERROR(4,(THIS_FILE, status, 
     516                     "Failed to resolve '%.*s'", 
     517                     (int)target->addr.host.slen, 
     518                     target->addr.host.ptr)); 
    521519        (*cb)(status, token, NULL); 
    522520        return; 
     
    568566     
    569567    if (status != PJ_SUCCESS) { 
    570         char errmsg[PJ_ERR_MSG_SIZE]; 
    571  
    572         /* Log error */ 
    573         pj_strerror(status, errmsg, sizeof(errmsg)); 
    574         PJ_LOG(4,(query->objname, "DNS A record resolution failed: %s",  
    575                   errmsg)); 
     568        PJ_PERROR(4,(query->objname, status, 
     569                     "DNS A record resolution failed")); 
    576570 
    577571        query->last_error = status; 
     
    631625     
    632626    if (status != PJ_SUCCESS) { 
    633         char errmsg[PJ_ERR_MSG_SIZE]; 
    634  
    635         /* Log error */ 
    636         pj_strerror(status, errmsg, sizeof(errmsg)); 
    637         PJ_LOG(4,(query->objname, "DNS AAAA record resolution failed: %s",  
    638                   errmsg)); 
     627        PJ_PERROR(4,(query->objname, status, 
     628                     "DNS AAAA record resolution failed")); 
    639629 
    640630        query->last_error = status; 
     
    661651 
    662652    if (status != PJ_SUCCESS) { 
    663         char errmsg[PJ_ERR_MSG_SIZE]; 
    664  
    665         /* Log error */ 
    666         pj_strerror(status, errmsg, sizeof(errmsg)); 
    667         PJ_LOG(4,(query->objname, "DNS A/AAAA record resolution failed: %s", 
    668                   errmsg)); 
     653        PJ_PERROR(4,(query->objname, status, 
     654                     "DNS A/AAAA record resolution failed")); 
    669655 
    670656        /* Call the callback */ 
  • pjproject/trunk/pjsip/src/pjsip/sip_transaction.c

    r5909 r5984  
    776776     */ 
    777777    if (pj_hash_count(mod_tsx_layer.htable) != 0) { 
    778         if (pjsip_endpt_atexit(mod_tsx_layer.endpt, &tsx_layer_destroy) != 
    779             PJ_SUCCESS) 
    780         { 
    781             PJ_LOG(3,(THIS_FILE, "Failed to register transaction layer " 
    782                                  "module destroy.")); 
     778        pj_status_t status; 
     779        status = pjsip_endpt_atexit(mod_tsx_layer.endpt, &tsx_layer_destroy); 
     780        if (status != PJ_SUCCESS) { 
     781            PJ_PERROR(3,(THIS_FILE, status, 
     782                    "Failed to register transaction layer module destroy.")); 
    783783        } 
    784784        return PJ_EBUSY; 
     
    20382038    if (sent < 0) { 
    20392039        pj_time_val delay = {0, 0}; 
    2040         char errmsg[PJ_ERR_MSG_SIZE]; 
    2041  
    2042         pj_strerror((pj_status_t)-sent, errmsg, sizeof(errmsg)); 
    2043  
    2044         PJ_LOG(2,(tsx->obj_name, "Transport failed to send %s! Err=%d (%s)", 
    2045                 pjsip_tx_data_get_info(tdata), -sent, errmsg)); 
     2040 
     2041        PJ_PERROR(2,(tsx->obj_name, (pj_status_t)-sent, 
     2042                      "Transport failed to send %s!", 
     2043                      pjsip_tx_data_get_info(tdata))); 
    20462044 
    20472045        /* Post the event for later processing, to avoid deadlock. 
  • pjproject/trunk/pjsip/src/pjsip/sip_transport.c

    r5971 r5984  
    15071507             * just fallback using pj_gethostip(), see ticket #1660. 
    15081508             */ 
    1509             PJ_LOG(5,(THIS_FILE,"Warning: unable to determine local " 
    1510                                 "interface, fallback to default interface!")); 
     1509            PJ_PERROR(5,(THIS_FILE, status, 
     1510                         "Warning: unable to determine local interface, " 
     1511                         "fallback to default interface!")); 
    15111512            status = pj_gethostip(af, &itf_addr); 
    15121513            if (status != PJ_SUCCESS) 
  • pjproject/trunk/pjsip/src/pjsip/sip_transport_tcp.c

    r5971 r5984  
    167167                       pj_status_t status) 
    168168{ 
    169     char errmsg[PJ_ERR_MSG_SIZE]; 
    170  
    171     pj_strerror(status, errmsg, sizeof(errmsg)); 
    172  
    173     PJ_LOG(3,(sender, "%s: %s [code=%d]", title, errmsg, status)); 
     169    PJ_PERROR(3,(sender, status, "%s: [code=%d]", title, status)); 
    174170} 
    175171 
     
    966962                                       readbuf, 0); 
    967963    if (status != PJ_SUCCESS && status != PJ_EPENDING) { 
    968         PJ_LOG(4, (tcp->base.obj_name,  
    969                    "pj_activesock_start_read() error, status=%d",  
    970                    status)); 
     964        PJ_PERROR(4, (tcp->base.obj_name, status, 
     965                      "pj_activesock_start_read() error")); 
    971966        return status; 
    972967    } 
     
    16871682                                    &enabled, sizeof(enabled)); 
    16881683        if (status != PJ_SUCCESS) { 
    1689             PJ_LOG(1, ("TRACE", "fail set reuseaddr")); 
    16901684            PJ_PERROR(4, (listener->factory.obj_name, status, 
    1691                 "Warning: error applying SO_REUSEADDR")); 
     1685                          "Warning: error applying SO_REUSEADDR")); 
    16921686        } 
    16931687    } 
  • pjproject/trunk/pjsip/src/pjsip/sip_transport_tls.c

    r5971 r5984  
    173173                       pj_status_t status) 
    174174{ 
    175     char errmsg[PJ_ERR_MSG_SIZE]; 
    176  
    177     pj_strerror(status, errmsg, sizeof(errmsg)); 
    178  
    179     PJ_LOG(3,(sender, "%s: %s [code=%d]", title, errmsg, status)); 
     175    PJ_PERROR(3,(sender, status, "%s: [code=%d]", title, status)); 
    180176} 
    181177 
     
    11061102                                     readbuf, 0); 
    11071103    if (status != PJ_SUCCESS && status != PJ_EPENDING) { 
    1108         PJ_LOG(4, (tls->base.obj_name,  
    1109                    "pj_ssl_sock_start_read() error, status=%d",  
    1110                    status)); 
     1104        PJ_PERROR(4, (tls->base.obj_name, status, 
     1105                     "pj_ssl_sock_start_read() error")); 
    11111106        return status; 
    11121107    } 
  • pjproject/trunk/pjsip/src/pjsip/sip_transport_udp.c

    r5971 r5984  
    645645                                &sobuf_size, sizeof(sobuf_size)); 
    646646    if (status != PJ_SUCCESS) { 
    647         char errmsg[PJ_ERR_MSG_SIZE]; 
    648         pj_strerror(status, errmsg, sizeof(errmsg)); 
    649         PJ_LOG(4,(THIS_FILE, "Error setting SO_RCVBUF: %s [%d]", errmsg, 
    650                   status)); 
     647        PJ_PERROR(4,(THIS_FILE, status, "Error setting SO_RCVBUF")); 
    651648    } 
    652649#endif 
     
    658655                                &sobuf_size, sizeof(sobuf_size)); 
    659656    if (status != PJ_SUCCESS) { 
    660         char errmsg[PJ_ERR_MSG_SIZE]; 
    661         pj_strerror(status, errmsg, sizeof(errmsg)); 
    662         PJ_LOG(4,(THIS_FILE, "Error setting SO_SNDBUF: %s [%d]", errmsg, 
    663                   status)); 
     657        PJ_PERROR(4,(THIS_FILE, status, "Error setting SO_SNDBUF")); 
    664658    } 
    665659#endif 
Note: See TracChangeset for help on using the changeset viewer.