[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 0/5] 9p queue 2022-02-10
|
From: |
Christian Schoenebeck |
|
Subject: |
Re: [PULL 0/5] 9p queue 2022-02-10 |
|
Date: |
Mon, 14 Feb 2022 13:09:48 +0100 |
On Montag, 14. Februar 2022 10:55:17 CET Peter Maydell wrote:
> On Mon, 14 Feb 2022 at 09:47, Christian Schoenebeck
>
> <qemu_oss@crudebyte.com> wrote:
> > So this is about the 'dirent' patch:
> > https://github.com/cschoenebeck/qemu/commit/de19c79dad6a2cad54ae04ce754d47
> > c07bf9bc93
> >
> > In conjunction with the 9p fuzzing tests:
> > https://wiki.qemu.org/Documentation/9p#Fuzzing
> >
> > I first thought it might be a false positive due to the unorthodox
> > handling of dirent duplication by that patch, but from the ASan output
> > below I am not really sure about that.
> >
> > Is there a way to get the content of local variables?
>
> Yes. You can build locally with the clang sanitizers enabled and then
> run under gdb and with the appropriate environment variables to tell the
> sanitizer to abort() on failures.
Well, it does no longer matter for this particular issue here, but it would be
useful if the CI scripts would already dump the local variables to the logs.
E.g. because the patch in question was about system dependant variations.
Another reason is the random data nature of fuzzing tests. Even though the
latter could probably be reproduced with an appropriate seed.
Best regards,
Christian Schoenebeck
- [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/10
- [PULL 4/5] tests/9pfs: Use g_autofree and g_autoptr where possible, Christian Schoenebeck, 2022/02/10
- [PULL 2/5] tests/9pfs: fix mkdir() being called twice, Christian Schoenebeck, 2022/02/10
- [PULL 3/5] tests/9pfs: Fix leak of local_test_path, Christian Schoenebeck, 2022/02/10
- [PULL 5/5] 9pfs: Fix segfault in do_readdir_many caused by struct dirent overread, Christian Schoenebeck, 2022/02/10
- [PULL 1/5] tests/9pfs: use g_autofree where possible, Christian Schoenebeck, 2022/02/10
- Re: [PULL 0/5] 9p queue 2022-02-10, Peter Maydell, 2022/02/13
- Re: [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/14
- Re: [PULL 0/5] 9p queue 2022-02-10, Greg Kurz, 2022/02/14
- Re: [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/14
- Re: [PULL 0/5] 9p queue 2022-02-10, Vitaly Chikunov, 2022/02/14
- Re: [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/14
- Re: [PULL 0/5] 9p queue 2022-02-10, Greg Kurz, 2022/02/15
- Re: [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/16
- Re: [PULL 0/5] 9p queue 2022-02-10, Greg Kurz, 2022/02/16
- Re: [PULL 0/5] 9p queue 2022-02-10, Philippe Mathieu-Daudé, 2022/02/16
- Re: [PULL 0/5] 9p queue 2022-02-10, Vitaly Chikunov, 2022/02/16
- Re: [PULL 0/5] 9p queue 2022-02-10, Christian Schoenebeck, 2022/02/16