Re: [Spice-devel] [PATCH spice-server] Avoid client to set too much surfaces

2016-10-27 Thread Jonathon Jongsma
wording suggestion: "display channel: don't let client set too many surfaces Limit the n_surfaces argument to avoid overflowing the surfaces array" I suppose it could also be an assert instead, but this works as well. I notice that red_worker_new() does have the following line:     spice_warn_i

Re: [Spice-devel] [PATCH spice-server] Avoid client to set too much surfaces

2016-10-20 Thread Victor Toso
Hi, On Thu, Oct 20, 2016 at 12:36:50PM +0100, Frediano Ziglio wrote: > Signed-off-by: Frediano Ziglio > --- > server/display-channel.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/server/display-channel.c b/server/display-channel.c > index 69edd35..0b8d6b5 100644 > --

[Spice-devel] [PATCH spice-server] Avoid client to set too much surfaces

2016-10-20 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- server/display-channel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/server/display-channel.c b/server/display-channel.c index 69edd35..0b8d6b5 100644 --- a/server/display-channel.c +++ b/server/display-channel.c @@ -1946,7 +1946,7 @@ Dis