Hi Hervé,
thanks for report and patch! I have applied it to master branch.
with regards
Vaclav
On St, bře 26, 2014 at 06:49:09 +0100, Herve Couplet wrote:
> 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
_______________________________________________
Semsdev mailing list
[email protected]
http://lists.iptel.org/mailman/listinfo/semsdev
