Make sure to cleanup old vm when xFilter is called multiple times

Author: marcobambini

src/cloudsync.h

@@ -16,7 +16,7 @@
 #include "sqlite3.h"
 #endif
 
-#define CLOUDSYNC_VERSION                       "0.8.11"
+#define CLOUDSYNC_VERSION                       "0.8.12"
 
 int sqlite3_cloudsync_init (sqlite3 *db, char **pzErrMsg, const sqlite3_api_routines *pApi);
 

src/vtab.c

@@ -389,6 +389,10 @@ int cloudsync_changesvtab_filter (sqlite3_vtab_cursor *cursor, int idxn, const c
     char *sql = build_changes_sql(db, idxs);
     if (sql == NULL) return SQLITE_NOMEM;
 
+    // the xFilter method may be called multiple times on the same sqlite3_vtab_cursor*
+    if (c->vm) sqlite3_finalize(c->vm);
+    c->vm = NULL;
+    
     int rc = sqlite3_prepare_v2(db, sql, -1, &c->vm, NULL);
     cloudsync_memory_free(sql);
     if (rc != SQLITE_OK) goto abort_filter;