Merge branch 'master' of git://git.fedorahosted.org/git/abrt

3 files changed, 13 insertions, 15 deletions

diff --git a/lib/Plugins/SQLite3.cpp b/lib/Plugins/SQLite3.cpp
index 788d43b..866b985 100644
--- a/lib/Plugins/SQLite3.cpp
+++ b/lib/Plugins/SQLite3.cpp
@@ -100,14 +100,16 @@ static void get_table(vector_database_rows_t& pTable,
 
     char **table;
     int ncol, nrow;
-    char *err;
+    char *err = NULL;
     int ret = sqlite3_get_table(db, sql, &table, &nrow, &ncol, &err);
     if (ret != SQLITE_OK)
     {
         string errstr = ssprintf("Error in SQL:'%s' error: %s", sql, err);
         free(sql);
+        sqlite3_free(err);
         throw CABRTException(EXCEP_PLUGIN, errstr);
     }
+    VERB2 log("%d rows returned by SQL:%s", nrow, sql);
     free(sql);
 
     pTable.clear();
@@ -143,14 +145,16 @@ static void execute_sql(sqlite3 *db, const char *fmt, ...)
     char *sql = xvasprintf(fmt, p);
     va_end(p);
 
-    char *err;
-    int ret = sqlite3_exec(db, sql, 0, 0, &err);
+    char *err = NULL;
+    int ret = sqlite3_exec(db, sql, /*callback:*/ NULL, /*callback param:*/ NULL, &err);
     if (ret != SQLITE_OK)
     {
         string errstr = ssprintf("Error in SQL:'%s' error: %s", sql, err);
         free(sql);
+        sqlite3_free(err);
         throw CABRTException(EXCEP_PLUGIN, errstr);
     }
+    VERB2 log("%d rows affected by SQL:%s", sqlite3_changes(db), sql);
     free(sql);
 }
 
@@ -184,7 +188,7 @@ static bool check_table(sqlite3 *db)
         /* Should never happen */
         error_msg_and_die("SQLite3 database is corrupted");
     }
-    if (!nrow || !nrow)
+    if (!nrow)
     {
         return false;
     }
@@ -201,7 +205,7 @@ static bool check_table(sqlite3 *db)
         return true;
     }
 
-    // TODO: after some time could be removed, and if a table is that old,
+    // TODO: after some time could be removed, and if the table is that old,
     // then simply drop it and create new one
 
     // hack for version 0 and 1
diff --git a/src/Daemon/CommLayerServerDBus.cpp b/src/Daemon/CommLayerServerDBus.cpp
index 8f3f504..27a18c2 100644
--- a/src/Daemon/CommLayerServerDBus.cpp
+++ b/src/Daemon/CommLayerServerDBus.cpp
@@ -315,11 +315,7 @@ static int handle_DeleteDebugDump(DBusMessage* call, DBusMessage* reply)
     }
 
     long unix_uid = get_remote_uid(call);
-    bool argout1 = DeleteDebugDump(argin1, to_string(unix_uid).c_str());
-
-    dbus_message_append_args(reply,
-                DBUS_TYPE_BOOLEAN, &argout1,
-                DBUS_TYPE_INVALID);
+    DeleteDebugDump(argin1, to_string(unix_uid).c_str());
 
     send_flush_and_unref(reply);
     return 0;
diff --git a/src/Gui/CCMainWindow.py b/src/Gui/CCMainWindow.py
index 88d931b..8e48050 100644
--- a/src/Gui/CCMainWindow.py
+++ b/src/Gui/CCMainWindow.py
@@ -257,11 +257,9 @@ class MainWindow():
         # this should work until we keep the row object in the last position
         dump = dumpsListStore.get_value(dumpsListStore.get_iter(path[0]), dumpsListStore.get_n_columns()-1)
         try:
-            if self.ccdaemon.DeleteDebugDump(dump.getUUID()):
-                self.hydrate()
-                treeview.emit("cursor-changed")
-            else:
-                print "Couldn't delete"
+            self.ccdaemon.DeleteDebugDump(dump.getUUID())
+            self.hydrate()
+            treeview.emit("cursor-changed")
         except Exception, e:
             print e