exfat: replace memcpy with structure assignment
Use structure assignment instead of memcpy. Signed-off-by: Tetsuhiro Kohada <kohada.t2@gmail.com> Acked-by: Sungjong Seo <sj1557.seo@samsung.com> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
This commit is contained in:
parent
d1441494cd
commit
90797eb877
7
dir.c
7
dir.c
@ -989,11 +989,8 @@ rewind:
|
||||
if (ei->hint_femp.eidx ==
|
||||
EXFAT_HINT_NONE ||
|
||||
candi_empty.eidx <=
|
||||
ei->hint_femp.eidx) {
|
||||
memcpy(&ei->hint_femp,
|
||||
&candi_empty,
|
||||
sizeof(candi_empty));
|
||||
}
|
||||
ei->hint_femp.eidx)
|
||||
ei->hint_femp = candi_empty;
|
||||
}
|
||||
|
||||
brelse(bh);
|
||||
|
2
inode.c
2
inode.c
@ -584,7 +584,7 @@ static int exfat_fill_inode(struct inode *inode, struct exfat_dir_entry *info)
|
||||
struct exfat_inode_info *ei = EXFAT_I(inode);
|
||||
loff_t size = info->size;
|
||||
|
||||
memcpy(&ei->dir, &info->dir, sizeof(struct exfat_chain));
|
||||
ei->dir = info->dir;
|
||||
ei->entry = info->entry;
|
||||
ei->attr = info->attr;
|
||||
ei->start_clu = info->start_clu;
|
||||
|
15
namei.c
15
namei.c
@ -345,8 +345,7 @@ static int exfat_find_empty_entry(struct inode *inode,
|
||||
hint_femp.eidx = EXFAT_HINT_NONE;
|
||||
|
||||
if (ei->hint_femp.eidx != EXFAT_HINT_NONE) {
|
||||
memcpy(&hint_femp, &ei->hint_femp,
|
||||
sizeof(struct exfat_hint_femp));
|
||||
hint_femp = ei->hint_femp;
|
||||
ei->hint_femp.eidx = EXFAT_HINT_NONE;
|
||||
}
|
||||
|
||||
@ -546,7 +545,7 @@ static int exfat_add_entry(struct inode *inode, const char *path,
|
||||
if (ret)
|
||||
goto out;
|
||||
|
||||
memcpy(&info->dir, p_dir, sizeof(struct exfat_chain));
|
||||
info->dir = *p_dir;
|
||||
info->entry = dentry;
|
||||
info->flags = ALLOC_NO_FAT_CHAIN;
|
||||
info->type = type;
|
||||
@ -678,7 +677,7 @@ static int exfat_find(struct inode *dir, struct qstr *qname,
|
||||
if (dentry < 0)
|
||||
return dentry; /* -error value */
|
||||
|
||||
memcpy(&info->dir, &cdir.dir, sizeof(struct exfat_chain));
|
||||
info->dir = cdir;
|
||||
info->entry = dentry;
|
||||
info->num_subdirs = 0;
|
||||
|
||||
@ -1137,7 +1136,7 @@ static int exfat_rename_file(struct inode *inode, struct exfat_chain *p_dir,
|
||||
if (!epnew)
|
||||
return -EIO;
|
||||
|
||||
memcpy(epnew, epold, DENTRY_SIZE);
|
||||
*epnew = *epold;
|
||||
if (exfat_get_entry_type(epnew) == TYPE_FILE) {
|
||||
epnew->dentry.file.attr |= cpu_to_le16(ATTR_ARCHIVE);
|
||||
ei->attr |= ATTR_ARCHIVE;
|
||||
@ -1157,7 +1156,7 @@ static int exfat_rename_file(struct inode *inode, struct exfat_chain *p_dir,
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
memcpy(epnew, epold, DENTRY_SIZE);
|
||||
*epnew = *epold;
|
||||
exfat_update_bh(new_bh, sync);
|
||||
brelse(old_bh);
|
||||
brelse(new_bh);
|
||||
@ -1225,7 +1224,7 @@ static int exfat_move_file(struct inode *inode, struct exfat_chain *p_olddir,
|
||||
if (!epnew)
|
||||
return -EIO;
|
||||
|
||||
memcpy(epnew, epmov, DENTRY_SIZE);
|
||||
*epnew = *epmov;
|
||||
if (exfat_get_entry_type(epnew) == TYPE_FILE) {
|
||||
epnew->dentry.file.attr |= cpu_to_le16(ATTR_ARCHIVE);
|
||||
ei->attr |= ATTR_ARCHIVE;
|
||||
@ -1245,7 +1244,7 @@ static int exfat_move_file(struct inode *inode, struct exfat_chain *p_olddir,
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
memcpy(epnew, epmov, DENTRY_SIZE);
|
||||
*epnew = *epmov;
|
||||
exfat_update_bh(new_bh, IS_DIRSYNC(inode));
|
||||
brelse(mov_bh);
|
||||
brelse(new_bh);
|
||||
|
Loading…
Reference in New Issue
Block a user