-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3841/
-----------------------------------------------------------

(Updated July 24, 2014, 4:01 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Joshua Colp.


Changes
-------

Committed in revision 419504


Bugs: ASTERISK-24050
    https://issues.asterisk.org/jira/browse/ASTERISK-24050


Repository: Asterisk


Description
-------

The CI testing found a regression with the last patch.  Yea!  The UPDATE was 
not functioning as expected.  I am pretty sure I tested UPDATES but obviously 
missed something.

sqlite3_prepare() does not handle multiple SQL statements in a single call.  If 
there are more than one present, it will only use the first statement and 
ignore the rest.

This patch separates the INSERT INTO and the UPDATE into two separate 
statements for use by sqlite3_prepare.  These two statements are then run when 
ast_db_put() is called.


Diffs
-----

  /trunk/main/db.c 419309 

Diff: https://reviewboard.asterisk.org/r/3841/diff/


Testing
-------

I turned on tracing in order to see this problem and also to confirm that this 
patch was functioning again as expected. 

I ran the testsuite test that was failing before the patch and then after the 
patch to confirm that the test no passes as well.


Thanks,

Michael Young

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to