Hello Jim,
Since I am unable to replicate the bug I'm providing you the following
patch which prints the extra values when they are taken from the
variables and after they are put into the array from where they will be
inserted into the db. Can you please apply it and provide the logs with
the patch? If any sensitive data in there send the logs on my mail.
Regards,
Ionut Ionita
OpenSIPS Developer
On 04/26/2017 07:01 PM, Jim DeVito wrote:
Also it should be noted I am using the db_flatstore module with the
acc module.
On Wed, Apr 26, 2017 at 11:48 AM, Bogdan-Andrei Iancu
<[email protected] <mailto:[email protected]>> wrote:
OK, thanks.
I suppose you have the "failed" flag set in do_accounting() ?
And before ending the INVITE processing, do you change the RURI
(you mentioned it originally has a username) ?
Regards,
Bogdan-Andrei Iancu
OpenSIPS Founder and Developer
http://www.opensips-solutions.com <http://www.opensips-solutions.com>
OpenSIPS Summit May 2017 Amsterdam
http://www.opensips.org/events/Summit-2017Amsterdam.html
<http://www.opensips.org/events/Summit-2017Amsterdam.html>
On 04/26/2017 06:33 PM, Jim DeVito wrote:
Yes. And correct the script is rejecting the call at this point
and returning the 600 to my upstream.
On Wed, Apr 26, 2017 at 11:28 AM, Bogdan-Andrei Iancu
<[email protected] <mailto:[email protected]>> wrote:
Again, does the strange text correspond to the to_tn extra
value ? And the call is rejected by you from script ? it not
ever proxied further, right ? Regards,
Bogdan-Andrei Iancu
OpenSIPS Founder and Developer
http://www.opensips-solutions.com
<http://www.opensips-solutions.com>
OpenSIPS Summit May 2017 Amsterdam
http://www.opensips.org/events/Summit-2017Amsterdam.html
<http://www.opensips.org/events/Summit-2017Amsterdam.html>
On 04/26/2017 06:25 PM, Jim DeVito wrote:
Hi Bogdan,
Sorry forgot the mention I am using 2.2.3 the latest stable
from the repo. I put a log line just after
send_reply("600","Busy Everywhere"); and $rU looks good
there. Is there another place I should put a log line to see
the value of $rU?
Thanks!!
On Wed, Apr 26, 2017 at 11:16 AM, Bogdan-Andrei Iancu
<[email protected] <mailto:[email protected]>> wrote:
Hi Jim, What OpenSIPS version do you use ? Is the x00
string corresponding to the to_tn extra field ? If yes,
is there any chance to have the $rU null (no username in
RURI) ? Regards,
Bogdan-Andrei Iancu
OpenSIPS Founder and Developer
http://www.opensips-solutions.com
<http://www.opensips-solutions.com>
OpenSIPS Summit May 2017 Amsterdam
http://www.opensips.org/events/Summit-2017Amsterdam.html
<http://www.opensips.org/events/Summit-2017Amsterdam.html>
On 04/26/2017 05:43 PM, Jim DeVito wrote:
Hi All,
So I am seeing the below record in the ACC output that
is causing me problems else where. Notice the
\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00 part.
INVITE|gK0c48d4aa||1464632400_16750753@REDACTED|600|Busy
Everywhere|1493217263|\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00|+1REDACTED|sip-proxy03|from-PSTN|REDACTED||8877||
In every other ACC entry the value is to be the user
part of the request URI as described here...
modparam("acc", "db_extra", "to_tn=$rU; (etc....)
Except when the call goes through the below failure route.
failure_route[orig_load_balance_fail] {
if (t_was_cancelled()) {
exit();
}
if (t_check_status("[56][0-9][0-9]") ||
t_local_replied("all")) {
if (lb_next()) {
t_on_failure("orig_load_balance_fail");
t_relay();
exit();
} else {
send_reply("600","Busy
Everywhere");
exit();
}
}
}
Thoughts?
Thanks!!
Jim D.
_______________________________________________
Users mailing list
[email protected] <mailto:[email protected]>
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
<http://lists.opensips.org/cgi-bin/mailman/listinfo/users>
--
-------------
Jim DeVito
Mobile 216.507.9497 <tel:%28216%29%20507-9497>
--
-------------
Jim DeVito
Mobile 216.507.9497 <tel:%28216%29%20507-9497>
--
-------------
Jim DeVito
Mobile 216.507.9497
_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
diff --git a/modules/acc/acc_extra.c b/modules/acc/acc_extra.c
index fb98478..467c2fd 100644
--- a/modules/acc/acc_extra.c
+++ b/modules/acc/acc_extra.c
@@ -297,6 +297,9 @@ int extra2strar( struct acc_extra *extra, struct sip_msg *rq,
}
}
+ LM_INFO("EXTRA VALUE as from pvar: flags %d value %.*s\n",
+ value.flags, value.rs.len, value.rs.s);
+
/* check for overflow */
if (n==MAX_ACC_EXTRA) {
LM_WARN("array to short -> ommiting extras for accounting\n");
@@ -311,6 +314,7 @@ int extra2strar( struct acc_extra *extra, struct sip_msg *rq,
/* set the value into the acc buffer */
if (value.rs.s+value.rs.len==static_detector[0] ||
value.rs.s==static_detector[1]) {
+ LM_INFO("setting the value into the static detector!\n");
val_arr[n].s = int_buf[idx] + r*INT2STR_MAX_LEN;
val_arr[n].len = value.rs.len;
memcpy(val_arr[n].s, value.rs.s, value.rs.len);
@@ -319,6 +323,10 @@ int extra2strar( struct acc_extra *extra, struct sip_msg *rq,
val_arr[n] = value.rs;
}
}
+
+
+ LM_INFO("EXTRA VALUE as to be inserted: value %.*s\n",
+ val_arr[n].len, val_arr[n].s);
}
done:
_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
