I get repeatable segfaults with acc.so in 4.3.0-dev2. Fedora 20 x86_64. Debug log attached. -A
Can you send the modparam lines for acc module?
Also, as you can reproduce, before starting kamailio, do:
uname -c unliminted
Start kamailio as root and reproduce. You should get a coredump file, send the backtrace from it with gdb.
Cheers, Daniel
On 07/12/14 17:59, Anthony Messina wrote:
I get repeatable segfaults with acc.so in 4.3.0-dev2. Fedora 20 x86_64. Debug log attached. -A
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
On Sunday, December 07, 2014 07:22:07 PM Daniel-Constantin Mierla wrote:
Can you send the modparam lines for acc module?
Here are related config lines:
#!define WITH_ACCDB #!define ACCDB_COMMENT
#!define FLT_ACC 1 #!define FLT_ACCMISSED 2 #!define FLT_ACCFAILED 3
#!ifdef ACCDB_COMMENT ALTER TABLE acc ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE acc ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE missed_calls ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; #!endif
loadmodule "acc.so"
# ----- acc params ----- /* what special events should be accounted ? */ modparam("acc", "early_media", 0) modparam("acc", "report_ack", 0) modparam("acc", "report_cancels", 0) /* by default we do not adjust the direct of the sequential requests. if you enable this parameter, be sure the enable "append_fromtag" in "rr" module */ modparam("acc", "detect_direction", 0) /* account triggers (flags) */ modparam("acc", "log_flag", FLT_ACC) modparam("acc", "log_missed_flag", FLT_ACCMISSED) modparam("acc", "log_extra", "src_user=$fU;src_domain=$fd;src_ip=$si;" "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") modparam("acc", "failed_transaction_flag", FLT_ACCFAILED) /* enhanced DB accounting */ #!ifdef WITH_ACCDB modparam("acc", "db_flag", FLT_ACC) modparam("acc", "db_missed_flag", FLT_ACCMISSED) modparam("acc", "db_url", DBURL) modparam("acc", "db_extra", "src_user=$fU;src_domain=$fd;src_ip=$si;" "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") #!endif
Also, as you can reproduce, before starting kamailio, do:
uname -c unliminted
Start kamailio as root and reproduce. You should get a coredump file, send the backtrace from it with gdb.
I'll need to work on this in a bit and send the results. I do know this does not happen if I comment the following as so:
##!define WITH_ACCDB ##!define ACCDB_COMMENT
The segfault also did not occur with my previous build from git master@9a46223
-A
Hello,
thanks for the backtrace, it helped to find quickly and fix the issue. Patch should be on master now.
Cheers, Daniel
On 07/12/14 19:34, Anthony Messina wrote:
On Sunday, December 07, 2014 07:22:07 PM Daniel-Constantin Mierla wrote:
Can you send the modparam lines for acc module?
Here are related config lines:
#!define WITH_ACCDB #!define ACCDB_COMMENT
#!define FLT_ACC 1 #!define FLT_ACCMISSED 2 #!define FLT_ACCFAILED 3
#!ifdef ACCDB_COMMENT ALTER TABLE acc ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE acc ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE acc ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_domain VARCHAR(128) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN src_ip varchar(64) NOT NULL default ''; ALTER TABLE missed_calls ADD COLUMN dst_ouser VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_user VARCHAR(64) NOT NULL DEFAULT ''; ALTER TABLE missed_calls ADD COLUMN dst_domain VARCHAR(128) NOT NULL DEFAULT ''; #!endif
loadmodule "acc.so"
# ----- acc params ----- /* what special events should be accounted ? */ modparam("acc", "early_media", 0) modparam("acc", "report_ack", 0) modparam("acc", "report_cancels", 0) /* by default we do not adjust the direct of the sequential requests. if you enable this parameter, be sure the enable "append_fromtag" in "rr" module */ modparam("acc", "detect_direction", 0) /* account triggers (flags) */ modparam("acc", "log_flag", FLT_ACC) modparam("acc", "log_missed_flag", FLT_ACCMISSED) modparam("acc", "log_extra", "src_user=$fU;src_domain=$fd;src_ip=$si;" "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") modparam("acc", "failed_transaction_flag", FLT_ACCFAILED) /* enhanced DB accounting */ #!ifdef WITH_ACCDB modparam("acc", "db_flag", FLT_ACC) modparam("acc", "db_missed_flag", FLT_ACCMISSED) modparam("acc", "db_url", DBURL) modparam("acc", "db_extra", "src_user=$fU;src_domain=$fd;src_ip=$si;" "dst_ouser=$tU;dst_user=$rU;dst_domain=$rd") #!endif
Also, as you can reproduce, before starting kamailio, do:
uname -c unliminted
Start kamailio as root and reproduce. You should get a coredump file, send the backtrace from it with gdb.
I'll need to work on this in a bit and send the results. I do know this does not happen if I comment the following as so:
##!define WITH_ACCDB ##!define ACCDB_COMMENT
The segfault also did not occur with my previous build from git master@9a46223
-A
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
On Sunday, December 07, 2014 08:28:55 PM Daniel-Constantin Mierla wrote:
Hello,
thanks for the backtrace, it helped to find quickly and fix the issue. Patch should be on master now.
I see. I'll rebuild and test. Thanks. -A
On 07/12/14 20:34, Anthony Messina wrote:
On Sunday, December 07, 2014 08:28:55 PM Daniel-Constantin Mierla wrote:
Hello,
thanks for the backtrace, it helped to find quickly and fix the issue. Patch should be on master now.
I see. I'll rebuild and test. Thanks. -A
Ok, report the results -- do check if accounting records are written correctly to database.
Cheers, Daniel
On Sunday, December 07, 2014 09:00:07 PM Daniel-Constantin Mierla wrote:
On 07/12/14 20:34, Anthony Messina wrote:
On Sunday, December 07, 2014 08:28:55 PM Daniel-Constantin Mierla wrote:
Hello,
thanks for the backtrace, it helped to find quickly and fix the issue. Patch should be on master now.
I see. I'll rebuild and test. Thanks. -A
Ok, report the results -- do check if accounting records are written correctly to database.
I can confirm that 0cf6fdc78d2171dba8d31370b84da99d8c94fb29 fixes the issue and the entries are added to the DB properly. Thanks again! -A
On 07/12/14 21:24, Anthony Messina wrote:
On Sunday, December 07, 2014 09:00:07 PM Daniel-Constantin Mierla wrote:
On 07/12/14 20:34, Anthony Messina wrote:
On Sunday, December 07, 2014 08:28:55 PM Daniel-Constantin Mierla wrote:
Hello,
thanks for the backtrace, it helped to find quickly and fix the issue. Patch should be on master now.
I see. I'll rebuild and test. Thanks. -A
Ok, report the results -- do check if accounting records are written correctly to database.
I can confirm that 0cf6fdc78d2171dba8d31370b84da99d8c94fb29 fixes the issue and the entries are added to the DB properly. Thanks again! -A
Thanks for reporting back to close this issue.
Daniel