Fix StepEncryptResources 24/226024/1
authorJunghyun Yeon <jungh.yeon@samsung.com>
Wed, 26 Feb 2020 06:36:01 +0000 (15:36 +0900)
committerJunghyun Yeon <jungh.yeon@samsung.com>
Wed, 26 Feb 2020 07:36:17 +0000 (07:36 +0000)
Revert if statement expression.

Change-Id: Ie147d0952ce8f3dec67fe00760baf9194e22c216
Signed-off-by: Junghyun Yeon <jungh.yeon@samsung.com>
(cherry picked from commit dc3d02324f7d161c92d86f459544c72561c71ade)

src/wgt/step/encryption/step_encrypt_resources.cc

index dffaa89..c5bbb9b 100644 (file)
@@ -165,7 +165,7 @@ bool StepEncryptResources::EncryptFile(const bf::path &src) {
     return false;
   }
   const std::size_t fileSize = info.st_size;
-  if (!fileSize) {
+  if (0 == fileSize) {
     LOG(ERROR) << src << " size is 0, so encryption will be skipped";
     return true;
   }
@@ -192,7 +192,7 @@ bool StepEncryptResources::EncryptFile(const bf::path &src) {
 
   do {
     bytesRead = ReadBytes(inChunk.get(), chunkSize, input);
-    if (!bytesRead) {
+    if (0 != bytesRead) {
       unsigned char* encrypted_data = nullptr;
       size_t encrypted_size = 0;
       // TODO(p.sikorski) check if it is Preloaded
@@ -266,11 +266,11 @@ bool StepEncryptResources::EncryptFile(const bf::path &src) {
   fclose(input);
 
   LOG(DEBUG) << "File encrypted successfully";
-  if (!unlink(src.c_str()))
+  if (0 != unlink(src.c_str()))
     return false;
 
   LOG(DEBUG) << "Rename encrypted file";
-  if (!std::rename(encFile.c_str(), src.c_str()))
+  if (0 != std::rename(encFile.c_str(), src.c_str()))
     return false;
 
   return true;