[sr-dev] [kamailio/kamailio] Free database column name as part of database result cleanup (#611)

William King notifications at github.com
Tue May 10 23:48:17 CEST 2016


The column name is currently not free'd. Some db bankends copy this
data so that memory is leaked. Some store internal database pointers
and those shouldn't be free'd. One returns a pointer to a stack
variable which shouldn't be done.

The patch cleans up all db backends to copy the column name and frees
that column name as part of the database result cleanup function.

Author: Chris Double <doublec at silentcircle.com>
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/611

-- Commit Summary --

  * Free database column name as part of database result cleanup

-- File Changes --

    M lib/srdb1/db_res.c (3)
    M modules/db_berkeley/km_bdb_res.c (9)
    M modules/db_mongodb/mongodb_dbase.c (9)
    M modules/db_mysql/km_res.c (9)
    M modules/db_postgres/km_res.c (10)
    M modules/db_text/dbt_api.c (9)
    M modules/db_unixodbc/res.c (8)
    M modules/xmlrpc/xmlrpc.c (2)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/611.patch
https://github.com/kamailio/kamailio/pull/611.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/611
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160510/2950c226/attachment-0001.html>


More information about the sr-dev mailing list