[Spice-devel] [PATCH 13/15] Change reds_get_client() to take RedsState arg

[Frediano Ziglio]

> 
> From: Jonathon Jongsma <jjongsma at redhat.com>
> 
> ---
>  server/reds.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 6d34554..fdf2be8 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -1639,7 +1639,7 @@ static int reds_find_client(RedsState *reds, RedClient
> *client)
>  }
>  
>  /* should be used only when there is one client */
> -static RedClient *reds_get_client(void)
> +static RedClient *reds_get_client(RedsState *reds)
>  {
>      spice_assert(reds->num_clients <= 1);
>  
> @@ -2908,7 +2908,7 @@ static void reds_migrate_channels_seamless(void)
>      RedClient *client;
>  
>      /* seamless migration is supported for only one client for now */
> -    client = reds_get_client();
> +    client = reds_get_client(reds);
>      red_client_migrate(client);
>  }
>  
> @@ -3022,11 +3022,11 @@ static SpiceCharDeviceState
> *attach_to_red_agent(SpiceCharDeviceInstance *sin)
>           * 2.b If this happens second ==> we already have spice migrate data
>           *     then restore state
>           */
> -        if (!spice_char_device_client_exists(reds->agent_state.base,
> reds_get_client())) {
> +        if (!spice_char_device_client_exists(reds->agent_state.base,
> reds_get_client(reds))) {
>              int client_added;
>  
>              client_added =
>              spice_char_device_client_add(reds->agent_state.base,
> -                                                        reds_get_client(),
> +
> reds_get_client(reds),
>                                                          TRUE, /* flow
>                                                          control */
>                                                          REDS_VDI_PORT_NUM_RECEIVE_BUFFS,
>                                                          REDS_AGENT_WINDOW_SIZE,

Acked-by: Frediano Ziglio <fziglio at redhat.com>

Frediano