Re: [Spice-devel] [PATCH 1/2] Add log for invalid/expired tickets

2012-03-05 Thread Christophe Fergeau
On Sun, Mar 04, 2012 at 11:59:48AM +0200, Yaniv Kaul wrote: > On 03/02/2012 05:52 PM, Christophe Fergeau wrote: > >Currently, when a ticket has already expired, or is invalid, there is > >no qemu log to tell what went wrong. This commit adds such a log. > > > >Fixes rhbz#787669 > > How 'heavy' is

Re: [Spice-devel] [PATCH 1/2] Add log for invalid/expired tickets

2012-03-04 Thread Yaniv Kaul
On 03/02/2012 05:52 PM, Christophe Fergeau wrote: Currently, when a ticket has already expired, or is invalid, there is no qemu log to tell what went wrong. This commit adds such a log. Fixes rhbz#787669 How 'heavy' is this red_printf? Can this cause DoS (if there's no print limiting to it) ?

Re: [Spice-devel] [PATCH 1/2] Add log for invalid/expired tickets

2012-03-02 Thread Alon Levy
On Fri, Mar 02, 2012 at 04:52:42PM +0100, Christophe Fergeau wrote: > Currently, when a ticket has already expired, or is invalid, there is > no qemu log to tell what went wrong. This commit adds such a log. ACK series. > > Fixes rhbz#787669 > --- > server/reds.c |5 + > 1 files changed

[Spice-devel] [PATCH 1/2] Add log for invalid/expired tickets

2012-03-02 Thread Christophe Fergeau
Currently, when a ticket has already expired, or is invalid, there is no qemu log to tell what went wrong. This commit adds such a log. Fixes rhbz#787669 --- server/reds.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/server/reds.c b/server/reds.c index 797d9d5..3a9