Hi all,
Thanks for the work to consolidate SER modules for Kamailio 4.0. This
has been a great improvement!
When I run kamdbctl dbuid (added in
http://git.sip-router.org/cgi-bin/gitweb.cgi?p=sip-router;a=commitdiff;h=1f…)
when using PostgreSQL as database engine, I get the following error:
~# kamdbctl dbuid
/usr/sbin/kamdbctl: line 464: dbuid_create: not found
I tested this on Kamailio 4.0.1, but it seems that the issue exists in
the latest git as well. The attached patch causes kamdbctl dbuid to
successfully create the uid tables.
Hope this helps,
-elactrum
currently isflagset core function accepts only one flag as argument. it
would be nice if more that one flag could be tested with one function
call, e.g.,
isflagset(1|2) or isflagset(1&2)
since i'm not familiar with adding syntax related stuff to core, is it
ok if i create a tracker wish list item on this?
-- juha
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task is now closed:
FS#337 - uac_redirect: get_redirects() with no second parameter raises a core dump
User who did this - Víctor Seva (linuxmaniac)
Reason for closing: Fixed
Additional comments about closing: Fixed applied on 4.0 branch too.
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=337
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.
Module: sip-router
Branch: 4.0
Commit: 51f40460096d9c249ce34d9b4fbecf366b5f974e
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=51f4046…
Author: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: Mon Sep 9 16:52:47 2013 +0200
uac_redirect: get_redirect() check reason value before using it.
This fix a core dump when get_redirects() config fuction is called with just
the one parameter.
(cherry picked from commit abf0026782c0ba4643feb25ded022e8c12725584)
---
modules/uac_redirect/rd_funcs.c | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/modules/uac_redirect/rd_funcs.c b/modules/uac_redirect/rd_funcs.c
index 19b4dad..d1e4d46 100644
--- a/modules/uac_redirect/rd_funcs.c
+++ b/modules/uac_redirect/rd_funcs.c
@@ -96,9 +96,12 @@ int get_redirect( struct sip_msg *msg , int maxt, int maxb,
}
if (max==0)
continue;
- /* put the response code into the acc_param reason struct */
- reason->code = t->uac[i].last_received;
- reason->code_s.s = int2bstr((unsigned long)reason->code, code_buf, &reason->code_s.len);
+ if(reason!=NULL)
+ {
+ /* put the response code into the acc_param reason struct */
+ reason->code = t->uac[i].last_received;
+ reason->code_s.s = int2bstr((unsigned long)reason->code, code_buf, &reason->code_s.len);
+ }
/* get the contact from it */
n = shmcontact2dset( msg, t->uac[i].reply, max, reason, bflags);
if ( n<0 ) {
Module: sip-router
Branch: master
Commit: abf0026782c0ba4643feb25ded022e8c12725584
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=abf0026…
Author: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org>
Date: Mon Sep 9 16:52:47 2013 +0200
uac_redirect: get_redirect() check reason value before using it.
This fix a core dump when get_redirects() config fuction is called with just
the one parameter.
---
modules/uac_redirect/rd_funcs.c | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/modules/uac_redirect/rd_funcs.c b/modules/uac_redirect/rd_funcs.c
index ca34e0c..aa8dbd0 100644
--- a/modules/uac_redirect/rd_funcs.c
+++ b/modules/uac_redirect/rd_funcs.c
@@ -96,9 +96,12 @@ int get_redirect( struct sip_msg *msg , int maxt, int maxb,
}
if (max==0)
continue;
- /* put the response code into the acc_param reason struct */
- reason->code = t->uac[i].last_received;
- reason->code_s.s = int2bstr((unsigned long)reason->code, code_buf, &reason->code_s.len);
+ if(reason!=NULL)
+ {
+ /* put the response code into the acc_param reason struct */
+ reason->code = t->uac[i].last_received;
+ reason->code_s.s = int2bstr((unsigned long)reason->code, code_buf, &reason->code_s.len);
+ }
/* get the contact from it */
n = shmcontact2dset( msg, t->uac[i].reply, max, reason, bflags);
if ( n<0 ) {
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#337 - uac_redirect: get_redirects() with no second parameter raises a core dump
User who did this - Daniel-Constantin Mierla (miconda)
----------
Thanks, can be pushed to repository.
----------
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=337#comment1068
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
Víctor Seva has taken ownership of the following task:
FS#337 - uac_redirect: get_redirects() with no second parameter raises a core dump
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=337
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
A new Flyspray task has been opened. Details are below.
User who did this - Víctor Seva (linuxmaniac)
Attached to Project - sip-router
Summary - uac_redirect: get_redirects() with no second parameter raises a core dump
Task Type - Bug Report
Category - Module
Status - Unconfirmed
Assigned To -
Operating System - All
Severity - Low
Priority - Normal
Reported Version - 4.0
Due in Version - Undecided
Due Date - Undecided
Details - We are getting a core dump on uac_redirect module when calling get_redirects() config function using just one parameter.
One or more files have been attached.
More information can be found at the following URL:
https://sip-router.org/tracker/index.php?do=details&task_id=337
You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.