Reviewed-by Joao Marcos Costa <jmcosta...@gmail.com>
Em ter., 3 de nov. de 2020 às 08:12, Richard Genoud <
richard.gen...@posteo.net> escreveu:
> sqfs_decompressor_init() returns a value, so it's better to use it than
> to force the return value to EINVAL (it could be ENOMEM)
>
> Signed-off-by: Richard Genoud <richard.gen...@posteo.net>
> ---
> fs/squashfs/sqfs.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/fs/squashfs/sqfs.c b/fs/squashfs/sqfs.c
> index 9682ea52557..fa3120aefd6 100644
> --- a/fs/squashfs/sqfs.c
> +++ b/fs/squashfs/sqfs.c
> @@ -1098,9 +1098,7 @@ int sqfs_probe(struct blk_desc *fs_dev_desc, struct
> disk_partition *fs_partition
> ctxt.sblk = sblk;
>
> ret = sqfs_decompressor_init(&ctxt);
> -
> if (ret) {
> - ret = -EINVAL;
> goto error;
> }
>
>
--
Atenciosamente,
João Marcos Costa
www.linkedin.com/in/jmarcoscosta/
https://github.com/jmarcoscosta
