Re: [Spice-devel] spice-gtk: Fix error handling in stream_get_current_frame()

2015-11-04 Thread Christophe Fergeau
Hey, On Fri, Oct 30, 2015 at 12:17:30PM +0100, Francois Gouget wrote: > *data must always be set to NULL on error. Looks good, ACK. Christophe > > Signed-off-by: Francois Gouget > --- > src/channel-display.c | 17 + > 1 file changed, 9 insertions(+),

Re: [Spice-devel] spice-gtk: Fix error handling in stream_get_current_frame()

2015-11-04 Thread Christophe Fergeau
On Wed, Nov 04, 2015 at 12:04:57PM +0100, Christophe Fergeau wrote: > Hey, > > On Fri, Oct 30, 2015 at 12:17:30PM +0100, Francois Gouget wrote: > > *data must always be set to NULL on error. > > Looks good, ACK. I've pushed this. Christophe signature.asc Description: PGP signature

Re: [Spice-devel] spice-gtk: Fix error handling in stream_get_current_frame()

2015-11-03 Thread Francois Gouget
Does this patch need changes? On Fri, 30 Oct 2015, Francois Gouget wrote: > *data must always be set to NULL on error. > > Signed-off-by: Francois Gouget > --- > src/channel-display.c | 17 + > 1 file changed, 9 insertions(+), 8 deletions(-) > >

[Spice-devel] spice-gtk: Fix error handling in stream_get_current_frame()

2015-10-30 Thread Francois Gouget
*data must always be set to NULL on error. Signed-off-by: Francois Gouget --- src/channel-display.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/channel-display.c b/src/channel-display.c index 46e9829..9e42dd9 100644 ---