Colin King <[email protected]> writes: > From: Colin Ian King <[email protected]> > > The variable ret is being set to -EINVAL however this is never read > and later it is being reassigned to a new value. The assignment is > redundant and hence can be removed. > > Addresses-Coverity: ("Unused Value") > Signed-off-by: Colin Ian King <[email protected]> > --- > fs/aio.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/fs/aio.c b/fs/aio.c > index f9f441b59966..3e290dfac10a 100644 > --- a/fs/aio.c > +++ b/fs/aio.c > @@ -1528,7 +1528,6 @@ static int aio_read(struct kiocb *req, const struct > iocb *iocb, > file = req->ki_filp; > if (unlikely(!(file->f_mode & FMODE_READ))) > return -EBADF; > - ret = -EINVAL; > if (unlikely(!file->f_op->read_iter)) > return -EINVAL;
Acked-by: Jeff Moyer <[email protected]>
