Don't delete the crashdump match on failure

If startCrashdumpAndRecovery() is called twice before crashdump
has completed, the second call will fail to start and delete the
static match from the first call.

This removes the reset of the match on failure, so the first
crashdump complete can be detected.

Tested:
Called startCrashdumpAndRecovery() twice and confirmed that the
match still triggers correctly.

Change-Id: Ib83c39fd5f409b4a0ba6110ee2f257089b89d6a3
Signed-off-by: Jason M. Bills <jason.m.bills@intel.com>
diff --git a/src/host_error_monitor.cpp b/src/host_error_monitor.cpp
index 5ef907d..873348f 100644
--- a/src/host_error_monitor.cpp
+++ b/src/host_error_monitor.cpp
@@ -462,7 +462,6 @@
             {
                 std::cerr << "failed to start Crashdump\n";
                 crashdumpTimer.cancel();
-                crashdumpCompleteMatch.reset();
             }
         },
         "com.intel.crashdump", "/com/intel/crashdump",