diff --git a/storage-manager/src/OpenTask.cpp b/storage-manager/src/OpenTask.cpp
index e94a8483d..19b3ae826 100644
--- a/storage-manager/src/OpenTask.cpp
+++ b/storage-manager/src/OpenTask.cpp
@@ -45,7 +45,7 @@ bool OpenTask::run()
         return the result
     */
     SMLogging* logger = SMLogging::get();
-    bool success;
+    int success;
     uint8_t buf[1024] = {0};
     
     if (getLength() > 1023)
@@ -55,7 +55,7 @@ bool OpenTask::run()
     }
     
     success = read(buf, getLength());
-    if (!success)
+    if (success<0)
     {
         handleError("OpenTask read2", errno);
         return false;
diff --git a/storage-manager/src/PingTask.cpp b/storage-manager/src/PingTask.cpp
index 4adb748b7..73e460cea 100644
--- a/storage-manager/src/PingTask.cpp
+++ b/storage-manager/src/PingTask.cpp
@@ -43,8 +43,8 @@ bool PingTask::run()
         return true;
     }
     // consume the msg
-    bool success = read(&buf, getLength());
-    if (!success)
+    int success = read(&buf, getLength());
+    if (success<0)
     {
         handleError("PingTask", errno);
         return false;