This patch fixes the computation of the allocation size for the query. It was not taking into account the length of the name of the procedure. Signed-off-by: Eric Leblond <eric@xxxxxx> --- util/db.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/util/db.c b/util/db.c index 3cfe057..5cf5fe1 100644 --- a/util/db.c +++ b/util/db.c @@ -54,7 +54,7 @@ static int disabled_interp_db(struct ulogd_pluginstance *upi) return 0; } -#define SQL_INSERTTEMPL "insert into X (Y) values (Z)" +#define SQL_INSERTTEMPL "SELECT P(Y)" #define SQL_VALSIZE 100 /* create the static part of our insert statement */ @@ -80,7 +80,8 @@ static int sql_createstmt(struct ulogd_pluginstance *upi) /* we need space for the key and a comma, as well as * enough space for the values */ size += strlen(upi->input.keys[i].name) + 1 + SQL_VALSIZE; - } + } + size += srtlen(procedure); ulogd_log(ULOGD_DEBUG, "allocating %u bytes for statement\n", size); -- 1.5.5.1 -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
