Re: [Qemu-devel] [PATCH v2] 9pfs: proxy: assert if unmarshal fails

2017-03-17 Thread Greg Kurz
On Fri, 17 Mar 2017 12:06:39 -0300 Philippe Mathieu-Daudé wrote: > On 03/17/2017 11:43 AM, Daniel P. Berrange wrote: > > On Fri, Mar 17, 2017 at 03:39:10PM +0100, Greg Kurz wrote: > >> Replies from the virtfs proxy are made up of a fixed-size header (8 bytes) > >> and a payload of variable size

Re: [Qemu-devel] [PATCH v2] 9pfs: proxy: assert if unmarshal fails

2017-03-17 Thread Greg Kurz
On Fri, 17 Mar 2017 14:43:00 + "Daniel P. Berrange" wrote: > On Fri, Mar 17, 2017 at 03:39:10PM +0100, Greg Kurz wrote: > > Replies from the virtfs proxy are made up of a fixed-size header (8 bytes) > > and a payload of variable size (maximum 64kb). When receiving a reply, > > the proxy backe

Re: [Qemu-devel] [PATCH v2] 9pfs: proxy: assert if unmarshal fails

2017-03-17 Thread Philippe Mathieu-Daudé
On 03/17/2017 11:43 AM, Daniel P. Berrange wrote: On Fri, Mar 17, 2017 at 03:39:10PM +0100, Greg Kurz wrote: Replies from the virtfs proxy are made up of a fixed-size header (8 bytes) and a payload of variable size (maximum 64kb). When receiving a reply, the proxy backend first reads the whole h

Re: [Qemu-devel] [PATCH v2] 9pfs: proxy: assert if unmarshal fails

2017-03-17 Thread Daniel P. Berrange
On Fri, Mar 17, 2017 at 03:39:10PM +0100, Greg Kurz wrote: > Replies from the virtfs proxy are made up of a fixed-size header (8 bytes) > and a payload of variable size (maximum 64kb). When receiving a reply, > the proxy backend first reads the whole header and then unmarshals it. > If the header i

[Qemu-devel] [PATCH v2] 9pfs: proxy: assert if unmarshal fails

2017-03-17 Thread Greg Kurz
Replies from the virtfs proxy are made up of a fixed-size header (8 bytes) and a payload of variable size (maximum 64kb). When receiving a reply, the proxy backend first reads the whole header and then unmarshals it. If the header is okay, it then does the same operation with the payload. Since th