[Spice-devel] [PATCH 1/9] reds: lookup corresponding channel id

Christophe Fergeau cfergeau at redhat.com
Mon Sep 8 02:44:57 PDT 2014


ACK, but please mention this bug in the commit log.

Christophe

On Thu, Aug 28, 2014 at 04:22:43PM +0200, Marc-André Lureau wrote:
> ping
> 
> this patch solves the assert from bug
> https://bugzilla.redhat.com/show_bug.cgi?id=1058625
> 
> 
> On Mon, Nov 18, 2013 at 11:28 AM, Marc-André Lureau <
> marcandre.lureau at gmail.com> wrote:
> 
> > In reds_send_link_ack(), lookup the channel with the same id as the link
> > message.
> > ---
> >  server/reds.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/server/reds.c b/server/reds.c
> > index 2a0002b..146dfc7 100644
> > --- a/server/reds.c
> > +++ b/server/reds.c
> > @@ -1453,7 +1453,8 @@ static int reds_send_link_ack(RedLinkInfo *link)
> >
> >      ack.error = SPICE_LINK_ERR_OK;
> >
> > -    channel = reds_find_channel(link->link_mess->channel_type, 0);
> > +    channel = reds_find_channel(link->link_mess->channel_type,
> > +                                link->link_mess->channel_id);
> >      if (!channel) {
> >          spice_assert(link->link_mess->channel_type == SPICE_CHANNEL_MAIN);
> >          spice_assert(reds->main_channel);
> > --
> > 1.8.3.1
> >
> >
> 
> 
> -- 
> Marc-André Lureau

> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20140908/937981b1/attachment.sig>


More information about the Spice-devel mailing list