Module: kamailio
Branch: master
Commit: 7289417c39939f61ea116fe8dc8d23cdea6401be
URL: https://github.com/kamailio/kamailio/commit/7289417c39939f61ea116fe8dc8d23c…
Author: Daniel-Constantin Mierla <miconda(a)gmail.com>
Committer: Daniel-Constantin Mierla <miconda(a)gmail.com>
Date: 2016-09-19T20:58:31+02:00
presence: free db result when get_dialog_state() retrieves no rows
- db result and columns in the query are allocated
---
Modified: modules/presence/presentity.c
---
Diff: https://github.com/kamailio/kamailio/commit/7289417c39939f61ea116fe8dc8d23c…
Patch: https://github.com/kamailio/kamailio/commit/7289417c39939f61ea116fe8dc8d23c…
---
diff --git a/modules/presence/presentity.c b/modules/presence/presentity.c
index b5d7b65..d81b90e 100644
--- a/modules/presence/presentity.c
+++ b/modules/presence/presentity.c
@@ -503,9 +503,11 @@ int get_dialog_state(presentity_t* presentity, char** state)
if(result == NULL)
return -3;
- // No results from query definitely means no dialog exists
- if (result->n <= 0)
+ /* no results from query definitely means no dialog exists */
+ if (result->n <= 0) {
+ pa_dbf.free_result(pa_db, result);
return 0;
+ }
// Loop the rows returned from the DB
for (i=0; i < result->n; i++)
Hi @vingarzan
I just want to check with you before I do a change related to this commit.
With this change Rx AAR for subscription to signalling status now uses the general flow-usage: AVP_EPC_Flow_Usage_No_Information instead of AVP_EPC_Flow_Usage_AF_Signaling as it previously did.
I want to change this so the the correct flow-usage is used (even though there is now a proper flow description as well). Let me know if agreed.
Regards
Richard.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/commit/ab3d42c47d52b1b7b315272c00daf12…
This helps first timers and casual users a lot, as they won't have to dig
through the history of Kamailio and various modules to find these
structures.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/718
-- Commit Summary --
* examples/[pis]cscf: adds example database structures for the P/I/S-CSCFs
-- File Changes --
A examples/icscf/icscf.mysql.sql (137)
A examples/pcscf/pcscf.mysql.sql (193)
A examples/scscf/scscf.mysql.sql (448)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/718.patchhttps://github.com/kamailio/kamailio/pull/718.diff
---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/718