[asterisk-bugs] [Asterisk 0018525]: [patch] ast_update2_realtime fails to update multiple columns with MySQL driver

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18525 
====================================================================== 
Reported By:                makoto
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   18525
Category:                   Addons/res_config_mysql
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           Addons-1.6.2.1 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-12-24 02:10 CST
Last Modified:              2010-12-24 11:58 CST
====================================================================== 
Summary:                    [patch] ast_update2_realtime fails to update
multiple columns with MySQL driver
Description: 
For example, the following code fails with MySQL driver.

  ast_update2_realtime("table", "id", "1234", SENTINEL, "foo", "yes",
"bar", "no", SENTINEL);


This doesn't happens when updating single column.

update2_mysql function in res_config_mysql.c generates an incorrect
SQL sentence when updating multiple columns because variable first
is not cleared in the second while loop.

Attached patch will fix the problem.

PostgreSQL driver seems to have the same problem.

====================================================================== 

---------------------------------------------------------------------- 
 (0129950) svnbot (reporter) - 2010-12-24 11:58
 https://issues.asterisk.org/view.php?id=18525#c129950 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 299583

U   branches/1.8/addons/res_config_mysql.c

------------------------------------------------------------------------
r299583 | tilghman | 2010-12-24 11:58:30 -0600 (Fri, 24 Dec 2010) | 7
lines

Reset 'first' variable after usage.

(closes issue https://issues.asterisk.org/view.php?id=18525)
 Reported by: makoto
 Patches: 
       res-config-mysql-update2.patch uploaded by makoto (license 38)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=299583 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-12-24 11:58 svnbot         Checkin                                      
2010-12-24 11:58 svnbot         Note Added: 0129950                          
======================================================================