diff --git a/adapter/uhdf2/manager/src/devmgr_service_stub.c b/adapter/uhdf2/manager/src/devmgr_service_stub.c
index ab4d6d6b2c72204a65a0cfccb9c6b001718afa31..ad7b82989bf7f0f7a2e35fe7e2ea79518afef818 100644
--- a/adapter/uhdf2/manager/src/devmgr_service_stub.c
+++ b/adapter/uhdf2/manager/src/devmgr_service_stub.c
@@ -152,7 +152,7 @@ int32_t DevmgrServiceStubDispatch(struct HdfRemoteService *stub, int code, struc
         HDF_LOGE("%{public}s devmgr service stub dispach failed, cmd id is %{public}d, ret = %{public}d", __func__,
             code, ret);
     }
-    return ret;
+    return HDF_SUCCESS;
 }
 
 static void RemoveModule(const char *module)
diff --git a/framework/tools/hdi-gen/ast/ast_enum_type.h b/framework/tools/hdi-gen/ast/ast_enum_type.h
index 0a37586bd06444a17b8091679432b6eb31ead104..7bfbff86614e2e4aee798629ceee166f7e42cb46 100644
--- a/framework/tools/hdi-gen/ast/ast_enum_type.h
+++ b/framework/tools/hdi-gen/ast/ast_enum_type.h
@@ -110,7 +110,7 @@ public:
         return members_[index];
     }
 
-    inline bool HasMember(std::string memberName)
+    inline bool HasMember(const std::string &memberName)
     {
         for (size_t i = 0; i < members_.size(); i++) {
             if (members_[i]->GetName() == memberName) {
diff --git a/framework/tools/hdi-gen/ast/ast_expr.cpp b/framework/tools/hdi-gen/ast/ast_expr.cpp
index e5f6d806f1becebf17a70b68a91cc3992899d8ab..075e0ad05959800c9e1d14b66236ddc6da8fc048 100644
--- a/framework/tools/hdi-gen/ast/ast_expr.cpp
+++ b/framework/tools/hdi-gen/ast/ast_expr.cpp
@@ -98,7 +98,7 @@ std::string ASTNumExpr::Dump(const std::string &prefix)
     sb.AppendFormat("%s", value_.c_str());
 
     if (isParenExpr) {
-        sb.Append("(");
+        sb.Append(")");
     }
 
     return sb.ToString();
@@ -115,7 +115,7 @@ std::string ASTEnumExpr::Dump(const std::string &prefix)
     sb.AppendFormat("%s", value_.c_str());
 
     if (isParenExpr) {
-        sb.Append("(");
+        sb.Append(")");
     }
 
     return sb.ToString();
diff --git a/framework/tools/hdi-gen/codegen/java_client_proxy_code_emitter.cpp b/framework/tools/hdi-gen/codegen/java_client_proxy_code_emitter.cpp
index 0108ec0f24ca33c4f8ed3df290f60c1bd9af5ff0..b05cb2cec6b5910da7f33e5b6cd966e9adbd1213 100644
--- a/framework/tools/hdi-gen/codegen/java_client_proxy_code_emitter.cpp
+++ b/framework/tools/hdi-gen/codegen/java_client_proxy_code_emitter.cpp
@@ -114,7 +114,7 @@ void JavaClientProxyCodeEmitter::EmitProxyImpl(StringBuilder &sb)
     EmitProxyConstants(sb, TAB);
     sb.Append("\n");
     sb.Append(TAB).AppendFormat(
-        "private static final HiLogLabel TAG = new HiLogLabel(HiLog.LOG_CORE, 0xD001510, \"%s\");\n",
+        "private static final HiLogLabel TAG = new HiLogLabel(HiLog.LOG_CORE, 0xD002510, \"%s\");\n",
         interfaceFullName_.c_str());
     sb.Append(TAB).Append("private final IRemoteObject remote;\n");
     sb.Append(TAB).Append("private static final int ERR_OK = 0;\n");