From: Mauro Carvalho Chehab <mchehab@redhat.com>
Date: Wed, 29 May 2013 10:41:30 +0000 (-0300)
Subject: ras-record: use sqlite3_reset to allow reusing the prepared statement
X-Git-Tag: v0.4.1~7
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=bba290838f3db98a646219730f607e017762e11f;p=users%2Fmchehab%2Frasdaemon.git

ras-record: use sqlite3_reset to allow reusing the prepared statement

Instead of using sqlite3_finalize, we should use sqlite3_reset, or
otherwise the prepared statement will be de-allocated.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
---

diff --git a/ras-record.c b/ras-record.c
index 052875b..8995c9e 100644
--- a/ras-record.c
+++ b/ras-record.c
@@ -169,7 +169,7 @@ int ras_store_mc_event(struct ras_events *ras, struct ras_mc_event *ev)
 
 	if (!priv || !priv->stmt)
 		return 0;
-	log(TERM, LOG_INFO, "store_event: %p\n", priv->stmt);
+	log(TERM, LOG_INFO, "mc_event store: %p\n", priv->stmt);
 
 	sqlite3_bind_text(priv->stmt,  1, ev->timestamp, -1, NULL);
 	sqlite3_bind_int (priv->stmt,  2, ev->error_count);
@@ -186,10 +186,10 @@ int ras_store_mc_event(struct ras_events *ras, struct ras_mc_event *ev)
 	sqlite3_bind_text(priv->stmt, 13, ev->driver_detail, -1, NULL);
 	rc = sqlite3_step(priv->stmt);
 	if (rc != SQLITE_OK && rc != SQLITE_DONE)
-		log(TERM, LOG_ERR, "Failed to do step on sqlite: error = %d\n", rc);
-	rc = sqlite3_finalize(priv->stmt);
+		log(TERM, LOG_ERR, "Failed to do mc_event step on sqlite: error = %d\n", rc);
+	rc = sqlite3_reset(priv->stmt);
 	if (rc != SQLITE_OK && rc != SQLITE_DONE)
-		log(TERM, LOG_ERR, "Failed to do finalize insert on sqlite: error = %d\n",
+		log(TERM, LOG_ERR, "Failed reset mc_event on sqlite: error = %d\n",
 		       rc);
 	log(TERM, LOG_INFO, "register inserted at db\n");