small fix on top of DeleteDebugDump change

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

3 files changed, 4 insertions, 4 deletions

diff --git a/src/CLI/dbus.cpp b/src/CLI/dbus.cpp
index 67c489e..2da9d74 100644
--- a/src/CLI/dbus.cpp
+++ b/src/CLI/dbus.cpp
@@ -160,8 +160,6 @@ void call_DeleteDebugDump(const char* uuid)
             DBUS_TYPE_INVALID);
 
     DBusMessage *reply = send_get_reply_and_unref(msg);
-    //it returns a single boolean value,
-    //but we don't use it (yet?)
 
     dbus_message_unref(reply);
 }
diff --git a/src/Daemon/CrashWatcher.cpp b/src/Daemon/CrashWatcher.cpp
index 1606f86..2973cfc 100644
--- a/src/Daemon/CrashWatcher.cpp
+++ b/src/Daemon/CrashWatcher.cpp
@@ -201,7 +201,9 @@ void DeleteDebugDump(const char *pUUID, const char *pUID)
         database->DeleteRow(pUUID, pUID);
         database->DisConnect();
 
-        delete_debug_dump_dir(row.m_sDebugDumpDir.c_str());
+        const char *dump_dir = row.m_sDebugDumpDir.c_str();
+        if (dump_dir[0] != '\0')
+            delete_debug_dump_dir(dump_dir);
     }
     catch (CABRTException& e)
     {
diff --git a/src/Daemon/Daemon.cpp b/src/Daemon/Daemon.cpp
index f9910dc..1fa324f 100644
--- a/src/Daemon/Daemon.cpp
+++ b/src/Daemon/Daemon.cpp
@@ -64,7 +64,7 @@
  *      "Please report this crash": calls Report() of all registered reporter plugins.
  *      Returns report_status_t (map_vector_string_t) - the status of each call.
  *      2nd parameter is the contents of user's abrt.conf.
- * - DeleteDebugDump(UUID): delete corresponding /var/cache/abrt/DIR. Returns bool
+ * - DeleteDebugDump(UUID): delete it from DB and delete corresponding /var/cache/abrt/DIR
  * - GetPluginsInfo(): returns vector_map_string_t
  * - GetPluginSettings(PluginName): returns map_plugin_settings_t (map_string_t)
  * - SetPluginSettings(PluginName, map_plugin_settings_t): returns void