diff options
| author | Benny Prijono <bennylp@teluu.com> | 2006-11-25 08:48:54 +0000 |
|---|---|---|
| committer | Benny Prijono <bennylp@teluu.com> | 2006-11-25 08:48:54 +0000 |
| commit | 3db966e7ebb20d9bb51cdb3a3b6d1993984ab547 (patch) | |
| tree | 79601c305cd43d9008bcdf18394b5a2c5962b3f9 /pjmedia | |
| parent | 152fa151e076d84f8a8b7d7962b053f47608cfaf (diff) | |
Minor fix in UDP media: transport should silently ignore detach command when it's not attached
git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@830 74dad513-b988-da41-8d7b-12977e46ad98
Diffstat (limited to 'pjmedia')
| -rw-r--r-- | pjmedia/src/pjmedia/transport_udp.c | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/pjmedia/src/pjmedia/transport_udp.c b/pjmedia/src/pjmedia/transport_udp.c index 31fe74c6..901840f8 100644 --- a/pjmedia/src/pjmedia/transport_udp.c +++ b/pjmedia/src/pjmedia/transport_udp.c @@ -583,19 +583,21 @@ static void transport_detach( pjmedia_transport *tp, pj_assert(tp); - /* User data is unreferenced on Release build */ - PJ_UNUSED_ARG(user_data); + if (udp->attached) { + /* User data is unreferenced on Release build */ + PJ_UNUSED_ARG(user_data); - /* As additional checking, check if the same user data is specified */ - pj_assert(user_data == udp->user_data); + /* As additional checking, check if the same user data is specified */ + pj_assert(user_data == udp->user_data); - /* First, mark transport as unattached */ - udp->attached = PJ_FALSE; + /* First, mark transport as unattached */ + udp->attached = PJ_FALSE; - /* Clear up application infos from transport */ - udp->rtp_cb = NULL; - udp->rtcp_cb = NULL; - udp->user_data = NULL; + /* Clear up application infos from transport */ + udp->rtp_cb = NULL; + udp->rtcp_cb = NULL; + udp->user_data = NULL; + } } |