Hi,

I found out that disabling the session timer on call leg A only is not working with the sources actually in the git repository. The option enable_aleg_session_timer=no with enable_session_timer=yes should only activate session timer on call leg B. Instead the session timer stay activated for Call leg B AND for call leg A with the default options.

Here is the fix I did and tested working :

diff --git a/apps/sbc/SBCCallLeg.cpp b/apps/sbc/SBCCallLeg.cpp
index 9185a4a..c48c468 100644
--- a/apps/sbc/SBCCallLeg.cpp
+++ b/apps/sbc/SBCCallLeg.cpp
@@ -794,9 +794,9 @@ void SBCCallLeg::onInvite(const AmSipRequest& req)
call_profile.sst_enabled = ctx.replaceParameters(call_profile.sst_enabled,
 "enable_session_timer", req);

- if ((call_profile.sst_aleg_enabled == "yes") ||
+ if ((call_profile.sst_aleg_enabled == "yes") &&
 (call_profile.sst_enabled == "yes")) {

 call_profile.eval_sst_config(ctx,req,call_profile.sst_a_cfg);
 if(applySSTCfg(call_profile.sst_a_cfg,&req) < 0) {
 throw AmSession::Exception(500, SIP_REPLY_SERVER_INTERNAL_ERROR);

I think it would be useful to integrate the fix into the git repository.

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

Reply via email to