]> git.ipfire.org Git - thirdparty/ulogd2.git/commit
output: SQLITE3: improve formatting of insert statement
authorJeremy Sowden <jeremy@azazel.net>
Tue, 30 Nov 2021 10:55:48 +0000 (10:55 +0000)
committerPablo Neira Ayuso <pablo@netfilter.org>
Mon, 3 Jan 2022 14:00:49 +0000 (15:00 +0100)
commit53d669d95c94aab4366b8fe9544d61369aa384b5
tree5cb0680291cd6f3823dd3b32744615b3609a07f7
parent1d9fd9d64b17b48dd0b38bdebb7086c88a649e1b
output: SQLITE3: improve formatting of insert statement

`sqlite3_createstmt` contains a variable `stmt_pos` which points to the
end of the SQL already written, where the next chunk should be appended.
Currently, this is assigned after every write:

  sprintf(stmt_pos, ...);
  stmt_pos = priv->stmt + strlen(priv->stmt);

However, since `sprintf` returns the number of bytes written, increment
`stmt_pos` by the return-value of `sprintf` in order to avoid the
repeated `strlen` calls.

Pablo mangled this original patch to add this chunk at the end of this
patch (originally submitted as a conversion to use strcpy).

+       for (i = 0; i < cols - 1; i++)
+               stmt_pos += sprintf(stmt_pos, "?,");

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
output/sqlite3/ulogd_output_SQLITE3.c