Re: [PATCH v2 2/2] exfat: check if cluster num is valid

To: Namjae Jeon <linkinjeon@xxxxxxxxxx>
Subject: Re: [PATCH v2 2/2] exfat: check if cluster num is valid
From: Tadeusz Struk <tadeusz.struk@xxxxxxxxxx>
Date: Mon, 16 May 2022 16:38:59 -0700
Cc: Sungjong Seo <sj1557.seo@xxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, stable@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, syzbot+a4087e40b9c13aad7892@xxxxxxxxxxxxxxxxxxxxxxxxx
In-reply-to: <CAKYAXd_kcZF0tHMX_CsR83qmX25PhdGQPJibMh1-30=5przrjQ@mail.gmail.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.0

On 5/16/22 16:31, Namjae Jeon wrote:

Looks good.
And it seems that WARN_ON() is no longer needed.

Right. Do you want me to send a follow up patch that drops the WARN_ONs?

You don't need to do it. I have applied this patch to #exfat dev
branch after removing it.
Note that I have combined 1/2 into 2/2 patch.
Thanks!


That's fine with me. Thanks for taking it.

--
Thanks,
Tadeusz

References:
- [PATCH v2 1/2] exfat: move is_valid_cluster to a common header
  - From: Tadeusz Struk
- [PATCH v2 2/2] exfat: check if cluster num is valid
  - From: Tadeusz Struk
- RE: [PATCH v2 2/2] exfat: check if cluster num is valid
  - From: Sungjong Seo
- Re: [PATCH v2 2/2] exfat: check if cluster num is valid
  - From: Tadeusz Struk
- Re: [PATCH v2 2/2] exfat: check if cluster num is valid
  - From: Namjae Jeon