staging: exfat: Clean up return codes - FFS_ERROR
Convert FFS_ERROR to -EINVAL Signed-off-by: Valdis Kletnieks <Valdis.Kletnieks@vt.edu> Link: https://lore.kernel.org/r/20191112021000.42091-6-Valdis.Kletnieks@vt.edu Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
7785913b07
commit
0a7ef8d948
|
@ -216,7 +216,6 @@ static inline u16 get_row_index(u16 i)
|
||||||
#define FFS_SEMAPHOREERR 6
|
#define FFS_SEMAPHOREERR 6
|
||||||
#define FFS_NOTOPENED 12
|
#define FFS_NOTOPENED 12
|
||||||
#define FFS_MAXOPENED 13
|
#define FFS_MAXOPENED 13
|
||||||
#define FFS_ERROR 19
|
|
||||||
|
|
||||||
#define NUM_UPCASE 2918
|
#define NUM_UPCASE 2918
|
||||||
|
|
||||||
|
|
|
@ -699,7 +699,7 @@ void sync_alloc_bitmap(struct super_block *sb)
|
||||||
static s32 __load_upcase_table(struct super_block *sb, sector_t sector,
|
static s32 __load_upcase_table(struct super_block *sb, sector_t sector,
|
||||||
u32 num_sectors, u32 utbl_checksum)
|
u32 num_sectors, u32 utbl_checksum)
|
||||||
{
|
{
|
||||||
int i, ret = FFS_ERROR;
|
int i, ret = -EINVAL;
|
||||||
u32 j;
|
u32 j;
|
||||||
struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
|
struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
|
||||||
struct bd_info_t *p_bd = &(EXFAT_SB(sb)->bd_info);
|
struct bd_info_t *p_bd = &(EXFAT_SB(sb)->bd_info);
|
||||||
|
@ -774,7 +774,7 @@ static s32 __load_upcase_table(struct super_block *sb, sector_t sector,
|
||||||
brelse(tmp_bh);
|
brelse(tmp_bh);
|
||||||
return FFS_SUCCESS;
|
return FFS_SUCCESS;
|
||||||
}
|
}
|
||||||
ret = FFS_ERROR;
|
ret = -EINVAL;
|
||||||
error:
|
error:
|
||||||
if (tmp_bh)
|
if (tmp_bh)
|
||||||
brelse(tmp_bh);
|
brelse(tmp_bh);
|
||||||
|
@ -784,7 +784,7 @@ error:
|
||||||
|
|
||||||
static s32 __load_default_upcase_table(struct super_block *sb)
|
static s32 __load_default_upcase_table(struct super_block *sb)
|
||||||
{
|
{
|
||||||
int i, ret = FFS_ERROR;
|
int i, ret = -EINVAL;
|
||||||
u32 j;
|
u32 j;
|
||||||
struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
|
struct fs_info_t *p_fs = &(EXFAT_SB(sb)->fs_info);
|
||||||
|
|
||||||
|
@ -1906,7 +1906,7 @@ static s32 __write_partial_entries_in_entry_set(struct super_block *sb,
|
||||||
return FFS_SUCCESS;
|
return FFS_SUCCESS;
|
||||||
err_out:
|
err_out:
|
||||||
pr_debug("%s failed\n", __func__);
|
pr_debug("%s failed\n", __func__);
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* write back all entries in entry set */
|
/* write back all entries in entry set */
|
||||||
|
@ -1931,7 +1931,7 @@ s32 write_partial_entries_in_entry_set(struct super_block *sb,
|
||||||
|
|
||||||
/* vaidity check */
|
/* vaidity check */
|
||||||
if (ep + count > ((struct dentry_t *)&es->__buf) + es->num_entries)
|
if (ep + count > ((struct dentry_t *)&es->__buf) + es->num_entries)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
dir.dir = GET_CLUSTER_FROM_SECTOR(es->sector);
|
dir.dir = GET_CLUSTER_FROM_SECTOR(es->sector);
|
||||||
dir.flags = es->alloc_flag;
|
dir.flags = es->alloc_flag;
|
||||||
|
|
|
@ -498,7 +498,7 @@ static int ffsGetVolInfo(struct super_block *sb, struct vol_info_t *info)
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!info)
|
if (!info)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -561,7 +561,7 @@ static int ffsLookupFile(struct inode *inode, char *path, struct file_id_t *fid)
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!fid || !path || (*path == '\0'))
|
if (!fid || !path || (*path == '\0'))
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -654,7 +654,7 @@ static int ffsCreateFile(struct inode *inode, char *path, u8 mode,
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!fid || !path || (*path == '\0'))
|
if (!fid || !path || (*path == '\0'))
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -703,7 +703,7 @@ static int ffsReadFile(struct inode *inode, struct file_id_t *fid, void *buffer,
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!buffer)
|
if (!buffer)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -835,7 +835,7 @@ static int ffsWriteFile(struct inode *inode, struct file_id_t *fid,
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!buffer)
|
if (!buffer)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -1241,7 +1241,7 @@ static int ffsMoveFile(struct inode *old_parent_inode, struct file_id_t *fid,
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!new_path || (*new_path == '\0'))
|
if (!new_path || (*new_path == '\0'))
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -1464,7 +1464,7 @@ static int ffsSetAttr(struct inode *inode, u32 attr)
|
||||||
if (p_fs->dev_ejected)
|
if (p_fs->dev_ejected)
|
||||||
ret = -EIO;
|
ret = -EIO;
|
||||||
else
|
else
|
||||||
ret = FFS_ERROR;
|
ret = -EINVAL;
|
||||||
|
|
||||||
if (p_fs->vol_type == EXFAT)
|
if (p_fs->vol_type == EXFAT)
|
||||||
release_entry_set(es);
|
release_entry_set(es);
|
||||||
|
@ -1756,7 +1756,7 @@ static int ffsMapCluster(struct inode *inode, s32 clu_offset, u32 *clu)
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!clu)
|
if (!clu)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -1908,7 +1908,7 @@ static int ffsCreateDir(struct inode *inode, char *path, struct file_id_t *fid)
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!fid || !path || (*path == '\0'))
|
if (!fid || !path || (*path == '\0'))
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* acquire the lock for file system critical section */
|
/* acquire the lock for file system critical section */
|
||||||
mutex_lock(&p_fs->v_mutex);
|
mutex_lock(&p_fs->v_mutex);
|
||||||
|
@ -1954,7 +1954,7 @@ static int ffsReadDir(struct inode *inode, struct dir_entry_t *dir_entry)
|
||||||
|
|
||||||
/* check the validity of pointer parameters */
|
/* check the validity of pointer parameters */
|
||||||
if (!dir_entry)
|
if (!dir_entry)
|
||||||
return FFS_ERROR;
|
return -EINVAL;
|
||||||
|
|
||||||
/* check if the given file ID is opened */
|
/* check if the given file ID is opened */
|
||||||
if (fid->type != TYPE_DIR)
|
if (fid->type != TYPE_DIR)
|
||||||
|
|
Loading…
Reference in New Issue