On 11/1/24 7:42 PM, Ming Lei wrote:
> .bi_size of bvec iterator should be initialized as real max size for
> walking, and .bi_bvec_done just counts how many bytes need to be
> skipped in the 1st bvec, so .bi_size isn't related with .bi_bvec_done.
>
> This patch fixes bvec iterator initialization, and the inner `size`
> check isn't needed any more, so revert Eric Dumazet's commit
> 7bc802acf193 ("iov-iter: do not return more bytes than requested in
> iov_iter_extract_bvec_pages()").
>
> Cc: Eric Dumazet <edumazet@xxxxxxxxxx>
> Fixes: e4e535bff2bc ("iov_iter: don't require contiguous pages in iov_iter_extract_bvec_pages")
> Reported-by: syzbot+71abe7ab2b70bca770fd@xxxxxxxxxxxxxxxxxxxxxxxxx
> Tested-by: syzbot+71abe7ab2b70bca770fd@xxxxxxxxxxxxxxxxxxxxxxxxx
> Signed-off-by: Ming Lei <ming.lei@xxxxxxxxxx>
> ---
> Hi Jens,
>
> If possible, please merge this one with Eric's commit.
I can just swap them out.
--
Jens Axboe
