From 3c65fba539e5932c8ac5129b54bae657b33278d9 Mon Sep 17 00:00:00 2001 From: Fabio Bas Date: Thu, 29 Jan 2015 09:54:48 +0100 Subject: [PATCH] Avoid double locking of sessions, users table QSqlQuery already executes not-empty queries, no need to call exec() --- servatrice/src/servatrice_database_interface.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/servatrice/src/servatrice_database_interface.cpp b/servatrice/src/servatrice_database_interface.cpp index 55f9bf61..1bd006f7 100644 --- a/servatrice/src/servatrice_database_interface.cpp +++ b/servatrice/src/servatrice_database_interface.cpp @@ -330,12 +330,12 @@ void Servatrice_DatabaseInterface::clearSessionTables() void Servatrice_DatabaseInterface::lockSessionTables() { - QSqlQuery("lock tables " + server->getDbPrefix() + "_sessions write, " + server->getDbPrefix() + "_users read", sqlDatabase).exec(); + QSqlQuery("lock tables " + server->getDbPrefix() + "_sessions write, " + server->getDbPrefix() + "_users read", sqlDatabase); } void Servatrice_DatabaseInterface::unlockSessionTables() { - QSqlQuery("unlock tables", sqlDatabase).exec(); + QSqlQuery("unlock tables", sqlDatabase); } bool Servatrice_DatabaseInterface::userSessionExists(const QString &userName)