Made sqlite regex cache "instance safe". Further work will continue as I figure out

how to do it :p)

2 files changed, 33 insertions, 21 deletions

diff --git a/libopie2/opiedb/osqlitedriver.cpp b/libopie2/opiedb/osqlitedriver.cpp
index 4eda9b9..f07d520 100644
--- a/libopie2/opiedb/osqlitedriver.cpp
+++ b/libopie2/opiedb/osqlitedriver.cpp
@@ -38,15 +38,12 @@
 #include <stdio.h>
 
 // fromLocal8Bit() does not work as expected. Thus it
 // is replaced by fromLatin1() (eilers)
 #define __BUGGY_LOCAL8BIT_
 
-        char *regex_raw;
-        regex_t regex_c;
-
 using namespace Opie::DB;
 using namespace Opie::DB::Internal;
 
 namespace {
     struct Query {
         OSQLError::ValueList errors;
@@ -86,38 +83,42 @@ void OSQLiteDriver::setUrl( const QString& url ) {
 void OSQLiteDriver::setOptions( const QStringList& ) {
 }
 
 /*
  * Functions to patch a regex search into sqlite
  */
-int sqliteRlikeCompare(const char *zPattern, const char *zString){
+int sqliteRlikeCompare(const char *zPattern, const char *zString, sqregex *reg){
         int res;
-        if (regex_raw == NULL || (strcmp (zPattern, regex_raw) != 0)){
-                if (regex_raw != NULL) {
-                    free(regex_raw);
-                    regfree(&regex_c);
-                }
-                regex_raw = (char *)malloc(strlen(zPattern)+1);
-                strncpy(regex_raw, zPattern, strlen(zPattern)+1);
-                res = regcomp(&regex_c, zPattern, REG_EXTENDED);
+
+        if (reg->regex_raw == NULL || (strcmp (zPattern, reg->regex_raw) != 0)){
+                if (reg->regex_raw != NULL) {
+                    free(reg->regex_raw);
+                    regfree(&reg->regex_c);
+                }
+                reg->regex_raw = (char *)malloc(strlen(zPattern)+1);
+                strncpy(reg->regex_raw, zPattern, strlen(zPattern)+1);
+                res = regcomp(&reg->regex_c, zPattern, REG_EXTENDED);
                 if ( res != 0 ) {
                     printf("Regcomp failed with code %u on string %s\n",res,zPattern);
-                    free(regex_raw);
-                    regex_raw=NULL;
+                    free(reg->regex_raw);
+                    reg->regex_raw=NULL;
                 return 0;
                 }
         }
-        res = (regexec(&regex_c, zString, 0, NULL, 0)==0);
+        res = (regexec(&reg->regex_c, zString, 0, NULL, 0)==0);
         return res;
 }
 
 void rlikeFunc(sqlite_func *context, int arg, const char **argv){
-        if( argv[0]==0 || argv[1]==0 ) return;
+        if( argv[0]==0 || argv[1]==0 || argv[2]==0){
+                printf("One of arguments Null!!\n");
+                return;
+        }
                 sqlite_set_result_int(context,
                 sqliteRlikeCompare((const char*)argv[0],
-        (const char*)argv[1]));
+        (const char*)argv[1], (sqregex*)argv[2]));
 }
 
 /*
  * try to open a db specified via setUrl
  * and options
  */
@@ -132,27 +133,30 @@ bool OSQLiteDriver::open() {
     if (m_sqlite == 0l ) {
         // FIXME set the last error
         qWarning("OSQLiteDriver::open: %s", error );
         free( error );
         return false;
     }
-    sqlite_create_function(m_sqlite,"rlike",2,rlikeFunc,NULL);
+    sqreg = (sqregex *)malloc(sizeof(sqreg));
+    sqlite_create_function(m_sqlite,"rlike",3,rlikeFunc,&sqreg);
     return true;
 }
 
 
 /* close the db
  * sqlite closes them without
  * telling failure or success
  */
 bool OSQLiteDriver::close() {
     if (m_sqlite )
         sqlite_close( m_sqlite ), m_sqlite=0l;
-        free(regex_raw);
-        regex_raw=NULL;
-        regfree(&regex_c);
+        if (sqreg->regex_raw != NULL){
+                free(sqreg->regex_raw);
+            sqreg->regex_raw=NULL;
+            regfree(&sqreg->regex_c);
+        }
     return true;
 }
 
 
 /* Query */
 OSQLResult OSQLiteDriver::query( OSQLQuery* qu) {
diff --git a/libopie2/opiedb/osqlitedriver.h b/libopie2/opiedb/osqlitedriver.h
index 9064e52..95c9e2f 100644
--- a/libopie2/opiedb/osqlitedriver.h
+++ b/libopie2/opiedb/osqlitedriver.h
@@ -1,19 +1,25 @@
 #ifndef OSQL_LITE_DRIVER_H
 #define OSQL_LITE_DRIVER_H
 
 #include <sqlite.h>
+#include <regex.h>
 
 #include "osqldriver.h"
 #include "osqlerror.h"
 #include "osqlresult.h"
 
 namespace Opie {
 namespace DB {
 namespace Internal {
 
+struct sqregex {
+        char *regex_raw;
+        regex_t regex_c;
+};
+
 class OSQLiteDriver : public OSQLDriver {
     Q_OBJECT
 public:
     OSQLiteDriver( QLibrary *lib = 0l );
     ~OSQLiteDriver();
     QString id()const;
@@ -23,20 +29,22 @@ public:
     void setOptions( const QStringList& );
     bool open();
     bool close();
     OSQLError lastError();
     OSQLResult query( OSQLQuery* );
     OSQLTable::ValueList tables()const;
+
 private:
     OSQLError m_lastE;
     OSQLResult m_result;
     OSQLResultItem m_items;
     int handleCallBack( int, char**, char** );
     static int call_back( void*, int, char**, char** );
     QString m_url;
     sqlite *m_sqlite;
+    sqregex *sqreg;
 };
 }
 }
 }
 
 #endif