qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 02/12] libvhost-user: fail vu_message_write() if sendmsg()


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v4 02/12] libvhost-user: fail vu_message_write() if sendmsg() is failing
Date: Wed, 8 May 2024 12:23:24 +0200
User-agent: Mozilla Thunderbird

On 8/5/24 09:44, Stefano Garzarella wrote:
In vu_message_write() we use sendmsg() to send the message header,
then a write() to send the payload.

If sendmsg() fails we should avoid sending the payload, since we
were unable to send the header.

Discovered before fixing the issue with the previous patch, where
sendmsg() failed on macOS due to wrong parameters, but the frontend
still sent the payload which the backend incorrectly interpreted
as a wrong header.

Signed-off-by: Stefano Garzarella <sgarzare@redhat.com>
---
  subprojects/libvhost-user/libvhost-user.c | 5 +++++
  1 file changed, 5 insertions(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]