7 Dec
2014
7 Dec
'14
6:59 p.m.
I get repeatable segfaults with acc.so in 4.3.0-dev2. Fedora 20 x86_64. Debug
log attached. -A
--
Anthony - https://messinet.com/ - https://messinet.com/~amessina/gallery
8F89 5E72 8DF0 BCF0 10BE 9967 92DC 35DC B001 4A4E
7 Dec
7 Dec
8:22 p.m.
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(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
8:34 p.m.
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
--
Anthony - https://messinet.com/ - https://messinet.com/~amessina/gallery
8F89 5E72 8DF0 BCF0 10BE 9967 92DC 35DC B001 4A4E
9:28 p.m.
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:
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
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(a)lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
9:34 p.m.
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
--
Anthony - https://messinet.com/ - https://messinet.com/~amessina/gallery
8F89 5E72 8DF0 BCF0 10BE 9967 92DC 35DC B001 4A4E
10 p.m.
On 07/12/14 20:34, Anthony Messina wrote:
On Sunday, December 07, 2014 08:28:55 PM
Daniel-Constantin Mierla wrote:
Ok, report the results -- do check if accounting records are written
correctly to database.
Cheers,
Daniel
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
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
10:24 p.m.
On Sunday, December 07, 2014 09:00:07 PM Daniel-Constantin Mierla wrote:
On 07/12/14 20:34, Anthony Messina wrote:
I can confirm that 0cf6fdc78d2171dba8d31370b84da99d8c94fb29 fixes the issue
and the entries are added to the DB properly. Thanks again! -A
--
Anthony - https://messinet.com/ - https://messinet.com/~amessina/gallery
8F89 5E72 8DF0 BCF0 10BE 9967 92DC 35DC B001 4A4E
On Sunday, December 07, 2014 08:28:55 PM
Daniel-Constantin Mierla wrote:
Ok, report the results -- do check if accounting records are written
correctly to database. 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
8 Dec
8 Dec
1:59 p.m.
On 07/12/14 21:24, Anthony Messina wrote:
On Sunday, December 07, 2014 09:00:07 PM
Daniel-Constantin Mierla wrote:
Thanks for
reporting back to close this issue.
Daniel
--
Daniel-Constantin Mierla
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
On 07/12/14 20:34, Anthony Messina wrote:
I can confirm that 0cf6fdc78d2171dba8d31370b84da99d8c94fb29
fixes the issue
and the entries are added to the DB properly. Thanks again! -A On Sunday, December 07, 2014 08:28:55 PM
Daniel-Constantin Mierla wrote:
Ok, report the results -- do check if accounting records are written
correctly to database. 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
3643
days inactive
3644
days old
7 comments
2 participants
participants (2)
-
Anthony Messina -
Daniel-Constantin Mierla