Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-12 Thread Manos Pitsidianakis
On Wed, 12 Jun 2024 11:56, Alex Bennée wrote: Manos Pitsidianakis writes: On Tue, 11 Jun 2024 at 18:01, Philippe Mathieu-Daudé wrote: On 11/6/24 14:23, Manos Pitsidianakis wrote: > A fuzzer case discovered by Zheyu Ma causes an assert failure. > > Add a check before the assert, and respond

Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-12 Thread Manos Pitsidianakis
On Wed, 12 Jun 2024 12:46, Alex Bennée wrote: Manos Pitsidianakis writes: A fuzzer case discovered by Zheyu Ma causes an assert failure. Add a check before the assert, and respond with an error before moving on to the next queue element. To reproduce the failure: cat << EOF | \ qemu-system

Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-12 Thread Alex Bennée
Manos Pitsidianakis writes: > A fuzzer case discovered by Zheyu Ma causes an assert failure. > > Add a check before the assert, and respond with an error before moving > on to the next queue element. > > To reproduce the failure: > > cat << EOF | \ > qemu-system-x86_64 \ > -display none -machine

Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-12 Thread Alex Bennée
Manos Pitsidianakis writes: > On Tue, 11 Jun 2024 at 18:01, Philippe Mathieu-Daudé > wrote: >> >> On 11/6/24 14:23, Manos Pitsidianakis wrote: >> > A fuzzer case discovered by Zheyu Ma causes an assert failure. >> > >> > Add a check before the assert, and respond with an error before moving >>

Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-11 Thread Manos Pitsidianakis
On Tue, 11 Jun 2024 at 18:01, Philippe Mathieu-Daudé wrote: > > On 11/6/24 14:23, Manos Pitsidianakis wrote: > > A fuzzer case discovered by Zheyu Ma causes an assert failure. > > > > Add a check before the assert, and respond with an error before moving > > on to the next queue element. > > > > T

Re: [PATCH v1] virtio-iommu: add error check before assert

2024-06-11 Thread Philippe Mathieu-Daudé
On 11/6/24 14:23, Manos Pitsidianakis wrote: A fuzzer case discovered by Zheyu Ma causes an assert failure. Add a check before the assert, and respond with an error before moving on to the next queue element. To reproduce the failure: cat << EOF | \ qemu-system-x86_64 \ -display none -machine

[PATCH v1] virtio-iommu: add error check before assert

2024-06-11 Thread Manos Pitsidianakis
A fuzzer case discovered by Zheyu Ma causes an assert failure. Add a check before the assert, and respond with an error before moving on to the next queue element. To reproduce the failure: cat << EOF | \ qemu-system-x86_64 \ -display none -machine accel=qtest -m 512M -machine q35 -nodefaults \