Testing err before returning value is not needed. Signed-off-by: Fabian Frederick <fabf@xxxxxxxxx> --- fs/ceph/ioctl.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/fs/ceph/ioctl.c b/fs/ceph/ioctl.c index a822a6e..b2d777a 100644 --- a/fs/ceph/ioctl.c +++ b/fs/ceph/ioctl.c @@ -54,10 +54,7 @@ static long __validate_layout(struct ceph_mds_client *mdsc, break; } mutex_unlock(&mdsc->mutex); - if (err) - return err; - - return 0; + return err; } static long ceph_ioctl_set_layout(struct file *file, void __user *arg) -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
