Re: [PATCH spice-server 07/10] red-channel-client: Avoid weird memory references using MarkerPipeItem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Tue, Sep 12, 2017 at 04:15:15AM -0400, Frediano Ziglio wrote:
> > > 
> > > On Mon, Sep 11, 2017 at 11:15:44AM +0100, Frediano Ziglio wrote:
> > > > Instead of having MarkerPipeItem pointing to a variable on an external
> > > > stuff with the possibility to forget to reset it or having possibly
> > > > dangling pointers use reference counting to keep the item and
> > > > mark the item when sent.
> > > 
> > > Look good to me, I would have added some more details in the log (that
> > > this stores a reference to an external variable because we need to know
> > > its value after the item has been processed and freed), this makes it
> > > easier to understand why you want reference counting.
> > > 
> > > Christophe
> > > 
> > 
> > How does it sound adding:
> > 
> > "Storing a pointer to an external variable was used to be able to use
> > this variable after the object was freed."
> 
> I probably would detail this even more.
> 

Suggestions?

> Christophe
> 

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]