fuse: remove the unneeded result variable
Return the value fuse_dev_release() directly instead of storing it in another redundant variable. Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: ye xingchen <ye.xingchen@zte.com.cn> Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
This commit is contained in:
parent
00d369bc2d
commit
e2283a7366
@ -545,7 +545,6 @@ static int cuse_channel_release(struct inode *inode, struct file *file)
|
|||||||
{
|
{
|
||||||
struct fuse_dev *fud = file->private_data;
|
struct fuse_dev *fud = file->private_data;
|
||||||
struct cuse_conn *cc = fc_to_cc(fud->fc);
|
struct cuse_conn *cc = fc_to_cc(fud->fc);
|
||||||
int rc;
|
|
||||||
|
|
||||||
/* remove from the conntbl, no more access from this point on */
|
/* remove from the conntbl, no more access from this point on */
|
||||||
mutex_lock(&cuse_lock);
|
mutex_lock(&cuse_lock);
|
||||||
@ -560,9 +559,7 @@ static int cuse_channel_release(struct inode *inode, struct file *file)
|
|||||||
cdev_del(cc->cdev);
|
cdev_del(cc->cdev);
|
||||||
}
|
}
|
||||||
|
|
||||||
rc = fuse_dev_release(inode, file); /* puts the base reference */
|
return fuse_dev_release(inode, file);
|
||||||
|
|
||||||
return rc;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static struct file_operations cuse_channel_fops; /* initialized during init */
|
static struct file_operations cuse_channel_fops; /* initialized during init */
|
||||||
|
Loading…
Reference in New Issue
Block a user