Module: sip-router
Branch: lazedo/kazoo
Commit: dae74e974a028082a3250805da83223220123c10
URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=dae74e9…
Author: Luis Azedo <luis(a)2600hz.com>
Committer: Luis Azedo <luis(a)2600hz.com>
Date: Wed Sep 10 15:23:45 2014 +0100
proposed change for dbtext query results.
this behaviour was observed by interacting with module presence.
presentity clean runs from timer every 6 secs.
if the first run returns a large set of rows, it iterates all rows doing
some notifications and deleting the presentity.
if it takes more then 6 secs to complete, the next run will override the
results internal structure pointer maintained in the connection level.
the proposed patch, maintains the query result internal structure within
the result, using the ptr field for that.
---
modules/db_text/dbt_api.c | 86 ++++++++++++++++++++-----------------------
modules/db_text/dbt_api.h | 3 +-
modules/db_text/dbt_base.c | 27 +++++++-------
modules/db_text/dbt_res.c | 4 ++-
modules/db_text/dbt_res.h | 6 ---
5 files changed, 59 insertions(+), 67 deletions(-)
Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=dae…
THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.
The following task has a new comment added:
FS#467 - coredump when invoking 'kamcmd ul.add ...' with path parameter set to 's:0' and debug level >= 3
User who did this - Thomas Arimont (topotomaso)
----------
Yes, you're right. I'm sorry.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=467#comment1621
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.
The following task has a new comment added:
FS#467 - coredump when invoking 'kamcmd ul.add ...' with path parameter set to 's:0' and debug level >= 3
User who did this - Daniel-Constantin Mierla (miconda)
----------
I assume 'ci.patch' is having a typo and you refer to 'ci.path', right?
I am going to apply the patch.
----------
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=467#comment1620
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.
The following task has a new comment added:
FS#467 - coredump when invoking 'kamcmd ul.add ...' with path parameter set to 's:0' and debug level >= 3
User who did this - Thomas Arimont (topotomaso)
----------
It is even worser. In the case I've described ci.patch is already NULL.
See patch proposal.
----------
One or more files have been attached.
More information can be found at the following URL:
http://sip-router.org/tracker/index.php?do=details&task_id=467#comment1619
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.