Re: [Spice-devel] [PATCH] dcc: remove possible dandling pointers

2015-11-25 Thread Frediano Ziglio
> > Can we fix the typo in the subject though? dandling -> dangling > Sorry, was already pushed. I'll have to remind this I really thought spelling was right with d! Frediano > > On Tue, 2015-11-24 at 17:42 +0100, Pavel Grunt wrote: > > On Tue, 2015-11-24 at 16:17 +, Frediano Ziglio wro

Re: [Spice-devel] [PATCH] dcc: remove possible dandling pointers

2015-11-24 Thread Jonathon Jongsma
Can we fix the typo in the subject though? dandling -> dangling On Tue, 2015-11-24 at 17:42 +0100, Pavel Grunt wrote: > On Tue, 2015-11-24 at 16:17 +, Frediano Ziglio wrote: > > Set pointers to NULL after releasing encoders. > > This to make sure no further access will occurs. > > > It is no

Re: [Spice-devel] [PATCH] dcc: remove possible dandling pointers

2015-11-24 Thread Pavel Grunt
On Tue, 2015-11-24 at 16:17 +, Frediano Ziglio wrote: > Set pointers to NULL after releasing encoders. > This to make sure no further access will occurs. > It is not a harm, Acked-by: Pavel Grunt > Signed-off-by: Frediano Ziglio > --- >  server/dcc-encoders.c | 5 + >  1 file changed, 5

Re: [Spice-devel] [PATCH] dcc: remove possible dandling pointers

2015-11-24 Thread Victor Toso
Hi, On Tue, Nov 24, 2015 at 04:17:36PM +, Frediano Ziglio wrote: > Set pointers to NULL after releasing encoders. > This to make sure no further access will occurs. > > Signed-off-by: Frediano Ziglio > --- > server/dcc-encoders.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git

[Spice-devel] [PATCH] dcc: remove possible dandling pointers

2015-11-24 Thread Frediano Ziglio
Set pointers to NULL after releasing encoders. This to make sure no further access will occurs. Signed-off-by: Frediano Ziglio --- server/dcc-encoders.c | 5 + 1 file changed, 5 insertions(+) diff --git a/server/dcc-encoders.c b/server/dcc-encoders.c index 1eaa604..a373412 100644 --- a/serv