[Prevent] Handle remove() return value
authorJan Olszak <j.olszak@samsung.com>
Mon, 10 Dec 2012 09:50:33 +0000 (10:50 +0100)
committerGerrit Code Review <gerrit2@kim11>
Tue, 11 Dec 2012 07:51:09 +0000 (16:51 +0900)
[Issue#] Return value not handled.
[Bug] no. 22805
[Cause] N/A
[Solution] Log error on remove fail.
[Verification] Build commons.

Change-Id: I1c20be3caaef13d43599b3ce5522d6cbde30c330

modules/test/src/test_results_collector.cpp

index 7daf3c6..d481aa5 100644 (file)
@@ -27,6 +27,7 @@
 #include <dpl/scoped_fclose.h>
 
 #include <string>
+#include <string.h>
 #include <cstdio>
 #include <fstream>
 #include <sstream>
@@ -438,8 +439,10 @@ class XmlCollector
             m_outputBuffer.erase(posBegin - 3, posEnd - posBegin + sizeof("</testcase>") + 2);
         }
 
+        if(remove(m_filename.c_str())!=0){
+            LogError("Can't remove file. Error: " << strerror(errno));
+        }
 
-        remove(m_filename.c_str());
         m_fp.Reset(fopen (m_filename.c_str(), "w"));
         Assert(!!m_fp && "File handle must not be null");
         fseek(m_fp.Get(), 0L, SEEK_SET);