Re: [spice-gtk v1 2/2] channel-webdav: do not handle vmc_data message twice

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

 



> 
> From: Victor Toso <me@xxxxxxxxxxxxxx>
> 
> This is a regression from commit aae07fb in 2017-03-27 by Victor Toso
> <me@xxxxxxxxxxxxxx>. The message is handled by channel-webdav and
> should not be exposed nor handled elsewhere.
> 
> Found this while inspecting the code, there is not real side effect
> as no one trying to handle port-data from webdav's channel-port.
> 
> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>

Acked

> ---
>  src/channel-webdav.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/channel-webdav.c b/src/channel-webdav.c
> index 195ae34..b5af5a9 100644
> --- a/src/channel-webdav.c
> +++ b/src/channel-webdav.c
> @@ -607,8 +607,10 @@ static void spice_webdav_handle_msg(SpiceChannel
> *channel, SpiceMsgIn *msg)
>  
>      parent_class = SPICE_CHANNEL_CLASS(spice_webdav_channel_parent_class);
>  
> -    if (type == SPICE_MSG_SPICEVMC_DATA)
> +    if (type == SPICE_MSG_SPICEVMC_DATA) {
>          webdav_handle_data_msg(channel, msg);
> +        return;
> +    }
>  
>      /* The only message that we need to handle ourselves is
>      SPICE_MSG_SPICEVMC_DATA
>       * as we want to read it with spice_vmc_input/output_stream to handle
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




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