Browse Source

Little test for the latest fix.

ccgus 12 years ago
parent
commit
8e3921257e
3 changed files with 6 additions and 0 deletions
  1. 1 0
      CHANGES_AND_TODO_LIST.txt
  2. 1 0
      CONTRIBUTORS.txt
  3. 4 0
      src/fmdb.m

+ 1 - 0
CHANGES_AND_TODO_LIST.txt

@@ -6,6 +6,7 @@ If you would like to contribute some code- awesome!  I just ask that you make it
 2013.05.24
     Merged in Chris Wright's date format additions to FMDatabase.
     Fixed a problem where executeUpdateWithFormat: + %@ as a placeholder and the value was nil would cause a bad value to be inserted.  Thanks to rustybox on github for the fix.
+     Fixed a variable argument crash if an incorrect number of arguments are passed in - patch by Joshua Tessier.
 
 2013.04.17
     Added two new methods to FMDatabase for setting crypto keys, which take NSData objects.  Thanks to Phillip Kast for the patch! <https://github.com/ccgus/fmdb/pull/135>

+ 1 - 0
CONTRIBUTORS.txt

@@ -39,5 +39,6 @@ Peter Carr
 Jim Correia
 Phillip Kast
 Chris Wright
+Joshua Tessier
 
 Aaaaannnd, Gus Mueller (that's me!)

+ 4 - 0
src/fmdb.m

@@ -727,6 +727,10 @@ int main (int argc, const char * argv[]) {
         
     }
     
+    {
+        rs = [db executeQuery:@"select * from t5 where a=?" withArgumentsInArray:@[]];
+        FMDBQuickCheck((![rs next]));
+    }
     
     // test attach for the heck of it.
     {