diff --git a/mpp/codec/dec/dummy/dummy_dec_api.c b/mpp/codec/dec/dummy/dummy_dec_api.c
index 710a2b7c..69deb499 100644
--- a/mpp/codec/dec/dummy/dummy_dec_api.c
+++ b/mpp/codec/dec/dummy/dummy_dec_api.c
@@ -162,7 +162,7 @@ MPP_RET dummy_dec_prepare(void *dec, MppPacket pkt, HalDecTask *task)
     data    = mpp_packet_get_data(pkt);
     length  = mpp_packet_get_length(pkt);
     if (length > p->stream_size) {
-        mpp_realloc(p->stream, RK_U8, length);
+        p->stream = mpp_realloc(p->stream, RK_U8, length);
         mpp_packet_set_data(p->task_pkt, p->stream);
         p->stream_size = length;
     }
diff --git a/mpp/hal/worker/mpp_device/mpp_device.c b/mpp/hal/worker/mpp_device/mpp_device.c
index 22faf5f7..e43073b2 100644
--- a/mpp/hal/worker/mpp_device/mpp_device.c
+++ b/mpp/hal/worker/mpp_device/mpp_device.c
@@ -109,8 +109,7 @@ static RK_S32 mpp_device_get_client_type(MppDevCtx ctx, MppCtxType type, MppCodi
     RK_S32 client_type = -1;
     MppDevCtxImpl *p;
 
-    if (NULL == ctx ||
-        (type >= MPP_CTX_BUTT || type < 0) ||
+    if (NULL == ctx || type >= MPP_CTX_BUTT ||
         (coding >= MPP_VIDEO_CodingMax || coding <= MPP_VIDEO_CodingUnused)) {
         mpp_err_f("found NULL input ctx %p coding %d type %d\n", ctx, coding, type);
         return MPP_ERR_NULL_PTR;
diff --git a/mpp/mpp_impl.cpp b/mpp/mpp_impl.cpp
index d51bdd8e..dd20ebb4 100644
--- a/mpp/mpp_impl.cpp
+++ b/mpp/mpp_impl.cpp
@@ -17,6 +17,8 @@
 
 #define MODULE_TAG "mpp_impl"
 
+#include <string.h>
+
 #include "mpp_mem.h"
 #include "mpp_env.h"
 #include "mpp_common.h"
diff --git a/osal/mpp_time.cpp b/osal/mpp_time.cpp
index 57359e9a..4b344afa 100644
--- a/osal/mpp_time.cpp
+++ b/osal/mpp_time.cpp
@@ -78,7 +78,7 @@ MppTimer mpp_timer_get(const char *name)
     MppTimerImpl *impl = mpp_calloc(MppTimerImpl, 1);
     if (impl) {
         impl->check = module_name;
-        snprintf(impl->name, sizeof(impl->name), name);
+        snprintf(impl->name, sizeof(impl->name), name, NULL);
     } else
         mpp_err_f("malloc failed\n");