<div dir="auto">Hi Gao,<div dir="auto"><br></div><div dir="auto">Thanks. and yes, the condition should be 'if (ret)' . Corrected it and few others too. With the new patch I am able to create erofs image. Will send a new patch with bumped up version. </div><div dir="auto"><br></div><div dir="auto">-Pratik</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 24 Jul, 2019, 12:39 PM Gao Xiang, <<a href="mailto:gaoxiang25@huawei.com">gaoxiang25@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Pratik,<br>
<br>
On 2019/7/24 12:55, Pratik Shinde wrote:<br>
> Handling error conditions that are missed in few scenarios.<br>
> also, mkfs command should return 1 on failure and 0 on success.<br>
> <br>
> Signed-off-by: Pratik Shinde <<a href="mailto:pratikshinde320@gmail.com" target="_blank" rel="noreferrer">pratikshinde320@gmail.com</a>><br>
> ---<br>
>  lib/inode.c | 10 ++++++++--<br>
>  mkfs/main.c |  8 +++++++-<br>
>  2 files changed, 15 insertions(+), 3 deletions(-)<br>
> <br>
> diff --git a/lib/inode.c b/lib/inode.c<br>
> index 179aa26..08d38c0 100644<br>
> --- a/lib/inode.c<br>
> +++ b/lib/inode.c<br>
> @@ -752,8 +752,14 @@ struct erofs_inode *erofs_mkfs_build_tree(struct erofs_inode *dir)<br>
>       }<br>
>       closedir(_dir);<br>
>  <br>
> -     erofs_prepare_dir_file(dir);<br>
> -     erofs_prepare_inode_buffer(dir);<br>
> +     ret = erofs_prepare_dir_file(dir);<br>
> +     if(!ret)<br>
> +             goto err_closedir;<br>
<br>
Maybe it should be "if (ret)"? Have you use this patch to generate some images?<br>
<br>
if(!ret)<br>
  ^   --- no space here<br>
<br>
> +<br>
> +     ret = erofs_prepare_inode_buffer(dir);<br>
> +     if(!ret)<br>
> +             goto err_closedir;<br>
<br>
ditto.<br>
<br>
> +<br>
>       if (IS_ROOT(dir))<br>
>               erofs_fixup_meta_blkaddr(dir);<br>
>  <br>
> diff --git a/mkfs/main.c b/mkfs/main.c<br>
> index 1348587..f73eb10 100644<br>
> --- a/mkfs/main.c<br>
> +++ b/mkfs/main.c<br>
> @@ -212,6 +212,12 @@ int main(int argc, char **argv)<br>
>       erofs_show_config();<br>
>  <br>
>       sb_bh = erofs_buffer_init();<br>
> +     if(IS_ERR(sb_bh)) {<br>
> +             err = PTR_ERR(sb_bh);<br>
> +             erofs_err("Failed to initialize super block buffer head : %s",<br>
<br>
erofs_err("Failed to initialize buffers: %s",<br>
<br>
Thanks,<br>
Gao Xiang<br>
<br>
> +                       erofs_strerror(err));<br>
> +             goto exit;<br>
> +     }<br>
>       err = erofs_bh_balloon(sb_bh, EROFS_SUPER_END);<br>
>       if (err < 0) {<br>
>               erofs_err("Failed to balloon erofs_super_block: %s",<br>
> @@ -254,5 +260,5 @@ exit:<br>
>                         erofs_strerror(err));<br>
>               return 1;<br>
>       }<br>
> -     return err;<br>
> +     return 0;<br>
>  }<br>
> <br>
</blockquote></div>