[tarantool-patches] [PATCH v1 01/10] box: store sql text and length in sql_request

[imeevma@tarantool.org]

Refactored sql_request structure to store pointer to sql string
data and it's length instead of pointer to msgpack
representation.
This is required to use this structure in sql.c where the query
has a different semantics and can be obtained from stack as a C
string.

Needed for #3505.
---
 src/box/execute.c | 6 ++++--
 src/box/execute.h | 2 ++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/box/execute.c b/src/box/execute.c
index e450444..ce9dd83 100644
--- a/src/box/execute.c
+++ b/src/box/execute.c
@@ -285,6 +285,9 @@ error:
 				return -1;
 		} else {
 			request->sql_text = value;
+			request->sql_text =
+				mp_decode_str(&request->sql_text,
+					      &request->sql_text_len);
 		}
 	}
 	if (request->sql_text == NULL) {
@@ -586,8 +589,7 @@ sql_prepare_and_execute(const struct sql_request *request,
 			struct sql_response *response, struct region *region)
 {
 	const char *sql = request->sql_text;
-	uint32_t len;
-	sql = mp_decode_str(&sql, &len);
+	uint32_t len = request->sql_text_len;
 	struct sqlite3_stmt *stmt;
 	sqlite3 *db = sql_get();
 	if (db == NULL) {
diff --git a/src/box/execute.h b/src/box/execute.h
index 77bfd79..79cee69 100644
--- a/src/box/execute.h
+++ b/src/box/execute.h
@@ -58,6 +58,8 @@ struct sql_request {
 	uint64_t sync;
 	/** SQL statement text. */
 	const char *sql_text;
+	/** Length of the SQL statement text. */
+	uint32_t sql_text_len;
 	/** Array of parameters. */
 	struct sql_bind *bind;
 	/** Length of the @bind. */
-- 
2.7.4