diff --git a/mpp/legacy/vpu_api.cpp b/mpp/legacy/vpu_api.cpp
index ba8b52be..960ef93c 100644
--- a/mpp/legacy/vpu_api.cpp
+++ b/mpp/legacy/vpu_api.cpp
@@ -230,15 +230,13 @@ public:
             rkvpu_close_cxt = (RK_S32 (*)(VpuCodecContext **ctx))
                               dlsym(rkapi_hdl, "vpu_close_context");
             mpp_log("dlopen vpu lib %s success\n", codec_paths[i]);
-        } else {
-            mpp_err("dlopen vpu lib failed\n");
         }
     }
 
     ~VpulibDlsym() {
         if (rkapi_hdl) {
             dlclose(rkapi_hdl);
-            mpp_log("dlclose vpu lib");
+            rkapi_hdl = NULL;
         }
     }
 };
diff --git a/mpp/vproc/rga/rga.cpp b/mpp/vproc/rga/rga.cpp
index f0bd134b..68516cc5 100644
--- a/mpp/vproc/rga/rga.cpp
+++ b/mpp/vproc/rga/rga.cpp
@@ -23,6 +23,7 @@
 
 #include "mpp_mem.h"
 #include "mpp_log.h"
+#include "mpp_common.h"
 
 #include "rga.h"
 #include "rga_api.h"
diff --git a/osal/mpp_runtime.cpp b/osal/mpp_runtime.cpp
index baedabb4..5362ea2e 100644
--- a/osal/mpp_runtime.cpp
+++ b/osal/mpp_runtime.cpp
@@ -125,10 +125,15 @@ MppRuntimeService::MppRuntimeService()
                         RK_S32 val = 0;
                         FILE *fp = fopen(path, "rb");
                         if (fp) {
-                            fread(&val, 1, 4, fp);
+                            size_t len = fread(&val, 1, 4, fp);
                             // zero for ion non-zero for drm ->
                             // zero     - disable drm
                             // non-zero - disable ion
+                            if (len != 4) {
+                                mpp_err("failed to read dts allocator value default 0\n");
+                                val = 0;
+                            }
+
                             if (val == 0) {
                                 allocator_valid[MPP_BUFFER_TYPE_DRM] = 0;
                                 mpp_log("found ion allocator in dts\n");