sr-dev July 2010

sr-dev@lists.kamailio.org

29 participants
223 discussions

git:master: modules/carrierroute Improved pkg memory usage
by Marius Zbihlei 07 Jul '10

07 Jul '10

Module: sip-router Branch: master Commit: 922df4f0e32bd953c662c796058c8bf93cfd62b6 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=922df4f… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Committer: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Wed Jul 7 13:37:18 2010 +0300 modules/carrierroute Improved pkg memory usage Remove a unnecessary pkg_malloc/pkg_free done for every cr_route() module export. This will decrease pkg memory fragmentation --- modules/carrierroute/cr_func.c | 22 +++++++++++++--------- 1 files changed, 13 insertions(+), 9 deletions(-) diff --git a/modules/carrierroute/cr_func.c b/modules/carrierroute/cr_func.c index a6da6e2..714a130 100644 --- a/modules/carrierroute/cr_func.c +++ b/modules/carrierroute/cr_func.c @@ -60,7 +60,7 @@ enum hash_algorithm { static const str SIP_URI = { .s="sip:", .len=4 }; static const str SIPS_URI = { .s="sips:", .len=5 }; static const str AT_SIGN = { .s="@", .len=1 }; - +static char g_rewrite_uri[MAX_URI_SIZE+1]; /** * Get the id that belongs to a string name from gparam_t structure. @@ -289,8 +289,15 @@ static int actually_rewrite(const struct route_rule *rs, str *dest, int_str avp_val; int strip = 0; - str l_user = *user; + str l_user; + + if( !rs || !dest || !msg || !user) { + LM_ERR("NULL parameter\n"); + return -1; + } + l_user = *user; + strip = (rs->strip > user->len ? user->len : rs->strip); strip = (strip < 0 ? 0 : strip); @@ -311,12 +318,12 @@ static int actually_rewrite(const struct route_rule *rs, str *dest, } else { len += SIP_URI.len; } - dest->len = 0; - dest->s = (char *)pkg_malloc(len + 1); - if (dest->s == NULL) { - PKG_MEM_ERROR; + if ( len > MAX_URI_SIZE ) { + LM_ERR("Calculated uri size too large: %d\n", len); return -1; } + + dest->s = g_rewrite_uri; dest->len = len; p = dest->s; if (msg->parsed_uri.type == SIPS_URI_T) { @@ -339,7 +346,6 @@ static int actually_rewrite(const struct route_rule *rs, str *dest, /* this could be an error, or a blacklisted destination */ if (rs->host.len == 0) { *p = '\0'; - pkg_free(dest->s); return -1; } memcpy(p, rs->host.s, rs->host.len); @@ -351,7 +357,6 @@ static int actually_rewrite(const struct route_rule *rs, str *dest, if (add_avp(AVP_VAL_STR | descavp->v.pve->spec.pvp.pvn.u.isname.type, descavp->v.pve->spec.pvp.pvn.u.isname.name, avp_val)<0) { LM_ERR("set AVP failed\n"); - pkg_free(dest->s); return -1; } } @@ -598,7 +603,6 @@ int cr_do_route(struct sip_msg * _msg, gparam_t *_carrier, if (ret < 0) { LM_ERR("Error in do_action()\n"); } - pkg_free(dest.s); unlock_and_out: release_data(rd);

1 0

git:andrei/tcp_tls_changes: tcp: fix dispatching closed connections to tcp readers
by Andrei Pelinescu-Onciul 07 Jul '10

07 Jul '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: d89437a3d7bc25a9c098a04c6ee69fc3848ff0b5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d89437a… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Wed Jul 7 11:59:30 2010 +0200 tcp: fix dispatching closed connections to tcp readers Under very heavy load it is possible that send2child() might try to send an already closed connection/fd to a tcp reader. This can happen only if the tcp connection is watched for read (POLLIN) by tcp_main (and not by a tcp reader), the connection becomes available for reading (either new data received, EOF or RST) and tcp_main chooses a specific tcp reader to send the connection to while in the same time the same tcp reader tries to send on the same connection, fails for some reason (no more space for buffering, EOF, RST a.s.o) and sends a close command back to tcp_main. Because send2child() executes first any pending commands from the choosen tcp_reader, this might lead to closing the fd before attempting to send it to the tcp_reader. Under normal circumstances the impact is only an extra syscall and some log messages, however it is possible (but highly unlikely) that after sending the close command the tcp_reader opens a new connection for sending and sends its fd back to tcp_main. This new fd might get the same number as the freshly closed fd and send2child might send the wrong (fd, tcp connection) pair. --- tcp_main.c | 20 ++++++++++++++------ 1 files changed, 14 insertions(+), 6 deletions(-) diff --git a/tcp_main.c b/tcp_main.c index 844b3a8..aa4fc52 100644 --- a/tcp_main.c +++ b/tcp_main.c @@ -2891,6 +2891,7 @@ close_again: su2a(&tcpconn->rcv.src_su, sizeof(tcpconn->rcv.src_su)), fd, tcpconn->flags, strerror(errno), errno); } + tcpconn->s=-1; } @@ -3836,10 +3837,20 @@ inline static int send2child(struct tcp_connection* tcpconn) * send a release command, but the master fills its socket buffer * with new connection commands => deadlock) */ /* answer tcp_send requests first */ - while(handle_ser_child(&pt[tcp_children[idx].proc_no], -1)>0); + while(unlikely((tcpconn->state != S_CONN_BAD) && + (handle_ser_child(&pt[tcp_children[idx].proc_no], -1)>0))); /* process tcp readers requests */ - while(handle_tcp_child(&tcp_children[idx], -1)>0); - + while(unlikely((tcpconn->state != S_CONN_BAD && + (handle_tcp_child(&tcp_children[idx], -1)>0)))); + + /* the above possible pending requests might have included a + command to close this tcpconn (e.g. CONN_ERROR, CONN_EOF). + In this case the fd is already closed here (and possible + even replaced by another one with the same number) so it + must not be sent to a reader anymore */ + if (unlikely(tcpconn->state == S_CONN_BAD || + (tcpconn->flags & F_CONN_FD_CLOSED))) + return -1; #ifdef SEND_FD_QUEUE /* if queue full, try to queue the io */ if (unlikely(send_fd(tcp_children[idx].unix_sock, &tcpconn, @@ -3961,8 +3972,6 @@ static inline int handle_new_connect(struct socket_info* si) DBG("handle_new_connect: new connection from %s: %p %d flags: %04x\n", su2a(&su, sizeof(su)), tcpconn, tcpconn->s, tcpconn->flags); if(unlikely(send2child(tcpconn)<0)){ - LOG(L_ERR,"ERROR: handle_new_connect: no children " - "available\n"); tcpconn->flags&=~F_CONN_READER; tcpconn_put(tcpconn); tcpconn_try_unhash(tcpconn); @@ -4142,7 +4151,6 @@ send_to_child: tcpconn->flags&=~(F_CONN_MAIN_TIMER|F_CONN_READ_W|F_CONN_WANTS_RD); tcpconn_ref(tcpconn); /* refcnt ++ */ if (unlikely(send2child(tcpconn)<0)){ - LOG(L_ERR,"ERROR: handle_tcpconn_ev: no children available\n"); tcpconn->flags&=~F_CONN_READER; #ifdef TCP_ASYNC if (tcpconn->flags & F_CONN_WRITE_W){

1 0

git:andrei/tcp_tls_changes: tcp: more complete error messages
by Andrei Pelinescu-Onciul 07 Jul '10

07 Jul '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: bf755a372f5d5928e275788104204d9c7927c142 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=bf755a3… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Tue Jul 6 23:34:17 2010 +0200 tcp: more complete error messages - more information in some error messages (very useful when debugging) - spelling fixes --- pass_fd.c | 4 ++-- tcp_main.c | 17 ++++++++++------- 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/pass_fd.c b/pass_fd.c index dd0f887..26e64f5 100644 --- a/pass_fd.c +++ b/pass_fd.c @@ -205,8 +205,8 @@ again: if (ret<0){ if (errno==EINTR) goto again; if ((errno!=EAGAIN) && (errno!=EWOULDBLOCK)) - LOG(L_CRIT, "ERROR: send_fd: sendmsg failed on %d: %s\n", - unix_socket, strerror(errno)); + LOG(L_CRIT, "ERROR: send_fd: sendmsg failed sending %d on %d:" + " %s (%d)\n", fd, unix_socket, strerror(errno), errno); } return ret; diff --git a/tcp_main.c b/tcp_main.c index 70bbe92..844b3a8 100644 --- a/tcp_main.c +++ b/tcp_main.c @@ -3542,12 +3542,13 @@ inline static int handle_ser_child(struct process_table* p, int fd_i) switch(cmd){ case CONN_ERROR: LOG(L_ERR, "handle_ser_child: ERROR: received CON_ERROR for %p" - " (id %d), refcnt %d\n", - tcpconn, tcpconn->id, atomic_get(&tcpconn->refcnt)); + " (id %d), refcnt %d, flags 0x%0x\n", + tcpconn, tcpconn->id, atomic_get(&tcpconn->refcnt), + tcpconn->flags); case CONN_EOF: /* forced EOF after full send, due to send flags */ #ifdef TCP_CONNECT_WAIT /* if the connection is pending => it might be on the way of - * reaching tcp_main (e.g. CONN_NEW_COMPLETE or + * reaching tcp_main (e.g. CONN_NEW_COMPLETE or * CONN_NEW_PENDING_WRITE) => it cannot be destroyed here */ if ( !(tcpconn->flags & F_CONN_PENDING) && tcpconn_try_unhash(tcpconn) ) @@ -3855,14 +3856,16 @@ inline static int send2child(struct tcp_connection* tcpconn) return -1; } }else{ - LOG(L_ERR, "ERROR: send2child: send_fd failed\n"); + LOG(L_ERR, "ERROR: send2child: send_fd failed for %p (flags 0x%0x)" + ", fd %d\n", tcpconn, tcpconn->flags, tcpconn->s); return -1; } } #else if (unlikely(send_fd(tcp_children[idx].unix_sock, &tcpconn, sizeof(tcpconn), tcpconn->s)<=0)){ - LOG(L_ERR, "ERROR: send2child: send_fd failed\n"); + LOG(L_ERR, "ERROR: send2child: send_fd failed for %p (flags 0x%0x)" + ", fd %d\n", tcpconn, tcpconn->flags, tcpconn->s); return -1; } #endif @@ -3987,7 +3990,7 @@ static inline int handle_new_connect(struct socket_info* si) * tcp_main is not interested in further io events that might be * queued for this fd) */ -inline static int handle_tcpconn_ev(struct tcp_connection* tcpconn, short ev, +inline static int handle_tcpconn_ev(struct tcp_connection* tcpconn, short ev, int fd_i) { #ifdef TCP_ASYNC @@ -4204,7 +4207,7 @@ inline static int handle_io(struct fd_map* fm, short ev, int idx) " idx %d\n", fm, fm->fd, fm->type, fm->data, idx); goto error; default: - LOG(L_CRIT, "BUG: handle_io: uknown fd type %d\n", fm->type); + LOG(L_CRIT, "BUG: handle_io: unknown fd type %d\n", fm->type); goto error; } return ret;

1 0

git:master: Fixed a memory leak on shm memory, caused by the dtrie implementation.
by Marius Zbihlei 07 Jul '10

07 Jul '10

Module: sip-router Branch: master Commit: 5146c47626d0fb9942ae914fc64c11bef728f5f0 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5146c47… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Committer: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Wed Jul 7 12:39:58 2010 +0300 Fixed a memory leak on shm memory, caused by the dtrie implementation. dtrie_destroy and dtrie_clear could leak 10 * sizeof(void*) for each node in the trie. Mainly affected carrierroute and userblacklist modules (cherry picked from commit 5c5cc3df5301e86ee78d79f52aa917e0f67f37e8) --- lib/trie/dtrie.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/trie/dtrie.c b/lib/trie/dtrie.c index 531340b..9965263 100644 --- a/lib/trie/dtrie.c +++ b/lib/trie/dtrie.c @@ -88,10 +88,13 @@ void dtrie_delete(struct dtrie_node_t *root, struct dtrie_node_t *node, if (delete_payload) { delete_payload(node->data); } + node->data = NULL; - + if (node != root) { LM_DBG("free node at %p\n", node); + shm_free(node->child); + node->child = NULL; shm_free(node); } } @@ -102,6 +105,7 @@ void dtrie_destroy(struct dtrie_node_t **root, dt_delete_func_t delete_payload, if ((root!=NULL) && (*root!=NULL)) { dtrie_delete(*root, *root, delete_payload, branches); LM_DBG("free root at %p\n", root); + shm_free((*root)->child); shm_free(*root); *root = NULL; }

1 0

git:kamailio_3.0: Fixed a memory leak on shm memory, caused by the dtrie implementation.
by Marius Zbihlei 07 Jul '10

07 Jul '10

Module: sip-router Branch: kamailio_3.0 Commit: 5c5cc3df5301e86ee78d79f52aa917e0f67f37e8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5c5cc3d… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Committer: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Date: Wed Jul 7 12:39:58 2010 +0300 Fixed a memory leak on shm memory, caused by the dtrie implementation. dtrie_destroy and dtrie_clear could leak 10 * sizeof(void*) for each node in the trie. Mainly affected carrierroute and userblacklist modules --- lib/trie/dtrie.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/lib/trie/dtrie.c b/lib/trie/dtrie.c index 531340b..9965263 100644 --- a/lib/trie/dtrie.c +++ b/lib/trie/dtrie.c @@ -88,10 +88,13 @@ void dtrie_delete(struct dtrie_node_t *root, struct dtrie_node_t *node, if (delete_payload) { delete_payload(node->data); } + node->data = NULL; - + if (node != root) { LM_DBG("free node at %p\n", node); + shm_free(node->child); + node->child = NULL; shm_free(node); } } @@ -102,6 +105,7 @@ void dtrie_destroy(struct dtrie_node_t **root, dt_delete_func_t delete_payload, if ((root!=NULL) && (*root!=NULL)) { dtrie_delete(*root, *root, delete_payload, branches); LM_DBG("free root at %p\n", root); + shm_free((*root)->child); shm_free(*root); *root = NULL; }

1 0

SF.net SVN: openser:[6015] branches/1.5/trie/dtrie.c
by Marius Zbihlei 07 Jul '10

07 Jul '10

Revision: 6015 http://openser.svn.sourceforge.net/openser/?rev=6015&view=rev Author: mariuszbihlei Date: 2010-07-07 09:31:13 +0000 (Wed, 07 Jul 2010) Log Message: ----------- Fixed a memory leak on pkg memory. dtrie_destroy and dtrie_clear could leak 10 * sizeof(void*) for each node in the trie. Mainly affected carrierroute and userblacklist modules Modified Paths: -------------- branches/1.5/trie/dtrie.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

1 1

Summary of Devel Meeting, Berlin 2010
by Daniel-Constantin Mierla 06 Jul '10

06 Jul '10

Hello, fyi: summary of the devel meeting in Berlin, June 8, 2010, was uploaded to: http://sip-router.org/wiki/meetings/berlin-2010-minutes Cheers, Daniel -- Daniel-Constantin Mierla http://www.asipto.com/

2 1

git:andrei/tcp_tls_changes: tls: support for partial encoding and reseting send_flags
by Andrei Pelinescu-Onciul 06 Jul '10

06 Jul '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: a08385f531dd0bf1044c8495e6e47c9aabfebd8a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a08385f… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Mon Jul 5 15:23:02 2010 +0200 tls: support for partial encoding and reseting send_flags Support for the extended version of the tls_encode() callback. On partial tls write and no more space in the internal static buffers for more records, return what records were encoded so far and ask to be called again after they are sent. When exiting after a partial write or after postponing the encode until more data has been read (WANT_READ), make sure that any request to close the connection after the first successful tcp level write will be postponed (reset SND_F_CON_CLOSE). --- modules/tls/tls_server.c | 56 ++++++++++++++++++++++++++++++++++++++------- modules/tls/tls_server.h | 4 ++- 2 files changed, 50 insertions(+), 10 deletions(-) diff --git a/modules/tls/tls_server.c b/modules/tls/tls_server.c index 600653c..f6d822c 100644 --- a/modules/tls/tls_server.c +++ b/modules/tls/tls_server.c @@ -607,16 +607,31 @@ typedef int (*tcp_low_level_send_t)(int fd, struct tcp_connection *c, * It is a callback that will be called by the tcp code, before a send * on TLS would be attempted. It should replace the input buffer with a * new static buffer containing the TLS processed data. + * If the input buffer could not be fully encoded (e.g. run out of space + * in the internal static buffer), it should set rest_buf and rest_len to + * the remaining part, so that it could be called again once the output has + * been used (sent). The send_flags used are also passed and they can be + * changed (e.g. to disallow a close() after a partial encode). * WARNING: it must always be called with c->write_lock held! * @param c - tcp connection * @param pbuf - pointer to buffer (value/result, on success it will be * replaced with a static buffer). * @param plen - pointer to buffer size (value/result, on success it will be * replaced with the size of the replacement buffer. + * @param rest_buf - (result) should be filled with a pointer to the + * remaining unencoded part of the original buffer if any, + * 0 otherwise. + * @param rest_len - (result) should be filled with the length of the + * remaining unencoded part of the original buffer (0 if + * the original buffer was fully encoded). + * @param send_flags - pointer to the send_flags that will be used for sending + * the message. * @return *plen on success (>=0), < 0 on error. */ int tls_encode_f(struct tcp_connection *c, - const char* *pbuf, unsigned int* plen) + const char** pbuf, unsigned int* plen, + const char** rest_buf, unsigned int* rest_len, + snd_flags_t* send_flags) { int n, offs; SSL* ssl; @@ -631,8 +646,11 @@ int tls_encode_f(struct tcp_connection *c, buf = *pbuf; len = *plen; - TLS_WR_TRACE("(%p, %p, %d) start (%s:%d* -> %s)\n", - c, buf, len, ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port, + *rest_buf = 0; + *rest_len = 0; + TLS_WR_TRACE("(%p, %p, %d, ... 0x%0x) start (%s:%d* -> %s)\n", + c, buf, len, send_flags->f, + ip_addr2a(&c->rcv.dst_ip), c->rcv.dst_port, su2a(&c->rcv.src_su, sizeof(c->rcv.src_su))); n = 0; offs = 0; @@ -656,6 +674,10 @@ int tls_encode_f(struct tcp_connection *c, c, tls_c->ct_wq?tls_c->ct_wq->queued:0); goto error_wq_full; } + /* buffer queued for a future send attempt, after first reading + some data (key exchange) => don't allow immediate closing of + the connection */ + send_flags->f &= ~SND_F_CON_CLOSE; goto end; } if (unlikely(tls_set_mbufs(c, &rd, &wr) < 0)) { @@ -718,12 +740,28 @@ redo_wr: goto error_wq_full; } tls_c->flags |= F_TLS_CON_WR_WANTS_RD; + /* buffer queued for a future send attempt, after first + reading some data (key exchange) => don't allow immediate + closing of the connection */ + send_flags->f &= ~SND_F_CON_CLOSE; break; /* or goto end */ case SSL_ERROR_WANT_WRITE: - /* error, no record fits in the buffer */ - BUG("write buffer too small (%d/%d bytes)\n", - wr.used, wr.size); - goto bug; + if (unlikely(offs == 0)) { + /* error, no record fits in the buffer or + no partial write enabled and buffer to small to fit + all the records */ + BUG("write buffer too small (%d/%d bytes)\n", + wr.used, wr.size); + goto bug; + } else { + /* offs != 0 => something was "written" */ + *rest_buf = buf + offs; + *rest_len = len - offs; + /* this function should be called again => disallow + immediate closing of the connection */ + send_flags->f &= ~SND_F_CON_CLOSE; + } + break; /* or goto end */ #if OPENSSL_VERSION_NUMBER >= 0x00907000L /*0.9.7*/ case SSL_ERROR_WANT_CONNECT: /* only if the underlying BIO is not yet connected @@ -772,8 +810,8 @@ redo_wr: end: *pbuf = (const char*)wr.buf; *plen = wr.used; - TLS_WR_TRACE("(%p) end (offs %d) => %d \n", - c, offs, *plen); + TLS_WR_TRACE("(%p) end (offs %d, rest_buf=%p rest_len=%d 0x%0x) => %d \n", + c, offs, *rest_buf, *rest_len, send_flags->f, *plen); return *plen; error: /*error_send:*/ diff --git a/modules/tls/tls_server.h b/modules/tls/tls_server.h index ba28067..62047d5 100644 --- a/modules/tls/tls_server.h +++ b/modules/tls/tls_server.h @@ -84,7 +84,9 @@ void tls_h_tcpconn_clean(struct tcp_connection *c); void tls_h_close(struct tcp_connection *c, int fd); int tls_encode_f(struct tcp_connection *c, - const char ** pbuf, unsigned int* plen); + const char ** pbuf, unsigned int* plen, + const char** rest_buf, unsigned int* rest_len, + snd_flags_t* send_flags) ; int tls_read_f(struct tcp_connection *c, int* flags);

1 0

git:andrei/tcp_tls_changes: tcp: support for tls partial encoding
by Andrei Pelinescu-Onciul 06 Jul '10

06 Jul '10

Module: sip-router Branch: andrei/tcp_tls_changes Commit: e316312628e61cbded9b991d21385b4e8180b538 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e316312… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Mon Jul 5 14:44:14 2010 +0200 tcp: support for tls partial encoding The tls_encode() callback supports now partial encoding. It can encode only part of the input buffer (e.g. tls partial write enabled and not enough space in the internal buffers for all the tls records) and ask to be called back later with the remaining un-encoded part. The send flags are also passed to it, so that it can modify them if needed (e.g. reset SND_F_CON_CLOSE on partial tls write or when reading some data is needed before continuing the tls write). --- tcp_main.c | 230 +++++++++++++++++++++++++++++++++++++++++++++++------------ tls_hooks.h | 19 ++++-- 2 files changed, 199 insertions(+), 50 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e31…

1 0

Dialog module: "CRITICAL bogus event 6 in state 2
by Klaus Feichtinger 06 Jul '10

06 Jul '10

Hello list, under special circumstances I have a problem with the DIALOG module of SIP-Router/Kamailio version 3.0.2. The dialog module is used in combination with presence, presence_xml, pua, pua_usrloc and pua_dialoginfo modules + db_mysql modules. The db_mode is set to "write_through", because of redundancy requirements. >From my point of view this problem has something to do with timing in general. E.g. a so called "data call" has a duration of about 50 ms only; INVITE....BYE). That problem occurs on this short calls only. Because of the problems in the dialog state machine, NOTIFY messages are incorrect. Even when the call is already finished, the NOTIFY message includes the state "confirmed" and causes a wrong status indication.... The detailed error message can be seen here: "Jun 9 16:00:39 debian /usr/sbin/kamailio[14992]: CRITICAL: dialog [dlg_hash.c:591]: bogus event 6 in state 2 for dlg [dialog-ID] with clid [Call-ID] and tags '1299370188-28358304-1276092068837' ''" I found in old mails of the developer list another error that looked nearly the same, but it differed in the event and state of the dialog state machine (the thread can be found under the link http://www.mail-archive.com/devel@lists.kamailio.org/msg03234.html). That bug should - according the information that I've found in the list - be solved. Therefore I will ask you: is the error as displayed above another well known error / bug, which (maybe) should already be solved? Is it a new bug? Thanks for any information. regards, Klaus Feichtinger -- GMX DSL: Internet-, Telefon- und Handy-Flat ab 19,99 EUR/mtl. Bis zu 150 EUR Startguthaben inklusive! http://portal.gmx.net/de/go/dsl

7 19

← Newer
1
...
15
16
17
18
19
20
21
22
23
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2010