diff --git a/base/src/event_reactor.cpp b/base/src/event_reactor.cpp
index e54d183e2d7bbd737e8af63751d0610114010d8c..ea5383776e8e76bf9288a72438e82d373bc49f4b 100644
--- a/base/src/event_reactor.cpp
+++ b/base/src/event_reactor.cpp
@@ -95,9 +95,6 @@ uint32_t EventReactor::ScheduleTimer(const TimerCallback& cb, uint32_t interval,
 {
     std::lock_guard<std::recursive_mutex> lock(mutex_);
     std::shared_ptr<TimerEventHandler> handler = std::make_shared<TimerEventHandler>(this, interval, once);
-    if (handler == nullptr) {
-        return TIMER_ERR_INVALID_VALUE;
-    }
     handler->SetTimerCallback(cb);
     uint32_t ret = handler->Initialize();
     if (ret != TIMER_ERR_OK) {
diff --git a/base/src/parcel.cpp b/base/src/parcel.cpp
index ae218eeff6cf76527033095bc70632c9582b5b2f..441a4df2f304048f8265591a94393eefcd345c67 100644
--- a/base/src/parcel.cpp
+++ b/base/src/parcel.cpp
@@ -964,8 +964,8 @@ const std::string Parcel::ReadString()
         return std::string();
     }
 
-    size_t readCapacity = dataLength + 1;
-    if ((readCapacity > (size_t)dataLength) && (readCapacity <= GetReadableBytes())) {
+    size_t readCapacity = static_cast<size_t>(dataLength) + 1;
+    if (readCapacity <= GetReadableBytes()) {
         const uint8_t *dest = ReadBuffer(readCapacity);
         if (dest != nullptr) {
             const auto *str = reinterpret_cast<const char *>(dest);
@@ -990,8 +990,8 @@ bool Parcel::ReadString(std::string &value)
         return false;
     }
 
-    size_t readCapacity = dataLength + 1;
-    if ((readCapacity > (size_t)dataLength) && (readCapacity <= GetReadableBytes())) {
+    size_t readCapacity = static_cast<size_t>(dataLength) + 1;
+    if (readCapacity <= GetReadableBytes()) {
         const uint8_t *dest = ReadBuffer(readCapacity);
         if (dest != nullptr) {
             const auto *str = reinterpret_cast<const char *>(dest);
@@ -1017,8 +1017,8 @@ const std::u16string Parcel::ReadString16()
         return std::u16string();
     }
 
-    size_t readCapacity = (dataLength + 1) * sizeof(char16_t);
-    if ((readCapacity > (size_t)dataLength) && (readCapacity <= GetReadableBytes())) {
+    size_t readCapacity = (static_cast<size_t>(dataLength) + 1) * sizeof(char16_t);
+    if ((readCapacity > (static_cast<size_t>(dataLength))) && (readCapacity <= GetReadableBytes())) {
         const uint8_t *str = ReadBuffer(readCapacity);
         if (str != nullptr) {
             const auto *u16Str = reinterpret_cast<const char16_t *>(str);
@@ -1043,7 +1043,7 @@ bool Parcel::ReadString16(std::u16string &value)
         return false;
     }
 
-    size_t readCapacity = (dataLength + 1) * sizeof(char16_t);
+    size_t readCapacity = (static_cast<size_t>(dataLength) + 1) * sizeof(char16_t);
     if (readCapacity <= GetReadableBytes()) {
         const uint8_t *str = ReadBuffer(readCapacity);
         if (str != nullptr) {
@@ -1075,8 +1075,8 @@ const std::u16string Parcel::ReadString16WithLength(int32_t &readLength)
         return std::u16string();
     }
 
-    size_t readCapacity = (dataLength + 1) * sizeof(char16_t);
-    if ((readCapacity > (size_t)dataLength) && (readCapacity <= GetReadableBytes())) {
+    size_t readCapacity = (static_cast<size_t>(dataLength) + 1) * sizeof(char16_t);
+    if ((readCapacity > (static_cast<size_t>(dataLength))) && (readCapacity <= GetReadableBytes())) {
         const uint8_t *str = ReadBuffer(readCapacity);
         if (str != nullptr) {
             const auto *u16Str = reinterpret_cast<const char16_t *>(str);
@@ -1106,8 +1106,8 @@ const std::string Parcel::ReadString8WithLength(int32_t &readLength)
         return std::string();
     }
 
-    size_t readCapacity = (dataLength + 1) * sizeof(char);
-    if ((readCapacity > (size_t)dataLength) && (readCapacity <= GetReadableBytes())) {
+    size_t readCapacity = (static_cast<size_t>(dataLength) + 1) * sizeof(char);
+    if (readCapacity <= GetReadableBytes()) {
         const uint8_t *str = ReadBuffer(readCapacity);
         if (str != nullptr) {
             const auto *u8Str = reinterpret_cast<const char *>(str);