Hi *

Something I came across in the code that I believe may not be entirely
correct.

If we look at the callee_status enumeration in AmB2ABCallerSession we
see the following code:

switch(ev->event_id) {
  case B2ABConnectAudio: {
    callee_status = Connected;

    DBG("ConnectAudio event received from other leg\n");
    B2ABConnectAudioEvent* ca =
dynamic_cast<B2ABConnectAudioEvent*>(ev);
    assert(ca);

    connector = ca->connector;
    connectSession();

    return;
  } break;

This seems to put the callee_status as 'connected' when a connect audio
event is received. The problem is that this is exactly the case when we
have early media (with the session not being 'connected' as in 200 OK
connected???? what are your thoughts?

should we not remove the following line from
AmB2ABCalleeSession::onEarlySessionStart

relayEvent(new B2ABConnectAudioEvent(connector)); (line to be
remove)???????


Cheers
Jason

_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev

Reply via email to