[Spice-devel] [PATCH spice-gtk 1/4] channel-main: Fix dangling references to freed file-xfer-tasks on agent cancel

Christophe Fergeau cfergeau at redhat.com
Mon Mar 4 23:31:17 PST 2013


On Mon, Mar 04, 2013 at 07:46:09PM +0100, Hans de Goede wrote:
> On 03/04/2013 04:59 PM, Christophe Fergeau wrote:
> >I don't know how many elements are typically in this loop, but calling
> >g_slist_remove from a g_list_foreach callback will give you quadratic
> >behaviour. Doing the iteration by hand could probably help avoiding this,
> >but if this list typically has few elements, it's probably not worth the
> >trouble.
> 
> The list typically has few elements, and there will be only one flush in
> the list per task, so g_slist_remove will be called once (or not at all).

Ok, sounds good then, ACK

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20130305/4c04fde6/attachment.pgp>


More information about the Spice-devel mailing list