Re: [PATCH 1/9] reds: lookup corresponding channel id

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

 



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@xxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgp9gdSn_AxM5.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]