linux-kernel - Re: [PATCH] fuse: BUG_ON's correction in fuse_dev_splice

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJfpegv-EQhvJUB0AUhJ=Xx8moHHQvkDGe-yUXHENyWvboBU3A@mail.gmail.com>
Date:   Thu, 1 Aug 2019 13:01:33 +0200
From:   Miklos Szeredi <miklos@...redi.hu>
To:     Vasily Averin <vvs@...tuozzo.com>
Cc:     linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org,
        Andrey Ryabinin <aryabinin@...tuozzo.com>
Subject: Re: [PATCH] fuse: BUG_ON's correction in fuse_dev_splice_write()

On Tue, Jul 23, 2019 at 8:33 AM Vasily Averin <vvs@...tuozzo.com> wrote:
>
> commit 963545357202 ("fuse: reduce allocation size for splice_write")
> changed size of bufs array, so first BUG_ON should be corrected too.
> Second BUG_ON become useless, first one also includes the second check:
> any unsigned nbuf value cannot be less than 0.

This patch seems broken: it assumes that pipe->nrbufs doesn't change.
Have you actually tested it?

Thanks,
Miklos