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
| ||
|
Date: Mon, 18 Mar 2019 14:00:05 +0100 From: Dmitry Vyukov <dvyukov@...gle.com> To: Stefano Brivio <sbrivio@...hat.com> Cc: Linus Torvalds <torvalds@...ux-foundation.org>, syzbot <syzbot+660883c56e2fa65d4497@...kaller.appspotmail.com>, David Miller <davem@...emloft.net>, Johan Hedberg <johan.hedberg@...il.com>, linux-bluetooth <linux-bluetooth@...r.kernel.org>, Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>, Linux List Kernel Mailing <linux-kernel@...r.kernel.org>, Marcel Holtmann <marcel@...tmann.org>, Michal Marek <mmarek@...e.com>, Netdev <netdev@...r.kernel.org>, syzkaller-bugs <syzkaller-bugs@...glegroups.com> Subject: Re: KASAN: slab-out-of-bounds Read in bacpy On Sun, Mar 17, 2019 at 8:25 PM Stefano Brivio <sbrivio@...hat.com> wrote: > > On Sun, 17 Mar 2019 18:11:49 +0100 > Dmitry Vyukov <dvyukov@...gle.com> wrote: > > > And in this case it seems to be working as intended bisecting it to a > > release tag. > > I guess what went wrong here is that it had to skip quite a few > commits, and the result isn't relevant anymore. Maybe you could improve > this by handling a: > > all runs: boot failed: can't ssh into the instance > > case as bad revision? And, also, if scp times out (as it did on > 912964eacb11), keep retrying instead of marking the revision as good? Skipping revisions does not make the result wrong. In the worst case git bisect will point to a range of commits. But marking non-building/booting revisions as bad can actually diverge bisection in the wrong direction and will result in pointing to completely irrelevant commits. git skip is meant exactly for such cases. And in this case, skipping did not affect the process, it just needed to do few more additional steps.
Powered by blists - more mailing lists