Changeset 5980


Ignore:
Timestamp:
May 9, 2019 4:35:41 AM (5 years ago)
Author:
nanang
Message:

Re #1298: Updated PJLIB to use PJ_ERROR consistently.

Location:
pjproject/trunk/pjlib
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • pjproject/trunk/pjlib/include/pj/pool.h

    r5533 r5980  
    178178   static void my_perror(const char *title, pj_status_t status) 
    179179   { 
    180         char errmsg[PJ_ERR_MSG_SIZE]; 
    181  
    182         pj_strerror(status, errmsg, sizeof(errmsg)); 
    183         PJ_LOG(1,(THIS_FILE, "%s: %s [status=%d]", title, errmsg, status)); 
     180        PJ_PERROR(1,(THIS_FILE, status, title)); 
    184181   } 
    185182 
  • pjproject/trunk/pjlib/src/pj/ioqueue_select.c

    r5849 r5980  
    751751            status = pj_sock_getsockname(old_sock, &local_addr, &addr_len); 
    752752            if (status != PJ_SUCCESS) { 
    753                 PJ_LOG(5,(THIS_FILE, "Error get socket name %d", status)); 
     753                PJ_PERROR(5,(THIS_FILE, status, "Error get socket name")); 
    754754                continue; 
    755755            } 
     
    758758            status = pj_sock_getpeername(old_sock, &rem_addr, &addr_len); 
    759759            if (status != PJ_SUCCESS) { 
    760                 PJ_LOG(5,(THIS_FILE, "Error get peer name %d", status)); 
     760                PJ_PERROR(5,(THIS_FILE, status, "Error get peer name")); 
    761761            } else { 
    762762                flags |= HAS_PEER_ADDR; 
     
    767767                status == PJ_STATUS_FROM_OS(EINVAL)) 
    768768            { 
    769                 PJ_LOG(5,(THIS_FILE, "Error get qos param %d", status)); 
     769                PJ_PERROR(5,(THIS_FILE, status, "Error get qos param")); 
    770770                continue; 
    771771            } 
    772772         
    773773            if (status != PJ_SUCCESS) { 
    774                 PJ_LOG(5,(THIS_FILE, "Error get qos param %d", status)); 
     774                PJ_PERROR(5,(THIS_FILE, status, "Error get qos param")); 
    775775            } else { 
    776776                flags |= HAS_QOS; 
     
    782782            status = pj_sock_close(old_sock); 
    783783            if (status != PJ_SUCCESS) { 
    784                 PJ_LOG(5,(THIS_FILE, "Error closing socket %d", status)); 
     784                PJ_PERROR(5,(THIS_FILE, status, "Error closing socket")); 
    785785            } 
    786786             
     
    792792                                &new_sock); 
    793793        if (status != PJ_SUCCESS) { 
    794             PJ_LOG(5,(THIS_FILE, "Error create socket %d", status)); 
     794            PJ_PERROR(5,(THIS_FILE, status, "Error create socket")); 
    795795            continue; 
    796796        } 
     
    805805            status == PJ_STATUS_FROM_OS(EINVAL)) 
    806806        { 
    807             PJ_LOG(5,(THIS_FILE, "Error set socket option %d", 
    808                       status)); 
     807            PJ_PERROR(5,(THIS_FILE, status, "Error set socket option")); 
    809808            continue; 
    810809        } 
     
    826825            status = pj_sock_set_qos_params(new_sock, &qos_params); 
    827826            if (status == PJ_STATUS_FROM_OS(EINVAL)) { 
    828                 PJ_LOG(5,(THIS_FILE, "Error set qos param %d", status)); 
     827                PJ_PERROR(5,(THIS_FILE, status, "Error set qos param")); 
    829828                continue; 
    830829            } 
     
    834833            status = pj_sock_connect(new_sock, &rem_addr, addr_len); 
    835834            if (status != PJ_SUCCESS) { 
    836                 PJ_LOG(5,(THIS_FILE, "Error connect socket %d", status)); 
     835                PJ_PERROR(5,(THIS_FILE, status, "Error connect socket")); 
    837836                continue; 
    838837            } 
     
    889888 
    890889    h->fd = PJ_INVALID_SOCKET; 
    891     PJ_PERROR(1,(THIS_FILE, status, "Error replacing socket [%d]", status)); 
     890    PJ_PERROR(1,(THIS_FILE, status, "Error replacing socket %d", old_sock)); 
    892891    pj_lock_release(h->ioqueue->lock); 
    893892    return PJ_ESOCKETSTOP; 
  • pjproject/trunk/pjlib/src/pj/ssl_sock_imp_common.c

    r5975 r5980  
    226226            /* Handshake failed in accepting, destroy our self silently. */ 
    227227 
    228             char errmsg[PJ_ERR_MSG_SIZE]; 
    229228            char buf[PJ_INET6_ADDRSTRLEN+10]; 
    230229 
    231             pj_strerror(status, errmsg, sizeof(errmsg)); 
    232             PJ_LOG(3,(ssock->pool->obj_name, "Handshake failed in accepting " 
    233                       "%s: %s", 
    234                       pj_sockaddr_print(&ssock->rem_addr, buf, sizeof(buf), 3), 
    235                       errmsg)); 
     230            PJ_PERROR(3,(ssock->pool->obj_name, status, 
     231                         "Handshake failed in accepting %s", 
     232                         pj_sockaddr_print(&ssock->rem_addr, buf, 
     233                                           sizeof(buf), 3))); 
    236234 
    237235            if (ssock->param.cb.on_accept_complete2) { 
     
    251249            if (ssock->param.timer_heap) { 
    252250                pj_time_val interval = {0, PJ_SSL_SOCK_DELAYED_CLOSE_TIMEOUT}; 
     251                pj_status_t status1; 
    253252 
    254253                ssock->ssl_state = SSL_STATE_NULL; 
     
    261260                ssock->timer.id = TIMER_CLOSE; 
    262261                pj_time_val_normalize(&interval); 
    263                 if (pj_timer_heap_schedule(ssock->param.timer_heap,  
    264                                            &ssock->timer, &interval) != 0) 
    265                 { 
    266                     PJ_LOG(3,(ssock->pool->obj_name, "Failed to schedule " 
    267                               "a delayed close. Race condition may occur.")); 
     262                status1 = pj_timer_heap_schedule(ssock->param.timer_heap,  
     263                                                 &ssock->timer, &interval); 
     264                if (status1 != PJ_SUCCESS) { 
     265                    PJ_PERROR(3,(ssock->pool->obj_name, status, 
     266                                 "Failed to schedule a delayed close. " 
     267                                 "Race condition may occur.")); 
    268268                    ssock->timer.id = TIMER_NONE; 
    269269                    pj_ssl_sock_close(ssock); 
  • pjproject/trunk/pjlib/src/pj/ssl_sock_ossl.c

    r5941 r5980  
    804804                status = GET_SSL_STATUS(ssock); 
    805805                if (cert->CA_file.slen) { 
    806                     PJ_LOG(1,(ssock->pool->obj_name, 
    807                               "Error loading CA list file '%s'", 
    808                               cert->CA_file.ptr)); 
     806                    PJ_PERROR(1,(ssock->pool->obj_name, status, 
     807                                "Error loading CA list file '%s'", 
     808                                cert->CA_file.ptr)); 
    809809                } 
    810810                if (cert->CA_path.slen) { 
    811                     PJ_LOG(1,(ssock->pool->obj_name, 
    812                               "Error loading CA path '%s'", 
    813                               cert->CA_path.ptr)); 
     811                    PJ_PERROR(1,(ssock->pool->obj_name, status, 
     812                                "Error loading CA path '%s'", 
     813                                cert->CA_path.ptr)); 
    814814                } 
    815815                SSL_CTX_free(ctx); 
     
    833833            if(rc != 1) { 
    834834                status = GET_SSL_STATUS(ssock); 
    835                 PJ_LOG(1,(ssock->pool->obj_name, "Error loading certificate " 
    836                           "chain file '%s'", cert->cert_file.ptr)); 
     835                PJ_PERROR(1,(ssock->pool->obj_name, status, 
     836                             "Error loading certificate chain file '%s'", 
     837                             cert->cert_file.ptr)); 
    837838                SSL_CTX_free(ctx); 
    838839                return status; 
     
    849850            if(rc != 1) { 
    850851                status = GET_SSL_STATUS(ssock); 
    851                 PJ_LOG(1,(ssock->pool->obj_name, "Error adding private key " 
    852                           "from '%s'", cert->privkey_file.ptr)); 
     852                PJ_PERROR(1,(ssock->pool->obj_name, status, 
     853                             "Error adding private key from '%s'", 
     854                             cert->privkey_file.ptr)); 
    853855                SSL_CTX_free(ctx); 
    854856                return status; 
     
    892894                    if (rc != 1) { 
    893895                        status = GET_SSL_STATUS(ssock); 
    894                         PJ_LOG(1, (ssock->pool->obj_name, "Error loading " 
    895                                    "chain certificate from buffer")); 
     896                        PJ_PERROR(1,(ssock->pool->obj_name, status, 
     897                              "Error loading chain certificate from buffer")); 
    896898                        X509_free(xcert); 
    897899                        BIO_free(cbio); 
     
    940942                    if (rc != 1) { 
    941943                        status = GET_SSL_STATUS(ssock); 
    942                         PJ_LOG(1, (ssock->pool->obj_name, "Error adding " 
    943                                    "private key from buffer")); 
     944                        PJ_PERROR(1,(ssock->pool->obj_name, status, 
     945                                     "Error adding private key from buffer")); 
    944946                        EVP_PKEY_free(pkey); 
    945947                        BIO_free(kbio); 
     
    10041006                              "Additional certificate '%s' loaded.", cf)); 
    10051007                } else { 
    1006                     pj_perror(1, ssock->pool->obj_name, GET_SSL_STATUS(ssock), 
    1007                               "Error loading certificate file '%s'", cf); 
     1008                    PJ_PERROR(1,(ssock->pool->obj_name, GET_SSL_STATUS(ssock), 
     1009                                 "Error loading certificate file '%s'", cf)); 
    10081010                    ERR_clear_error(); 
    10091011                } 
     
    13061308static void set_entropy(pj_ssl_sock_t *ssock) 
    13071309{ 
    1308     int ret; 
     1310    int ret = 0; 
    13091311 
    13101312    switch (ssock->param.entropy_type) { 
     
    13241326            break; 
    13251327        case PJ_SSL_ENTROPY_NONE: 
    1326             default: 
    1327             return; 
     1328        default: 
    13281329            break; 
    13291330    } 
    13301331 
    13311332    if (ret < 0) { 
    1332         PJ_LOG(3, (ssock->pool->obj_name, "SSL failed to reseed with " 
    1333                                           "entropy type %d", 
    1334                                           ssock->param.entropy_type)); 
     1333        PJ_LOG(3, (ssock->pool->obj_name, 
     1334                   "SSL failed to reseed with entropy type %d " 
     1335                   "[native err=%d]", 
     1336                   ssock->param.entropy_type, ret)); 
    13351337    } 
    13361338} 
Note: See TracChangeset for help on using the changeset viewer.