<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8681">View Change</a></p><p>Patch set 4:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think converting the string to an int is the right<br>approach.  The origtime is a timestamp which has a varying<br>integer<br>size type depending upon the machine.  The time_t type could be<br>"long int" or "long long int".<br>> In ASTERISK-27760, you said the MariaDB had a problem with the<br>empty string and not a number string.  It seems that the MariaDB<br>is<br>treating the duration and origtime columns as "INTEGER NOT NULL"<br>instead of "INTEGER" or "INTEGER NULL" type columns.  To avoid<br>type<br>conversion problems we should instead make sure that the duration<br>and origtime strings are not empty in store_file() right before<br>the<br>call to ast_odbc_direct_execute().  If they are empty then we can<br>substitute the "0" string instead.<br>> if (ast_strlen_zero(idata.origtime)) {<br>idata.origtime = "0";<br>}<br>if (ast_strlen_zero(idata.duration)) {<br>idata.duration = "0";<br>}</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Good point on the time_t being a long on some machines. I guess if<br>this is the approach we should update the database schema also in<br>/contrib/realtime/mysql_voicemail.sql since its specified as an<br>integer in there (which is why I chose an int for the conversion in<br>the first place). Both approaches (int to DB and string to DB) will<br>start overflowing peoples database integer in 2038.<br>I'm on holiday now for a while. Ill try and look at these changes<br>when I return.<br>Cheers, Nic.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think it is the database implementation that determines the maximum values allowed by integer columns.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8681">change 8681</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/8681"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8543ad480a08c98be78bde1ee870e6e6c84b2c5b </div>
<div style="display:none"> Gerrit-Change-Number: 8681 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Nic Colledge <nic@njcolledge.net> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nic Colledge <nic@njcolledge.net> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 10 Apr 2018 22:40:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>