linux-kernel - Re: [LTP] [fs] ce436509a8: ltp.openat203.fail

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: <20200428153022.GC4244@yuki.lan>
Date:   Tue, 28 Apr 2020 17:30:22 +0200
From:   Cyril Hrubis <chrubis@...e.cz>
To:     Aleksa Sarai <cyphar@...har.com>
Cc:     kernel test robot <rong.a.chen@...el.com>,
        Josh Triplett <josh@...htriplett.org>,
        linux-arch@...r.kernel.org, Jens Axboe <axboe@...nel.dk>,
        Arnd Bergmann <arnd@...db.de>, linux-kernel@...r.kernel.org,
        lkp@...ts.01.org, Alexander Viro <viro@...iv.linux.org.uk>,
        linux-fsdevel@...r.kernel.org, io-uring@...r.kernel.org,
        ltp@...ts.linux.it
Subject: Re: [LTP] [fs] ce436509a8: ltp.openat203.fail

Hi!
> > > commit: ce436509a8e109330c56bb4d8ec87d258788f5f4 ("[PATCH v4 2/3] fs: openat2: Extend open_how to allow userspace-selected fds")
> > > url: https://github.com/0day-ci/linux/commits/Josh-Triplett/Support-userspace-selected-fds/20200414-102939
> > > base: https://git.kernel.org/cgit/linux/kernel/git/shuah/linux-kselftest.git next
> > 
> > This commit adds fd parameter to the how structure where LTP test was
> > previously passing garbage, which obviously causes the difference in
> > errno.
> > 
> > This could be safely ignored for now, if the patch gets merged the test
> > needs to be updated.
> 
> It wouldn't be a bad idea to switch the test to figure out the ksize of
> the struct, so that you only add bad padding after that. But then again,
> this would be a bit ugly -- having CHECK_FIELDS would make this simpler.

Any pointers how can be the size figured out without relying on the
E2BIG we are testing for? Does the kernel export it somewhere?

-- 
Cyril Hrubis
chrubis@...e.cz