commit a3f6aa7f71c65df847d955940165550fb34f080c
author Jayanth Othayoth <ojayanth@in.ibm.com> Tue Jul 04 02:15:04 2017 -0500
committer Jayanth Othayoth <ojayanth@in.ibm.com> Tue Jul 18 07:55:40 2017 -0500
tree 7c0218bf4c3c1c05179565e7696186c0c15fb67d
parent d02153c9906ada5c78fb32a53a3b0d7c62821be4

Handle filesystem exceptions during dump file removal.

Change-Id: If77e549ead60e6b6e21cd9069fbc0a324db6bc80
Signed-off-by: Jayanth Othayoth <ojayanth@in.ibm.com>

dump_entry.cpp

diff --git a/dump_entry.cpp b/dump_entry.cpp
index 373d8e4..7b9965c 100644
--- a/dump_entry.cpp
+++ b/dump_entry.cpp
@@ -1,8 +1,8 @@
+#include <phosphor-logging/log.hpp>
+
 #include "dump_entry.hpp"
 #include "dump_manager.hpp"
 
-#include <phosphor-logging/log.hpp>
-
 namespace phosphor
 {
 namespace dump
@@ -13,11 +13,16 @@
 void Entry::delete_()
 {
     //Delete Dump file from Permanent location
-    if (!fs::remove(file))
+    try
     {
-        log<level::INFO>("Dump file doesn't exist.",
-                         entry("Name=%s", file.c_str()));
+        fs::remove(file);
     }
+    catch (fs::filesystem_error& e)
+    {
+        //Log Error message and continue
+        log<level::ERR>(e.what());
+    }
+
     // Remove Dump entry D-bus object
     parent.erase(id);
 }