diff --git a/fs/hmdfs/hmdfs_server.c b/fs/hmdfs/hmdfs_server.c
index dd4508b80e8eda2d1d70ce317fb43277800a16c9..ea3697f331282180adb1012743e06ced3cb841e7 100644
--- a/fs/hmdfs/hmdfs_server.c
+++ b/fs/hmdfs/hmdfs_server.c
@@ -252,7 +252,7 @@ static int parse_data_sec_level(const char *sl_value, size_t sl_value_len)
 {
 	int i;
 
-	for (i = 0; i <= sizeof(datasl_str) / sizeof(datasl_str[0]); i++) {
+	for (i = 0; i < sizeof(datasl_str) / sizeof(datasl_str[0]); i++) {
 		if (!strncmp(sl_value, datasl_str[i], strlen(datasl_str[i])))
 			return i + DATA_SEC_LEVEL0;
 	}
@@ -266,7 +266,7 @@ static int check_sec_level(struct hmdfs_peer *node, const char *file_name)
 	int ret = 0;
 	struct path root_path;
 	struct path file_path;
-	char *value;
+	char *value = NULL;
 	size_t value_len = DATA_SEC_LEVEL_LENGTH;
 
 	if (node->devsl <= 0) {
diff --git a/fs/hmdfs/inode_local.c b/fs/hmdfs/inode_local.c
index 84904f50d27df708bfe8ad6dbf50bd9477cf1f5a..561b45dbb4654ea3139b7eea476956341c148304 100644
--- a/fs/hmdfs/inode_local.c
+++ b/fs/hmdfs/inode_local.c
@@ -901,7 +901,6 @@ int hmdfs_permission(struct inode *inode, int mask)
 {
 #ifdef CONFIG_HMDFS_FS_PERMISSION
 	unsigned int mode = inode->i_mode;
-	struct hmdfs_inode_info *hii = hmdfs_i(inode);
 	kuid_t cur_uid = current_fsuid();
 
 	if (uid_eq(cur_uid, ROOT_UID) || uid_eq(cur_uid, SYSTEM_UID))