X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fappfw%2Fpkgmgr-info.git;a=blobdiff_plain;f=src%2Fpkgmgrinfo_certinfo.c;h=3b346bc28e8b56ae62430d5743556d89a3f472b7;hp=6f3de1f1fa3877556528fbab8afd09d84d847a66;hb=923c0a1db9905d7d727b13eaf4174e66699e940c;hpb=46e0e1115fa53315290eed79335439d185ca937c

diff --git a/src/pkgmgrinfo_certinfo.c b/src/pkgmgrinfo_certinfo.c
index 6f3de1f..3b346bc 100644
--- a/src/pkgmgrinfo_certinfo.c
+++ b/src/pkgmgrinfo_certinfo.c
@@ -583,9 +583,24 @@ static int _pkginfo_save_cert_index_info(sqlite3 *db, char *cert_info[])
 		if (cert_info[i] == NULL)
 			continue;
 		idx = 1;
-		sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
-		sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
-		sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
+		ret = sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
+		if (ret != SQLITE_OK) {
+			_LOGE("bind failed: %s", sqlite3_errmsg(db));
+			sqlite3_finalize(stmt);
+			return PMINFO_R_ERROR;
+		}
+		ret = sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
+		if (ret != SQLITE_OK) {
+			_LOGE("bind failed: %s", sqlite3_errmsg(db));
+			sqlite3_finalize(stmt);
+			return PMINFO_R_ERROR;
+		}
+		ret = sqlite3_bind_text(stmt, idx++, cert_info[i], -1, SQLITE_STATIC);
+		if (ret != SQLITE_OK) {
+			_LOGE("bind failed: %s", sqlite3_errmsg(db));
+			sqlite3_finalize(stmt);
+			return PMINFO_R_ERROR;
+		}
 
 		ret = sqlite3_step(stmt);
 		if (ret != SQLITE_DONE) {
@@ -639,14 +654,18 @@ API int pkgmgrinfo_save_certinfo(const char *pkgid, pkgmgrinfo_instcertinfo_h ha
 
 	if (_pkginfo_save_cert_index_info(db, info->cert_info)) {
 		_LOGE("failed to save cert index info, rollback now");
-		sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		ret = sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		if (ret != SQLITE_OK)
+			LOGE("Rollback is failed. error(%s)", sqlite3_errmsg(db));
 		sqlite3_close_v2(db);
 		return PMINFO_R_ERROR;
 	}
 
 	if (_pkginfo_save_cert_info(db, pkgid, info->cert_info)) {
 		_LOGE("failed to save cert info, rollback now");
-		sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		ret = sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		if (ret != SQLITE_OK)
+			LOGE("Rollback is failed. error(%s)", sqlite3_errmsg(db));
 		sqlite3_close_v2(db);
 		return PMINFO_R_ERROR;
 	}
@@ -654,7 +673,9 @@ API int pkgmgrinfo_save_certinfo(const char *pkgid, pkgmgrinfo_instcertinfo_h ha
 	ret = sqlite3_exec(db, "COMMIT", NULL, NULL, NULL);
 	if (ret != SQLITE_OK) {
 		_LOGE("failed to commit transaction, rollback now");
-		sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		ret = sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		if (ret != SQLITE_OK)
+			LOGE("Rollback is failed. error(%s)", sqlite3_errmsg(db));
 		sqlite3_close_v2(db);
 		return PMINFO_R_ERROR;
 	}
@@ -750,7 +771,9 @@ API int pkgmgrinfo_delete_usr_certinfo(const char *pkgid, uid_t uid)
 
 	if (_pkginfo_delete_certinfo(db, pkgid)) {
 		_LOGE("failed to delete certinfo of %s, rollback now", pkgid);
-		sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		ret = sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		if (ret != SQLITE_OK)
+			LOGE("Rollback is failed. error(%s)", sqlite3_errmsg(db));
 		sqlite3_close_v2(db);
 		return PMINFO_R_ERROR;
 	}
@@ -758,7 +781,9 @@ API int pkgmgrinfo_delete_usr_certinfo(const char *pkgid, uid_t uid)
 	ret = sqlite3_exec(db, "COMMIT", NULL, NULL, NULL);
 	if (ret != SQLITE_OK) {
 		_LOGE("failed to commit transaction, rollback now");
-		sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		ret = sqlite3_exec(db, "ROLLBACK", NULL, NULL, NULL);
+		if (ret != SQLITE_OK)
+			LOGE("Rollback is failed. error(%s)", sqlite3_errmsg(db));
 		sqlite3_close_v2(db);
 		return PMINFO_R_ERROR;
 	}