-rw-r--r-- | libopie2/opiedb/osqlitedriver.cpp | 46 | ||||
-rw-r--r-- | libopie2/opiedb/osqlitedriver.h | 8 |
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 @@ -41,9 +41,6 @@ // is replaced by fromLatin1() (eilers) #define __BUGGY_LOCAL8BIT_ - char *regex_raw; - regex_t regex_c; - using namespace Opie::DB; using namespace Opie::DB::Internal; @@ -89,32 +86,36 @@ 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(®ex_c); - } - regex_raw = (char *)malloc(strlen(zPattern)+1); - strncpy(regex_raw, zPattern, strlen(zPattern)+1); - res = regcomp(®ex_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(®->regex_c); + } + reg->regex_raw = (char *)malloc(strlen(zPattern)+1); + strncpy(reg->regex_raw, zPattern, strlen(zPattern)+1); + res = regcomp(®->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(®ex_c, zString, 0, NULL, 0)==0); + res = (regexec(®->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])); } /* @@ -135,7 +136,8 @@ bool OSQLiteDriver::open() { 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; } @@ -147,9 +149,11 @@ bool OSQLiteDriver::open() { bool OSQLiteDriver::close() { if (m_sqlite ) sqlite_close( m_sqlite ), m_sqlite=0l; - free(regex_raw); - regex_raw=NULL; - regfree(®ex_c); + if (sqreg->regex_raw != NULL){ + free(sqreg->regex_raw); + sqreg->regex_raw=NULL; + regfree(&sqreg->regex_c); + } return true; } 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 @@ -2,6 +2,7 @@ #define OSQL_LITE_DRIVER_H #include <sqlite.h> +#include <regex.h> #include "osqldriver.h" #include "osqlerror.h" @@ -11,6 +12,11 @@ namespace Opie { namespace DB { namespace Internal { +struct sqregex { + char *regex_raw; + regex_t regex_c; +}; + class OSQLiteDriver : public OSQLDriver { Q_OBJECT public: @@ -26,6 +32,7 @@ public: OSQLError lastError(); OSQLResult query( OSQLQuery* ); OSQLTable::ValueList tables()const; + private: OSQLError m_lastE; OSQLResult m_result; @@ -34,6 +41,7 @@ private: static int call_back( void*, int, char**, char** ); QString m_url; sqlite *m_sqlite; + sqregex *sqreg; }; } } |