X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fsrc%2Flookups%2Fsqlite.c;h=dc4439153bd991650855d068405fbf99284fd1cb;hb=refs%2Ftags%2Fexim-4_94_RC0;hp=ee8930d3c96413dd2673a69349207f609dd84974;hpb=c5db348c5e29e93e51389fa0079f829967c5da82;p=user%2Fhenk%2Fcode%2Fexim.git diff --git a/src/src/lookups/sqlite.c b/src/src/lookups/sqlite.c index ee8930d3c..dc4439153 100644 --- a/src/src/lookups/sqlite.c +++ b/src/src/lookups/sqlite.c @@ -2,7 +2,8 @@ * Exim - an Internet mail transport agent * *************************************************/ -/* Copyright (c) University of Cambridge 1995 - 2017 */ +/* Copyright (c) University of Cambridge 1995 - 2018 */ +/* Copyright (c) The Exim Maintainers 2020 */ /* See the file NOTICE for conditions of use and distribution. */ #include "../exim.h" @@ -18,16 +19,18 @@ /* See local README for interface description. */ static void * -sqlite_open(uschar *filename, uschar **errmsg) +sqlite_open(const uschar * filename, uschar ** errmsg) { sqlite3 *db = NULL; int ret; -ret = sqlite3_open(CS filename, &db); -if (ret != 0) +if (!filename || !*filename) filename = sqlite_dbfile; +if (*filename != '/') + *errmsg = US"absolute file name expected for \"sqlite\" lookup"; +else if ((ret = sqlite3_open(CCS filename, &db)) != 0) { *errmsg = (void *)sqlite3_errmsg(db); - debug_printf("Error opening database: %s\n", *errmsg); + DEBUG(D_lookup) debug_printf_indent("Error opening database: %s\n", *errmsg); } sqlite3_busy_timeout(db, 1000 * sqlite_lock_timeout); @@ -45,7 +48,6 @@ static int sqlite_callback(void *arg, int argc, char **argv, char **azColName) { gstring * res = *(gstring **)arg; -int i; /* For second and subsequent results, insert \n */ @@ -55,7 +57,7 @@ if (res) if (argc > 1) { /* For multiple fields, include the field name too */ - for (i = 0; i < argc; i++) + for (int i = 0; i < argc; i++) { uschar *value = US((argv[i] != NULL)? argv[i]:""); res = lf_quote(US azColName[i], value, Ustrlen(value), res); @@ -71,16 +73,17 @@ return 0; static int -sqlite_find(void *handle, uschar *filename, const uschar *query, int length, - uschar **result, uschar **errmsg, uint *do_cache) +sqlite_find(void * handle, const uschar * filename, const uschar * query, + int length, uschar ** result, uschar ** errmsg, uint * do_cache, + const uschar * opts) { int ret; gstring * res = NULL; -ret = sqlite3_exec(handle, CS query, sqlite_callback, &res, (char **)errmsg); +ret = sqlite3_exec(handle, CS query, sqlite_callback, &res, CSS errmsg); if (ret != SQLITE_OK) { - debug_printf("sqlite3_exec failed: %s\n", *errmsg); + debug_printf_indent("sqlite3_exec failed: %s\n", *errmsg); return FAIL; } @@ -132,7 +135,7 @@ if (opt != NULL) return NULL; /* No options recognized */ while ((c = *t++) != 0) if (c == '\'') count++; if (count == 0) return s; -t = quoted = store_get(Ustrlen(s) + count + 1); +t = quoted = store_get(Ustrlen(s) + count + 1, is_tainted(s)); while ((c = *s++) != 0) { @@ -166,15 +169,15 @@ fprintf(f, " Exim version %s\n", EXIM_VERSION_STR); } static lookup_info _lookup_info = { - US"sqlite", /* lookup name */ - lookup_absfilequery, /* query-style lookup, starts with file name */ - sqlite_open, /* open function */ - NULL, /* no check function */ - sqlite_find, /* find function */ - sqlite_close, /* close function */ - NULL, /* no tidy function */ - sqlite_quote, /* quoting function */ - sqlite_version_report /* version reporting */ + .name = US"sqlite", /* lookup name */ + .type = lookup_absfilequery, /* query-style lookup, starts with file name */ + .open = sqlite_open, /* open function */ + .check = NULL, /* no check function */ + .find = sqlite_find, /* find function */ + .close = sqlite_close, /* close function */ + .tidy = NULL, /* no tidy function */ + .quote = sqlite_quote, /* quoting function */ + .version_report = sqlite_version_report /* version reporting */ }; #ifdef DYNLOOKUP