eddyleo / doubango

Automatically exported from code.google.com/p/doubango
0 stars 0 forks source link

Unable to promote audio to audio/video (ICE disabled) #117

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Do a voice call
2. Click on add video
3. New media stream fails to be added

What version of the product are you using? On what operating system?
Win 7, r703

Please provide any additional information below.

Before adding the new media type, the rtp manager is stopped. In it's function 
to stop, the transport is beeing freed

// Free transport to force next call to start() to create new one with new 
sockets
    TSK_OBJECT_SAFE_FREE(self->transport);

Later on, when the media is about to change, in the tdav_session_av_get_lo 
function:

tdav_session_av.c line 331
if(!self->rtp_manager || (!self->ice_ctx && !self->rtp_manager->transport)){ 
... }

With ICE disabled, (!self->ice_ctx && !self->rtp_manager->transport) will 
always result in TRUE, therefor fail the media change.

Original issue reported on code.google.com by radu.vul...@gmail.com on 31 May 2012 at 10:04

GoogleCodeExporter commented 9 years ago

Original comment by boss...@yahoo.fr on 31 May 2012 at 2:58

GoogleCodeExporter commented 9 years ago
Fixed in r706

Original comment by boss...@yahoo.fr on 1 Jun 2012 at 2:52