Module: kamailio
Branch: master
Commit: 5bafc03222bf6fcc59dbcf8aec30def9cdb24581
URL: https://github.com/kamailio/kamailio/commit/5bafc03222bf6fcc59dbcf8aec30def…
Author: Olle E. Johansson <oej(a)edvina.net>
Committer: Olle E. Johansson <oej(a)edvina.net>
Date: 2015-01-17T20:51:09+01:00
sipcapture Add some missing \n in log messages
---
Modified: modules/sipcapture/sipcapture.c
---
Diff: https://github.com/kamailio/kamailio/commit/5bafc03222bf6fcc59dbcf8aec30def…
Patch: https://github.com/kamailio/kamailio/commit/5bafc03222bf6fcc59dbcf8aec30def…
---
diff --git a/modules/sipcapture/sipcapture.c b/modules/sipcapture/sipcapture.c
index 42df2c3..070482c 100644
--- a/modules/sipcapture/sipcapture.c
+++ b/modules/sipcapture/sipcapture.c
@@ -794,16 +794,16 @@ static int mod_init(void) {
}
if ((insert_retries <0) || ( insert_retries > 500)) {
- LM_ERR("insert_retries should be a value between 0 and 500");
+ LM_ERR("insert_retries should be a value between 0 and 500\n");
return -1;
}
if (( 0 == insert_retries) && (insert_retry_timeout != 0)){
- LM_ERR("insert_retry_timeout has no meaning when insert_retries is not set");
+ LM_ERR("insert_retry_timeout has no meaning when insert_retries is not set\n");
}
if ((insert_retry_timeout <0) || ( insert_retry_timeout > 300)) {
- LM_ERR("insert_retry_timeout should be a value between 0 and 300");
+ LM_ERR("insert_retry_timeout should be a value between 0 and 300\n");
return -1;
}
@@ -964,7 +964,7 @@ static int child_init(int rank)
while (c){
if (!c->db_url.s || !c->db_url.len ){
- LM_ERR("DB URL not set for capture mode:[%.*s]", c->name.len, c->name.s);
+ LM_ERR("DB URL not set for capture mode:[%.*s]\n", c->name.len, c->name.s);
return -1;
}
c->db_con = c->db_funcs.init(&c->db_url);
With the sip capture module loaded, but not activated, this error message is repeated many times - with no traffic.
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
0(93552) ERROR: sipcapture [hep.c:61]: hep_msg_received(): sipcapture:hep_msg_received HEP is not enabled
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/40
Friends,
For a long time I've been able to cause core dumps by configuring stuff badly. It should not be possible. We should
validate input and not crash because of missing or bad configurations.
In many cases there's a core dump after load of the module, in some times a modules seems to cause core dump
when Kamailio exits.
I will try to isolate where possible and create a very, very minimal configuration file to be able to repeat it.
...and as you see, I will file bug reports. :-)
I am also trying to spot where I see log messages that are not properly ended with \n
or error messages that are repeated too many times, just annoying the Kamailio admin.
Let's work on this to make 4.3 a bit more stable for beginners. First impression is important :-)
Cheers,
/O