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

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

<p>Author: Chris Double <a href="mailto:doublec@silentcircle.com">doublec@silentcircle.com</a></p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/kamailio/kamailio/pull/611'>https://github.com/kamailio/kamailio/pull/611</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Free database column name as part of database result cleanup</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-0">lib/srdb1/db_res.c</a>
    (3)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-1">modules/db_berkeley/km_bdb_res.c</a>
    (9)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-2">modules/db_mongodb/mongodb_dbase.c</a>
    (9)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-3">modules/db_mysql/km_res.c</a>
    (9)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-4">modules/db_postgres/km_res.c</a>
    (10)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-5">modules/db_text/dbt_api.c</a>
    (9)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-6">modules/db_unixodbc/res.c</a>
    (8)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/kamailio/kamailio/pull/611/files#diff-7">modules/xmlrpc/xmlrpc.c</a>
    (2)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/kamailio/kamailio/pull/611.patch'>https://github.com/kamailio/kamailio/pull/611.patch</a></li>
  <li><a href='https://github.com/kamailio/kamailio/pull/611.diff'>https://github.com/kamailio/kamailio/pull/611.diff</a></li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly or <a href="https://github.com/kamailio/kamailio/pull/611">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZS5RnOGwaUWI3y62oWtsSezpXuHmks5qAP0hgaJpZM4Ibi7N.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/pull/611"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>