Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Claudio Fontana
On 16.09.2015 14:51, Paolo Bonzini wrote: > > > On 16/09/2015 13:27, Claudio Fontana wrote: See my answer to Paolo: http://lists.nongnu.org/archive/html/qemu-devel/2015-07/msg05341.html >> Sorry for not noticing the previous discussion.. >> >> Still it would seem more sensible to say ex

Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Paolo Bonzini
On 16/09/2015 13:27, Claudio Fontana wrote: >> > See my answer to Paolo: >> > http://lists.nongnu.org/archive/html/qemu-devel/2015-07/msg05341.html > Sorry for not noticing the previous discussion.. > > Still it would seem more sensible to say explicitly how big the field is I > think, > especi

Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Marc-André Lureau
- Original Message - > On 16.09.2015 11:33, Marc-André Lureau wrote: > > - Original Message - > >> On 15.09.2015 18:07, marcandre.lur...@redhat.com wrote: > >>> From: Marc-André Lureau > >>> > >>> ivshmem_read() only reads sizeof(long) from the input buffer. Accepting > >>> more

Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Claudio Fontana
On 16.09.2015 11:33, Marc-André Lureau wrote: > Hi > > - Original Message - >> On 15.09.2015 18:07, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau >>> >>> ivshmem_read() only reads sizeof(long) from the input buffer. Accepting >>> more could lead to fifo8 abort() on 32bit

Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Marc-André Lureau
Hi - Original Message - > On 15.09.2015 18:07, marcandre.lur...@redhat.com wrote: > > From: Marc-André Lureau > > > > ivshmem_read() only reads sizeof(long) from the input buffer. Accepting > > more could lead to fifo8 abort() on 32bit systems if fifo is not empty. > > > > Signed-off-b

Re: [Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-16 Thread Claudio Fontana
On 15.09.2015 18:07, marcandre.lur...@redhat.com wrote: > From: Marc-André Lureau > > ivshmem_read() only reads sizeof(long) from the input buffer. Accepting > more could lead to fifo8 abort() on 32bit systems if fifo is not empty. > > Signed-off-by: Marc-André Lureau > --- > hw/misc/ivshmem.

[Qemu-devel] [PATCH v3 03/46] ivhsmem: read do not accept more than sizeof(long)

2015-09-15 Thread marcandre . lureau
From: Marc-André Lureau ivshmem_read() only reads sizeof(long) from the input buffer. Accepting more could lead to fifo8 abort() on 32bit systems if fifo is not empty. Signed-off-by: Marc-André Lureau --- hw/misc/ivshmem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw