<p>The avp population of column values from avp_db_query equaling NULL are skipped, sacrificing the integrity of the actual database result.</p>

<p>Basically, the old method checked if the column value structure's nul member was 1 and skipped to the next avp:</p>

<pre><code>if (RES_ROWS(db_res)[i].values[j].nul)
        goto next_avp;
</code></pre>

<p>I moved this check to column values of type DB_INT, since that is the default type, and instead of skipping the avp, assign the value type to AVP_VAL_NULL:</p>

<pre><code>avp_type |= AVP_VAL_NULL;
</code></pre>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>&nbsp;&nbsp;<a href='https://github.com/OpenSIPS/opensips/pull/693'>https://github.com/OpenSIPS/opensips/pull/693</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Maintain database result integrity by supporting null column values</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/OpenSIPS/opensips/pull/693/files#diff-0">modules/avpops/avpops_db.c</a>
    (12)
  </li>
</ul>

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

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href="https://github.com/OpenSIPS/opensips/pull/693">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AFOcifAxXAJDi_RRL-vMZvGS5tEQNAHJks5pEPUggaJpZM4Ge41v.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/OpenSIPS/opensips/pull/693"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>