fuse: remove the unneeded result variable

ye xingchen · Miklos Szeredi · commit e2283a736676 · 2022-11-23T09:10:49.000+01:00
Return the value fuse_dev_release() directly instead of storing it in
another redundant variable.

Reported-by: Zeal Robot &lt;zealci@zte.com.cn&gt;
Signed-off-by: ye xingchen &lt;ye.xingchen@zte.com.cn&gt;
Signed-off-by: Miklos Szeredi &lt;mszeredi@redhat.com&gt;
diff --git a/fs/fuse/cuse.c b/fs/fuse/cuse.c
@@ -545,7 +545,6 @@ static int cuse_channel_release(struct inode *inode, struct file *file)
 {
 	struct fuse_dev *fud = file->private_data;
 	struct cuse_conn *cc = fc_to_cc(fud->fc);
-	int rc;
 
 	/* remove from the conntbl, no more access from this point on */
 	mutex_lock(&cuse_lock);
@@ -560,9 +559,7 @@ static int cuse_channel_release(struct inode *inode, struct file *file)
 		cdev_del(cc->cdev);
 	}
 
-	rc = fuse_dev_release(inode, file);	/* puts the base reference */
-
-	return rc;
+	return fuse_dev_release(inode, file);
 }
 
 static struct file_operations cuse_channel_fops; /* initialized during init */

Original file line number	Diff line number	Diff line change
`@@ -545,7 +545,6 @@ static int cuse_channel_release(struct inode inode, struct file file)`
`545`	`545`	`{`
`546`	`546`	`struct fuse_dev *fud = file->private_data;`
`547`	`547`	`struct cuse_conn *cc = fc_to_cc(fud->fc);`
`548`		`- int rc;`
`549`	`548`
`550`	`549`	`/* remove from the conntbl, no more access from this point on */`
`551`	`550`	`mutex_lock(&cuse_lock);`
`@@ -560,9 +559,7 @@ static int cuse_channel_release(struct inode inode, struct file file)`
`560`	`559`	`cdev_del(cc->cdev);`
`561`	`560`	`}`
`562`	`561`
`563`		`- rc = fuse_dev_release(inode, file); /* puts the base reference */`
`564`		`-`
`565`		`- return rc;`
	`562`	`+ return fuse_dev_release(inode, file);`
`566`	`563`	`}`
`567`	`564`
`568`	`565`	`static struct file_operations cuse_channel_fops; /* initialized during init */`