Re: [PATCH] net: use peer when purging queue in qemu_flush_or_purge_queue_packets()

2020-05-18 Thread Jason Wang
On 2020/5/18 下午4:00, Philippe Mathieu-Daudé wrote: Hi Jason, On 5/18/20 5:34 AM, Jason Wang wrote: On 2020/5/11 下午12:21, Alexander Bulekov wrote: On 200511 1204, Jason Wang wrote: The sender of packet will be checked in the qemu_net_queue_purge() but we use NetClientState not its peer when

Re: [PATCH] net: use peer when purging queue in qemu_flush_or_purge_queue_packets()

2020-05-18 Thread Philippe Mathieu-Daudé
Hi Jason, On 5/18/20 5:34 AM, Jason Wang wrote: On 2020/5/11 下午12:21, Alexander Bulekov wrote: On 200511 1204, Jason Wang wrote: The sender of packet will be checked in the qemu_net_queue_purge() but we use NetClientState not its peer when trying to purge the incoming queue in qemu_flush_or_pu

Re: [PATCH] net: use peer when purging queue in qemu_flush_or_purge_queue_packets()

2020-05-17 Thread Jason Wang
On 2020/5/11 下午12:21, Alexander Bulekov wrote: On 200511 1204, Jason Wang wrote: The sender of packet will be checked in the qemu_net_queue_purge() but we use NetClientState not its peer when trying to purge the incoming queue in qemu_flush_or_purge_packets(). This will trigger the assert in v

Re: [PATCH] net: use peer when purging queue in qemu_flush_or_purge_queue_packets()

2020-05-10 Thread Alexander Bulekov
On 200511 1204, Jason Wang wrote: > The sender of packet will be checked in the qemu_net_queue_purge() but > we use NetClientState not its peer when trying to purge the incoming > queue in qemu_flush_or_purge_packets(). This will trigger the assert > in virtio_net_reset since we can't pass the send

[PATCH] net: use peer when purging queue in qemu_flush_or_purge_queue_packets()

2020-05-10 Thread Jason Wang
The sender of packet will be checked in the qemu_net_queue_purge() but we use NetClientState not its peer when trying to purge the incoming queue in qemu_flush_or_purge_packets(). This will trigger the assert in virtio_net_reset since we can't pass the sender check. Fix by using the peer. Reporte