Changeset 2885


Ignore:
Timestamp:
Aug 17, 2009 9:06:31 AM (10 years ago)
Author:
nanang
Message:

Misc fix (ticket #915):

  • Removed self detach() in destroy() in transport SRTP.
  • Added few more assertion checks in transport SRTP.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • pjproject/trunk/pjmedia/src/pjmedia/transport_srtp.c

    r2832 r2885  
    333333    unsigned i; 
    334334 
     335    pj_assert(opt); 
     336 
    335337    pj_bzero(opt, sizeof(pjmedia_srtp_setting)); 
    336338    opt->close_member_tp = PJ_TRUE; 
     
    358360    unsigned i; 
    359361 
    360     PJ_ASSERT_RETURN(endpt && p_tp, PJ_EINVAL); 
     362    PJ_ASSERT_RETURN(endpt && tp && p_tp, PJ_EINVAL); 
    361363 
    362364    /* Check crypto availability */ 
     
    459461    int              au_rx_idx = 0; 
    460462    int              crypto_suites_cnt; 
     463 
     464    PJ_ASSERT_RETURN(tp && tx && rx, PJ_EINVAL); 
    461465 
    462466    if (srtp->session_inited) { 
     
    586590    err_status_t err; 
    587591 
     592    PJ_ASSERT_RETURN(srtp, PJ_EINVAL); 
     593 
    588594    if (!p_srtp->session_inited) 
    589595        return PJ_SUCCESS; 
     
    659665    pj_status_t status; 
    660666 
     667    PJ_ASSERT_RETURN(tp && rem_addr && addr_len, PJ_EINVAL); 
     668 
    661669    /* Save the callbacks */ 
    662670    srtp->rtp_cb = rtp_cb; 
     
    779787{ 
    780788    transport_srtp *srtp = (transport_srtp *) tp; 
     789     
     790    PJ_ASSERT_RETURN(tp, PJ_EINVAL); 
    781791 
    782792    return pjmedia_transport_simulate_lost(srtp->member_tp, dir, pct_lost); 
     
    788798    pj_status_t status; 
    789799 
     800    PJ_ASSERT_RETURN(tp, PJ_EINVAL); 
     801 
    790802    pj_lock_acquire(srtp->mutex); 
    791803 
    792     pjmedia_transport_detach(tp, NULL); 
    793      
    794804    if (srtp->setting.close_member_tp && srtp->member_tp) { 
    795805        pjmedia_transport_close(srtp->member_tp); 
     
    10871097 
    10881098        pjmedia_sdp_media *m_rem; 
    1089          
     1099 
    10901100        m_rem = sdp_remote->media[media_index]; 
    10911101 
     
    14901500    pj_status_t status; 
    14911501 
     1502    PJ_ASSERT_RETURN(tp, PJ_EINVAL); 
     1503 
    14921504    status = pjmedia_transport_media_stop(srtp->member_tp); 
    14931505    if (status != PJ_SUCCESS) 
     
    15101522        return PJ_SUCCESS; 
    15111523 
    1512     PJ_ASSERT_RETURN(*pkt_len>0, PJ_EINVAL); 
     1524    PJ_ASSERT_RETURN(tp && pkt && (*pkt_len>0), PJ_EINVAL); 
    15131525    PJ_ASSERT_RETURN(srtp->session_inited, PJ_EINVALIDOP); 
    15141526 
Note: See TracChangeset for help on using the changeset viewer.