[Home]

Summary:ASTERISK-12112: [patch] Set fetchid to indicate bad connection id
Reporter:Atis Lezdins (atis)Labels:
Date Opened:2008-05-29 13:11:06Date Closed:2008-05-30 11:44:20
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/app_mysql
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) mysql_fetch_bad_conn.patch
Description:MySQL(Fetch fetchid connid var1 var2..) is setting fetchid depending on rows left, but don't touch it in case of bad connection id.

This requires additional check in dialplan to make sure connection id is correct, if fetchid needs to be used in loops.

This patch sets fetchid to 0 if connection id is wrong.
Comments:By: Michiel van Baak (mvanbaak) 2008-05-29 14:09:53

ok with me.
Thanks
By: Digium Subversion (svnbot) 2008-05-30 11:42:51

Repository: asterisk-addons
Revision: 612

U   trunk/apps/app_addon_sql_mysql.c

------------------------------------------------------------------------
r612 | tilghman | 2008-05-30 11:42:48 -0500 (Fri, 30 May 2008) | 6 lines

When the result id is invalid, indicate that the fetch returned no rows
(closes issue ASTERISK-12112)
Reported by: atis
Patches:
      mysql_fetch_bad_conn.patch uploaded by atis (license 242)

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

http://svn.digium.com/view/asterisk-addons?view=rev&revision=612
By: Digium Subversion (svnbot) 2008-05-30 11:44:20

Repository: asterisk-addons
Revision: 613

_U  branches/1.6.0/
U   branches/1.6.0/apps/app_addon_sql_mysql.c

------------------------------------------------------------------------
r613 | tilghman | 2008-05-30 11:44:19 -0500 (Fri, 30 May 2008) | 14 lines

Merged revisions 612 via svnmerge from
https://origsvn.digium.com/svn/asterisk-addons/trunk

........
r612 | tilghman | 2008-05-30 11:49:06 -0500 (Fri, 30 May 2008) | 6 lines

When the result id is invalid, indicate that the fetch returned no rows
(closes issue ASTERISK-12112)
Reported by: atis
Patches:
      mysql_fetch_bad_conn.patch uploaded by atis (license 242)

........

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

http://svn.digium.com/view/asterisk-addons?view=rev&revision=613