Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-11 Thread Stefan Hajnoczi
On Mon, Jan 10, 2022 at 07:43:06PM +, Raphael Norwitz wrote: > On Mon, Jan 10, 2022 at 04:36:34AM -0500, Michael S. Tsirkin wrote: > > On Thu, Jan 06, 2022 at 06:47:26AM +, Raphael Norwitz wrote: > > > Signed-off-by: Raphael Norwitz > > > > > > Raphael any chance you can add a bit more

Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-10 Thread Michael S. Tsirkin
On Mon, Jan 10, 2022 at 07:43:06PM +, Raphael Norwitz wrote: > On Mon, Jan 10, 2022 at 04:36:34AM -0500, Michael S. Tsirkin wrote: > > On Thu, Jan 06, 2022 at 06:47:26AM +, Raphael Norwitz wrote: > > > Signed-off-by: Raphael Norwitz > > > > > > Raphael any chance you can add a bit more

Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-10 Thread Raphael Norwitz
On Mon, Jan 10, 2022 at 04:36:34AM -0500, Michael S. Tsirkin wrote: > On Thu, Jan 06, 2022 at 06:47:26AM +, Raphael Norwitz wrote: > > Signed-off-by: Raphael Norwitz > > > Raphael any chance you can add a bit more to commit logs? > E.g. what happens right now if you pass more? > Sure -

Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-10 Thread Michael S. Tsirkin
On Thu, Jan 06, 2022 at 06:47:26AM +, Raphael Norwitz wrote: > Signed-off-by: Raphael Norwitz Raphael any chance you can add a bit more to commit logs? E.g. what happens right now if you pass more? > --- > subprojects/libvhost-user/libvhost-user.c | 6 ++ > 1 file changed, 6

Re: [PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-10 Thread David Hildenbrand
On 06.01.22 07:47, Raphael Norwitz wrote: > Signed-off-by: Raphael Norwitz > --- > subprojects/libvhost-user/libvhost-user.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/subprojects/libvhost-user/libvhost-user.c > b/subprojects/libvhost-user/libvhost-user.c > index

[PATCH v2 1/5] libvhost-user: Add vu_rem_mem_reg input validation

2022-01-05 Thread Raphael Norwitz
Signed-off-by: Raphael Norwitz --- subprojects/libvhost-user/libvhost-user.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/subprojects/libvhost-user/libvhost-user.c b/subprojects/libvhost-user/libvhost-user.c index 787f4d2d4f..a6dadeb637 100644 ---