Add checking return value

[platform/core/security/ode.git] / server / file-footer.cpp

diff --git a/server/file-footer.cpp b/server/file-footer.cpp
index 008c55c..7531db4 100644 (file)
--- a/server/file-footer.cpp
+++ b/server/file-footer.cpp
@@ -31,7 +31,7 @@ namespace {
 
 const std::string getFileName(const std::string &key)
 {
-       KeyGenerator::data hash = KeyGenerator::SHA256(KeyGenerator::data(key.begin(), key.end()));
+       BinaryData hash = KeyGenerator::SHA256(BinaryData(key.begin(), key.end()));
        std::stringstream fileName;
 
        fileName << "/opt/etc/.ode_";
@@ -56,14 +56,14 @@ bool FileFooter::exist(const std::string &key)
        return file.exists();
 }
 
-const FileFooter::data FileFooter::read(const std::string &key)
+const BinaryData FileFooter::read(const std::string &key)
 {
        std::string fileName(getFileName(key));
 
        INFO(SINK, "Footer file : " + fileName);
 
        runtime::File file(fileName);
-       data value(file.size());
+       BinaryData value(file.size());
 
        file.open(O_RDONLY);
        file.read(value.data(), value.size());
@@ -71,7 +71,7 @@ const FileFooter::data FileFooter::read(const std::string &key)
        return value;
 }
 
-void FileFooter::write(const std::string &key, const data &value)
+void FileFooter::write(const std::string &key, const BinaryData &value)
 {
        std::string fileName(getFileName(key));
 
@@ -79,7 +79,12 @@ void FileFooter::write(const std::string &key, const data &value)
 
        runtime::File file(fileName);
 
-       file.create(S_IRUSR | S_IWUSR);
+       if (!file.exists()) {
+               file.create(S_IRUSR | S_IWUSR);
+       } else {
+               file.open(O_RDWR);
+       }
+
        file.write(value.data(), value.size());
 }