[prev in list] [next in list] [prev in thread] [next in thread] 

List:       semsdev
Subject:    [Semsdev] git:master: b/f: sbc: honor empty session_timer profile variable in initialization
From:       Stefan Sayer <stefan.sayer () googlemail ! com>
Date:       2012-04-25 15:19:50
Message-ID: 20120425151950.6857EEF8074 () rimmer ! ryngle ! com
[Download RAW message or body]

Module: sems
Branch: master
Commit: 3689154b85e5ecccf6dd27629267e2320fb49af6
URL:    http://git.sip-router.org/cgi-bin/gitweb.cgi/sems/?a=commit;h=3689154b85e5ecccf6dd27629267e2320fb49af6


Author: Stefan Sayer <stefan.sayer@googlemail.com>
Committer: Stefan Sayer <stefan.sayer@googlemail.com>
Date:   Wed Apr 25 17:19:31 2012 +0200

b/f: sbc: honor empty session_timer profile variable in initialization

---

 apps/sbc/SBCCallProfile.cpp |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/apps/sbc/SBCCallProfile.cpp b/apps/sbc/SBCCallProfile.cpp
index df6994d..5790863 100644
--- a/apps/sbc/SBCCallProfile.cpp
+++ b/apps/sbc/SBCCallProfile.cpp
@@ -203,7 +203,7 @@ bool SBCCallProfile::readFromConfiguration(const string& name,
       dstcfg.setParameter(cfgkey, SBCFactory::cfg.getParameter(cfgkey)); \
     }
 
-  if (sst_enabled != "no") {
+  if (sst_enabled.size() && sst_enabled != "no") {
     if (NULL == SBCFactory::session_timer_fact) {
       ERROR("session_timer module not loaded thus SST not supported, but "
 	    "required for profile '%s' (%s)\n", name.c_str(), profile_file_name.c_str());
@@ -219,7 +219,7 @@ bool SBCCallProfile::readFromConfiguration(const string& name,
     CP_SST_CFGVAR("", "accept_501_reply", sst_b_cfg);
   }
 
-  if (sst_aleg_enabled != "no") {
+  if (sst_aleg_enabled.size() && sst_aleg_enabled != "no") {
     sst_a_cfg.setParameter("enable_session_timer", "yes");
     // create sst_a_cfg superimposing values from aleg_*
     CP_SST_CFGVAR("aleg_", "session_expires", sst_a_cfg);
@@ -423,7 +423,7 @@ bool SBCCallProfile::readFromConfiguration(const string& name,
 
     INFO("SBC:      SST on A leg enabled: '%s'\n", sst_aleg_enabled.empty() ?
 	 "no" : sst_aleg_enabled.c_str());
-    if (sst_aleg_enabled != "no") {
+    if (sst_aleg_enabled.size() && sst_aleg_enabled != "no") {
       INFO("SBC:              session_expires=%s\n",
 	   sst_a_cfg.getParameter("session_expires").c_str());
       INFO("SBC:              minimum_timer=%s\n",
@@ -437,7 +437,7 @@ bool SBCCallProfile::readFromConfiguration(const string& name,
     }
     INFO("SBC:      SST on B leg enabled: '%s'\n", sst_enabled.empty() ?
 	 "no" : sst_enabled.c_str());
-    if (sst_enabled != "no") {
+    if (sst_enabled.size() && sst_enabled != "no") {
       INFO("SBC:              session_expires=%s\n",
 	   sst_b_cfg.getParameter("session_expires").c_str());
       INFO("SBC:              minimum_timer=%s\n",

_______________________________________________
Semsdev mailing list
Semsdev@lists.iptel.org
http://lists.iptel.org/mailman/listinfo/semsdev


[prev in list] [next in list] [prev in thread] [next in thread]